| Summary: | [S60] Links on www.sonera.fi don't work and open floating Toolbar | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Rasvan Alexandrescu <rasvan.alexandrescu> | ||||||||||
| Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||||||
| Status: | CLOSED FIXED | ||||||||||||
| Severity: | Normal | ||||||||||||
| Priority: | P2 | ||||||||||||
| Version: | 420+ | ||||||||||||
| Hardware: | S60 Emulator | ||||||||||||
| OS: | S60 3rd edition | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Rasvan Alexandrescu
2006-12-15 12:49:52 PST
Created attachment 11868 [details]
KeyEventHandler modified to look for any node under pointer
Comment on attachment 11868 [details]
KeyEventHandler modified to look for any node under pointer
FocusedElementType should have returned as anchor instead of none.
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. Created attachment 11909 [details]
variable for element type added
variable for element type added to the keyeventhandler
Comment on attachment 11909 [details]
variable for element type added
You can just check if iElType == TBrCtlDefs::EElementNone when activating second time.
Created attachment 11937 [details]
added Sachin's comments
Created attachment 11939 [details]
3.1 patch
Comment on attachment 11939 [details]
3.1 patch
r=me
Comment on attachment 11937 [details]
added Sachin's comments
r=me
Completed: At revision: 18363 MHAE-6WEJCH Verified and Closed. re-purposing InTSW keyword for use by QtWebkit team |