You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by br...@apache.org on 2016/01/06 16:12:43 UTC

[1/2] allura git commit: [#8010] ticket:885 Cleanup js which used .title-pane

Repository: allura
Updated Branches:
  refs/heads/master 996d82b12 -> 19c532faa


[#8010] ticket:885 Cleanup js which used .title-pane


Project: http://git-wip-us.apache.org/repos/asf/allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/allura/commit/19c532fa
Tree: http://git-wip-us.apache.org/repos/asf/allura/tree/19c532fa
Diff: http://git-wip-us.apache.org/repos/asf/allura/diff/19c532fa

Branch: refs/heads/master
Commit: 19c532faa724ac684e18dd88c51deac8d11e0863
Parents: 904ca25
Author: Denis Kotov <de...@gmail.com>
Authored: Wed Jan 6 00:02:30 2016 +0200
Committer: Dave Brondsema <da...@brondsema.net>
Committed: Wed Jan 6 10:12:16 2016 -0500

----------------------------------------------------------------------
 Allura/allura/public/nf/js/allura-base.js | 24 ------------------------
 1 file changed, 24 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/allura/blob/19c532fa/Allura/allura/public/nf/js/allura-base.js
----------------------------------------------------------------------
diff --git a/Allura/allura/public/nf/js/allura-base.js b/Allura/allura/public/nf/js/allura-base.js
index 573448d..52d6cb5 100644
--- a/Allura/allura/public/nf/js/allura-base.js
+++ b/Allura/allura/public/nf/js/allura-base.js
@@ -18,30 +18,6 @@
 */
 
 (function($) {
-    // Setup title-pane widgets
-    $('.title-pane .title').click(function(e) {
-        e.preventDefault();
-        $(this).closest('.title-pane')
-            .find('> .content').toggle('fast', function() {
-                $(this)
-                    .closest('.title-pane').toggleClass('closed').end()
-                    .toggleClass('hidden');
-            });
-    });
-    if(window.location.hash) {
-        // Nested comment (reply) hash link contains a /, which must be escaped
-        var splited_link = window.location.hash.split('/'),
-            united_link = "";
-        for (var i = 0; i < splited_link.length; i++) {
-            if (i == 0) {
-                united_link += splited_link[i];
-            } else {
-                united_link += "\\/" + splited_link[i];
-            }
-        }
-        $(united_link + '.title-pane').removeClass('closed');
-    }
-
     // Setup editable widgets
     $('div.editable, span.editable, h1.editable')
         .find('.viewer')


[2/2] allura git commit: [#8010] ticket:876 Fix link hash escaping for nested comments

Posted by br...@apache.org.
[#8010] ticket:876 Fix link hash escaping for nested comments


Project: http://git-wip-us.apache.org/repos/asf/allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/allura/commit/904ca25f
Tree: http://git-wip-us.apache.org/repos/asf/allura/tree/904ca25f
Diff: http://git-wip-us.apache.org/repos/asf/allura/diff/904ca25f

Branch: refs/heads/master
Commit: 904ca25f7a3c86a456f38a5b8286efa72a90fa6b
Parents: 996d82b
Author: DeV1doR <de...@ukr.net>
Authored: Fri Dec 11 17:21:53 2015 +0200
Committer: Dave Brondsema <da...@brondsema.net>
Committed: Wed Jan 6 10:12:16 2016 -0500

----------------------------------------------------------------------
 Allura/allura/public/nf/js/allura-base.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/allura/blob/904ca25f/Allura/allura/public/nf/js/allura-base.js
----------------------------------------------------------------------
diff --git a/Allura/allura/public/nf/js/allura-base.js b/Allura/allura/public/nf/js/allura-base.js
index 3f6d45e..573448d 100644
--- a/Allura/allura/public/nf/js/allura-base.js
+++ b/Allura/allura/public/nf/js/allura-base.js
@@ -30,7 +30,16 @@
     });
     if(window.location.hash) {
         // Nested comment (reply) hash link contains a /, which must be escaped
-        $(window.location.hash.replace('/', '\\/') + '.title-pane').removeClass('closed');
+        var splited_link = window.location.hash.split('/'),
+            united_link = "";
+        for (var i = 0; i < splited_link.length; i++) {
+            if (i == 0) {
+                united_link += splited_link[i];
+            } else {
+                united_link += "\\/" + splited_link[i];
+            }
+        }
+        $(united_link + '.title-pane').removeClass('closed');
     }
 
     // Setup editable widgets