Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIx dot in anchor #169

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

FIx dot in anchor #169

wants to merge 2 commits into from

Conversation

amosbird
Copy link

No description provided.

@areina
Copy link
Member

areina commented Mar 15, 2018

Hello @amosbird!

I would like to try/test this, could you explain us which is the use case? Why is this needed?

Thank you!

@amosbird
Copy link
Author

Hi, here is a case that breaks the old code in webkit, chrome and even IE, only firefox works.

https://la.wentropy.com/e3ih.html#//apple_ref/func/string.sub

by escaping the dot, this works for all browsers except firefox.

https://la.wentropy.com/e3ih.html#//apple_ref/func/string%252Esub

@areina
Copy link
Member

areina commented Mar 15, 2018

Cool, thanks for the example. I'll take a look later

@kidd
Copy link
Member

kidd commented Mar 17, 2018

Wow, so either we support ff or all the others? Man, the browsers' world is so broken :/

Thoughts? Ideas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants