Merge pull request #287 from IbeVanmeenen/issue_280

[AdminBundle] Make it possible to open a link in new tab in the sidebar
Kunstmaan · Apr 8, 2015 · 620d8e2 · 620d8e2
2 parents a5b12a2 + 05aaa3b
commit 620d8e2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/src/Kunstmaan/AdminBundle/Resources/ui/js/_app-sidebar-tree.js b/src/Kunstmaan/AdminBundle/Resources/ui/js/_app-sidebar-tree.js
@@ -29,7 +29,11 @@ kunstmaanbundles.sidebartree = (function($, window, undefined) {
             // DOCS: http://www.jstree.com/api/#/?q=.jstree%20Event&f=changed.jstree
             var href = data.event.currentTarget.href;
 
-            document.location.href = href;
+            if (data.event.ctrlKey || data.event.metaKey) {
+                window.open(href);
+            } else {
+                document.location.href = href;
+            }
         });
 
 

diff --git a/src/Kunstmaan/AdminBundle/Resources/ui/js/_page-editor.js b/src/Kunstmaan/AdminBundle/Resources/ui/js/_page-editor.js
@@ -304,11 +304,11 @@ kunstmaanbundles.pageEditor = (function(window, undefined) {
 
     // Key Combinations
     keyCombinations = function() {
-    $(document).on('keydown', function(e) {
-        if((e.ctrlKey || e.metaKey) && e.which === 83) {
-            e.preventDefault();
+        $(document).on('keydown', function(e) {
+            if((e.ctrlKey || e.metaKey) && e.which === 83) {
+                e.preventDefault();
 
-            $body.addClass('app--loading');
+                $body.addClass('app--loading');
                 $('#pageadminform').submit();
             };
         });