There is use case for the '#' pound sign.
As my personal use case goes i have a directory of movie franchises in a glossary view (from views, terms are taxonomy terms) and each "group in the glossary heading" has an anchor which uses the '#' pound sign.
So the pathauto term URL should for example for ghostbusters read "/cinema/by-franchise/g#ghostbusters" (the g is a field on the term page as a truncated first letter of the term name)Even i go to the settings and change the setting for '#' under punctuation to not alter '#' it still says "invalid character for the above URL.
- 🇮🇳India mohit_aghera Rajkot
I've addressed the feedback in #40
I think node tests aren't required as such.Fixed a few more things in the PR
- convert it to use OOP hooks
- Fixed a few things in tests.
- Tests seems green now.Hiding all the patches in favour of MR.
- @mohit_aghera opened merge request.
- 🇦🇺Australia griffynh Sydney
As we haven't had an update for eight years—I'm going to close this issue.
Please feel free to reopen if there is any new information.
- 🇺🇸United States smustgrave
Sounds like there may be more discussion for this. Could someone update the issue summary for how this could be an improvement? Not sure PNMI is the right status but NR didn't spark any conversation. So figured i'd put there before it gets closed out.
- @atul_ghate opened merge request.
- 🇩🇪Germany Anybody Porta Westfalica
Finally I gave up the hope that this will ever be finished by anyone (including me). So maybe we should better put our time into implementing a contrib solution?
Maybe in https://www.drupal.org/project/theme_tokens → ?
Seems there are enough people here to do that... - 🇺🇸United States smustgrave
Only did a light review but you get the flow.
Overall looks good though.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
avpaderno → made their first commit to this issue’s fork.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
I apologize: I saw the comments on the merge request, but I was not able to find this issue when I was ready to change the merge request.