Pseudo anchors should allow whitespace and special characters

[Ryan J Ollos]

It doesn't seem to be possible to use whitespace or special characters in a pseudo anchor (#10284).

For example, I've wanted to have an anchor to class Environment(Component) in trunk/trac/env.py​.

• trunk/trac/env.py#/Environment​ anchors to an earlier instance of the Environment token
• None of the following work:
  • trunk/trac/env.py#/class%20Environment​
  • trunk/trac/env.py#/Environment(​
  • trunk/trac/env.py#/Environment%28​
  • trunk/trac/env.py#/'Environment('​
  • trunk/trac/env.py#/"Environment("​

[Hong Choi <nik@…>]

Personally, as far as trac is passing search terms via uri, the first method - uri encoding - is the most widely compatible way. So, here's the clue for that.

trac/htdocs/js/search.js

@@ -66 +66 @@
         var direction = h[1];
         var case_insensitive = h.match(/\/i$/);
         if (direction == '/' || direction == '?') {
-          var hterm = h.substr(2);
+          var hterm = decodeURIComponent(h.substr(2));
           if (case_insensitive)
             hterm = hterm.substr(0, hterm.length - 2).toLowerCase();
           $('.searchword0').each(function() {

[Jun Omae]

At least, there is no cross-browser solution in past years. We should verify the proposed changes work on supported browsers.

FYI: ​javascript - Encoding of window.location.hash - Stack Overflow

[Ryan J Ollos]

Milestone renamed