Closed (fixed)
Project:
Scald: Media Management made easy
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
24 Jan 2014 at 12:51 UTC
Updated:
26 May 2014 at 22:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
nagy.balint commentedA way to make a true # fragment is to set the fragment to FALSE and external to TRUE.
Attached a patch to fix the issue.
(Of course an other solution could be to not put a # fragment there at all and simply prevent the default from JS.)
Comment #2
jcisio commentedIt's weird that it does not work. The default is already prevented with "e.preventDefault();" (dnd-library.js l336). I can not reproduce the issue.
Comment #3
nagy.balint commentedI just checked on simplytest.me on chrome 34 and the markup is as follows:
Insert
As you can see the # is converted into a %23, and my patch fixes that as it should be done with the fragment attribute and not the href attribute but fragment must have a value so this is the only way.
My comment about preventDefault was mostly meant to the other method of doing this without the #.
Usually they use # to make a link not navigate anywhere, but in fact if there is no # and preventDefault is used then it will not navigate anywhere either, so # is unnecessary in fact if preventDefault is used.
However even if we don't want to change that, there is still a problem as in this case its converted at least on chrome to something else and is certainly not the intended behaviour.
Comment #4
aron novakAs i see the patch would definitely make the things more sane, this conversion is indeed unwanted. This patch does not make any harm, but helps in this confusing situation.
Comment #5
jcisio commentedDefinitely. Committed and pushed. Thanks.