Found and fixed pattern-lab/patternlab-php#277 within PL Node

Keyboard support :) Closes #107
standardized postMessage across some usages. IE10 and up seemed fine. closes #108

Author: bmuenzenmeyer

config.json

@@ -25,7 +25,7 @@
  		"tools-all": true,
  		"tools-follow": false,
  		"tools-reload": false,
- 		"tools-shortcuts": false,
+ 		"tools-shortcuts": true,
  		"tools-docs": true
  	},
   "patternStates": {

public/styleguide/js/postmessage.js

@@ -1,7 +1,7 @@
 /*!
  * Basic postMessage Support - v0.1
  *
- * Copyright (c) 2013 Dave Olsen, http://dmolsen.com
+ * Copyright (c) 2013-2014 Dave Olsen, http://dmolsen.com
  * Licensed under the MIT license
  *
  * Handles the postMessage stuff in the pattern, view-all, and style guide templates.
@@ -15,9 +15,11 @@ if (self != top) {
 	//   - all get path
 	//   - pattern & view all get a pattern partial, styleguide gets all
 	//   - pattern shares lineage
-	var options = { "path": window.location.toString() };
-	options.patternpartial = (patternPartial != "") ? patternPartial : "all";
-	if (lineage != "") {
+	var path = window.location.toString();
+	var parts = path.split("?");
+	var options = { "path": parts[0] };
+	options.patternpartial = (patternPartial !== "") ? patternPartial : "all";
+	if (lineage !== "") {
 		options.lineage = lineage;
 	}
 
@@ -26,20 +28,22 @@ if (self != top) {
 
 	// find all links and add an onclick handler for replacing the iframe address so the history works
 	var aTags = document.getElementsByTagName('a');
-	for (a in aTags) {
-		aTags[a].onclick = function(e) {
+	for (var i = 0; i < aTags.length; i++) {
+		aTags[i].onclick = function(e) {
 			e.preventDefault();
-			window.location.replace(this.getAttribute("href"));
+			var href = this.getAttribute("href");
+			if (href != "#") {
+				window.location.replace(href);
+			}
 		};
 	}
-
+	
 	// bind the keyboard shortcuts for various viewport resizings + pattern search
 	var keys = [ "s", "m", "l", "d", "h", "f" ];
 	for (var i = 0; i < keys.length; i++) {
 		jwerty.key('ctrl+shift+'+keys[i],  function (k,t) {
 			return function(e) {
-				var obj = JSON.stringify({ "keyPress": "ctrl+shift+"+k });
-				parent.postMessage(obj,t);
+				parent.postMessage({ "keyPress": "ctrl+shift+"+k },t);
 				return false;
 			}
 		}(keys[i],targetOrigin));
@@ -51,8 +55,7 @@ if (self != top) {
 		jwerty.key('ctrl+shift+'+i, function (k,t) {
 			return function(e) {
 				var targetOrigin = (window.location.protocol == "file:") ? "*" : window.location.protocol+"//"+window.location.host;
-				var obj = JSON.stringify({ "keyPress": "ctrl+shift+"+k });
-				parent.postMessage(obj,t);
+				parent.postMessage({ "keyPress": "ctrl+shift+"+k },t);
 				return false;
 			}
 		}(i,targetOrigin));
@@ -65,40 +68,44 @@ if (self != top) {
 var body = document.getElementsByTagName('body');
 body[0].onclick = function() {
 	var targetOrigin = (window.location.protocol == "file:") ? "*" : window.location.protocol+"//"+window.location.host;
-	parent.postMessage( { "bodyclick": "bodyclick" }, targetOrigin)
+	parent.postMessage({ "bodyclick": "bodyclick" },targetOrigin);
 };
 
 // watch the iframe source so that it can be sent back to everyone else.
 function receiveIframeMessage(event) {
-
+	
+	var path;
+	var data = (typeof event.data !== "string") ? event.data : JSON.parse(event.data);
+	
 	// does the origin sending the message match the current host? if not dev/null the request
 	if ((window.location.protocol != "file:") && (event.origin !== window.location.protocol+"//"+window.location.host)) {
 		return;
 	}
 
 	// see if it got a path to replace
-	if (event.data.path != undefined) {
+	if (data.path !== undefined) {
 
-		if (patternPartial != "") {
+		if (patternPartial !== "") {
 
 			// handle patterns and the view all page
-			var re   = /patterns\/(.*)$/;
-			var path = window.location.protocol+"//"+window.location.host+window.location.pathname.replace(re,'')+event.data.path;
+			var re = /patterns\/(.*)$/;
+			path = window.location.protocol+"//"+window.location.host+window.location.pathname.replace(re,'')+data.path+'?'+Date.now();
 			window.location.replace(path);
 
 		} else {
 
 			// handle the style guide
-			var path = window.location.protocol+"//"+window.location.host+window.location.pathname.replace("styleguide\/html\/styleguide.html","")+event.data.path;
+			path = window.location.protocol+"//"+window.location.host+window.location.pathname.replace("styleguide\/html\/styleguide.html","")+data.path+'?'+Date.now();
 			window.location.replace(path);
 
 		}
 
-	} else if (event.data.reload != undefined) {
+	} else if (data.reload !== undefined) {
 
 		// reload the location if there was a message to do so
 		window.location.reload();
+		
 	}
 
 }
-window.addEventListener("message", receiveIframeMessage, false);
+window.addEventListener("message", receiveIframeMessage, false);