Github user michellephung commented on the pull request:
https://github.com/apache/couchdb-fauxton/pull/319#issuecomment-88188557
@robertkowalski yikes! that's quite the test case :D
I can write the unit test, but before I do that I was wondering if what I'm
doing is the best way to go about fixing the problem, or if this workaround
adds more problems then it fixes, since it adds a bit of complexity to the
what's being rendered in the html.
After some time thinking about this, I was thinking of optimizing my
approach to the fix.
we are trying to show the url to follow the pattern :
`database/mydbname/_design/myddocname/_view/myviewname`, but translating the
document name to a url-safe string, can introduce '%', which breaks when used
with jquery, and escaping %, would break the url
I was thinking of translating everything to a url-safe string, then take
out % and replace it with `_percent_` for the html attributes that are used
with jquery. Then anyone looking at the code wouldn't have to deal so much with
the specific . / (space) issues, and they'd just have to deal with the one %
special character.
any concerns? is there a better approach i'm overlooking?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
