CLOSED FIXED 11843
[S60] Links on www.sonera.fi don't work and open floating Toolbar 
https://bugs.webkit.org/show_bug.cgi?id=11843
Summary [S60] Links on www.sonera.fi don't work and open floating Toolbar
Rasvan Alexandrescu
Reported 2006-12-15 12:49:52 PST
[S60]Links on webpage http://www.sonera.fi don't work and clicking a link just opens browsing Toolbar.
Attachments
KeyEventHandler modified to look for any node under pointer (1.37 KB, patch)
2006-12-15 12:57 PST, Rasvan Alexandrescu 		Sachin.Padma: review-
DetailsFormatted DiffDiff
variable for element type added (5.59 KB, patch)
2006-12-18 15:50 PST, Rasvan Alexandrescu 		Sachin.Padma: review-
DetailsFormatted DiffDiff
added Sachin's comments (5.34 KB, patch)
2006-12-20 10:01 PST, Rasvan Alexandrescu 		yongjun.zhang: review+
DetailsFormatted DiffDiff
3.1 patch (5.36 KB, patch)
2006-12-20 14:47 PST, Rasvan Alexandrescu 		yongjun.zhang: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Rasvan Alexandrescu
Comment 1 2006-12-15 12:57:22 PST
Created attachment 11868 [details] KeyEventHandler modified to look for any node under pointer
Sachin Padma
Comment 2 2006-12-18 07:15:24 PST
Comment on attachment 11868 [details] KeyEventHandler modified to look for any node under pointer FocusedElementType should have returned as anchor instead of none.
Rasvan Alexandrescu
Comment 3 2006-12-18 08:42:06 PST
For this particular case would work, but what if we have as a node an input box or any selection button or mail anchor, etc ? The toolbar will have priority over the node functionality which contradicts the specification. The solution I proposed covers all this cases.
Rasvan Alexandrescu
Comment 4 2006-12-18 15:50:38 PST
Created attachment 11909 [details] variable for element type added variable for element type added to the keyeventhandler
Sachin Padma
Comment 5 2006-12-20 08:30:22 PST
Comment on attachment 11909 [details] variable for element type added You can just check if iElType == TBrCtlDefs::EElementNone when activating second time.
Rasvan Alexandrescu
Comment 6 2006-12-20 10:01:33 PST
Created attachment 11937 [details] added Sachin's comments
Rasvan Alexandrescu
Comment 7 2006-12-20 14:47:31 PST
Created attachment 11939 [details] 3.1 patch
Yongjun Zhang
Comment 8 2006-12-20 14:49:10 PST
Comment on attachment 11939 [details] 3.1 patch r=me
Yongjun Zhang
Comment 9 2006-12-20 15:07:07 PST
Comment on attachment 11937 [details] added Sachin's comments r=me
Yongjun Zhang
Comment 10 2006-12-20 15:08:20 PST
Completed: At revision: 18363
Krishna
Comment 11 2006-12-22 09:56:23 PST
MHAE-6WEJCH Verified and Closed.
Joel Parks
Comment 12 2011-03-21 11:52:57 PDT
re-purposing InTSW keyword for use by QtWebkit team
Note You need to log in before you can comment on or make changes to this bug.