WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
8817
REGRESSION: TinyMCE: Clicking on a link has no effect
https://bugs.webkit.org/show_bug.cgi?id=8817
Summary
REGRESSION: TinyMCE: Clicking on a link has no effect
Justin Garcia
Reported
2006-05-09 18:04:50 PDT
When you hover over a link inside the editable region, the href that appears in the status bar is correct, but when you click on the link nothing happens and the status bar text changes to Run script: "void(0);" We think this is a regression in TinyMCE's sources.
Attachments
Add attachment
proposed patch, testcase, etc.
Timothy Hatcher
Comment 1
2006-05-09 21:57:28 PDT
This might not be a regression in TinyMCE but an intentional change to prevent
bug 7156
. This might install an click handler that calls void(0) to prevent the navigation. It is hard to say by looking their whitespace-stripped script file.
Alice Liu
Comment 2
2006-06-06 07:52:48 PDT
<
rdar://problem/4575096
>
Darin Adler
Comment 3
2006-07-16 06:23:16 PDT
(In reply to
comment #1
)
> It is hard to say by looking their whitespace-stripped script file.
Full source to TinyMCE is at their website, so someone should look at that and verify that this is intentional on their part.
David Carson
Comment 4
2006-07-18 16:09:05 PDT
Both Opera 9.0 and FF 1.5.0.4 exhibit similar behaviour. Clicking the link does nothing. FF and Safari ToT does allow you to right click the link and open in new window/new tab, which opens the link to sourceforge. Checking the code, it seems that the experience behaviour is correct. Here is the code that handles link selection in MCE: // Patch the darned link if (lnk && e.type == "mousedown") { lnk.setAttribute("mce_real_href", lnk.getAttribute("href")); lnk.setAttribute("href", "javascript:void(0);"); } // Patch back if (lnk && e.type == "click") { window.setTimeout(function() { lnk.setAttribute("href", lnk.getAttribute("mce_real_href")); lnk.removeAttribute("mce_real_href"); }, 10); } As can be seen, javascript:void(0) is correct.
David Carson
Comment 5
2006-07-18 16:23:34 PDT
Not a bug
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug