|View Issue Details [ Jump to Notes ]||[ Issue History ] [ Print ]|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|Target Version||Fixed in Version|
|Summary||0002430: onClick in 'a' tag is ignored|
|Description||Whilst onClick appears to be working for buttons, in 'a' tags the attribute is ignored.|
|Steps To Reproduce||Visit http://www.unsatisfactorysoftware.co.uk/netsurf/onclick.html|
Click on the link, which should redirect to the NetSurf homepage.
|Additional Information||The code is:|
It is my understanding that the onclick should execute, and if it doesn't return false then the usual href kicks in.
|Tags||No tags attached.|
|Fixed in CI build #|
|Reported in CI build #||3421|
|URL of problem page||http://www.unsatisfactorysoftware.co.uk/netsurf/onclick.html|
Dave Higton (developer)
Curious. The source of the cited URL is tiny. When I open the page and click repeatedly on the link, the throubber does something briefly each time - NS is doing something. The log say "Dispatching 'click' against"... but nowhere in the log does the target URL (netsurf-browser.org) appear. Surely it should appear, at least once, somewhere in the log?
Are we just reloading the page instead of visiting the target?
Dave Higton (developer)
|It's not Amiga-specific - my observations in my note above are from RISC OS with CI #3421.|
Chris Young (developer)
I think it's following the href and not running the onclick code.
The log does say it is dispatching the click but doesn't then go on to say "Hooray! An event!". So it looks like it isn't registering the onclick from the tag as an event, from the limited amount I understand about this stuff.
It could well be "not yet implemented".
|2016-02-21 15:54||Chris Young||New Issue|
|2016-02-21 15:54||Chris Young||File Added: ns.log|
|2016-02-21 15:55||Chris Young||Additional Information Updated||View Revisions|
|2016-02-21 18:13||Chris Young||Steps to Reproduce Updated||View Revisions|
|2016-02-21 18:13||Chris Young||Additional Information Updated||View Revisions|
|2016-02-21 20:51||Dave Higton||Note Added: 0001331|
|2016-02-21 20:53||Dave Higton||Note Added: 0001332|
|2016-02-21 21:15||Chris Young||Note Added: 0001334|
|2016-03-07 13:37||Vincent Sanders||Status||new => acknowledged|
|2016-03-07 13:37||Vincent Sanders||Additional Information Updated||View Revisions|