External Link detection is case sensitive #2446

Closed
sergioharo opened this Issue Sep 14, 2011 · 4 comments

Projects

None yet

5 participants

@sergioharo

The logic that determines whether to change a page via AJAX or via a page load is not case insensitive.

Say that I'm on http://jquerymobile.com/demos/1.0b3/

The following links open via AJAX (as expected):

<a href="docs/pages/index.html" class="ui-link-inherit">Pages &amp; dialogs</a>

<a href="http://jquerymobile.com/demos/1.0b3/docs/pages/index.html" class="ui-link-inherit">Pages &amp; dialogs</a>

The following opens as an external link:

<a href="http://jqueryMOBILE.com/demos/1.0b3/docs/pages/index.html" class="ui-link-inherit">Pages &amp; dialogs</a>

@jblas
Contributor
jblas commented Sep 14, 2011

Yeah, apparently only the hostname part of a URL should be considered case-insensitive. Everything else should be case-sensitive.

http://www.w3.org/TR/WD-html40-970708/htmlweb.html

@johnbender
Contributor

@sergioharo

The relevant information from the document:

URLs in general are case-sensitive (with the exception of machine names). There may be URLs, or parts of URLs, where case doesn't matter, but identifying these may not be easy. Users should always consider that URLs are case-sensitive.
@johnbender johnbender closed this Sep 15, 2011
@johnbender johnbender reopened this Sep 15, 2011
@johnbender
Contributor

Hasty in my closing of the issue. Re-opening to address in the future.

@johnbender johnbender was assigned Aug 9, 2012
@gabrielschulhof
Contributor

Reproduced on jsfiddle with latest and greatest: http://jsfiddle.net/5E2V4/1/show/light/

@gabrielschulhof gabrielschulhof self-assigned this Jun 10, 2014
@gabrielschulhof gabrielschulhof added this to the 1.4.4 milestone Jun 10, 2014
@agcolom agcolom added a commit to agcolom/jquery-mobile that referenced this issue Nov 26, 2014
@gabrielschulhof @agcolom gabrielschulhof + agcolom Navigation: Compare domains case-insensitively inside $.mobile.path
Closes gh-7486
Fixes gh-2446
ffb6635
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment