Bug 25525

Summary: [Gtk] Tooltips should be fully keyboard accessible
Product: WebKit Reporter: Joanmarie Diggs <jdiggs>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: alex, apinheiro, cgarcia, commit-queue, mario, walker.willie, xan.lopez
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on: 48986    
Bug Blocks: 25531    
Attachments:
Description Flags
Patch to fix tooltip location and text when using the keyboard
xan.lopez: review-
Updated patch using focusedOrMainFrame()
xan.lopez: review+
New patch that converts tooltip area to window coords none

Joanmarie Diggs
Reported 2009-05-03 13:19:43 PDT
Steps to reproduce: 1. Load the test case (https://bugs.webkit.org/attachment.cgi?id=29966&action=view) 2. Hover the mouse over the "Phone number" link. This will cause the tooltip associated with that link to appear. Leave the mouse pointer there. 3. Press Tab until focus is on the Submit button. 4. Press Ctrl+F1 (the traditional Gtk app command to show/hide the tooltip associated with the focused item). Expected results: The tooltip associated with the Submit button would be displayed -- and ideally displayed near the Submit button. Actual results: The tooltip associated with the "Phone number" link is displayed -- and, at least on my system, displayed at the bottom of the window. Comments: The reason for wanting Ctrl+F1 to toggle the visibility of the tooltip associated with the focused object is so that users who prefer to (or need to) use the keyboard instead of the mouse can access the title attribute information. The reason for wanting it displayed in the vicinity of the focused object is for users with low vision who currently have to do additional visual work to locate the desired information.
Attachments
Patch to fix tooltip location and text when using the keyboard (3.13 KB, patch)
2010-11-04 04:02 PDT, Carlos Garcia Campos 		xan.lopez: review-
DetailsFormatted DiffDiff
Updated patch using focusedOrMainFrame() (3.03 KB, patch)
2010-11-25 00:17 PST, Carlos Garcia Campos 		xan.lopez: review+
DetailsFormatted DiffDiff
New patch that converts tooltip area to window coords (3.07 KB, patch)
2010-11-29 04:10 PST, Carlos Garcia Campos 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alejandro G. Castro
Comment 1 2009-12-21 04:37:45 PST
This bug depends on bug 32819. After applying that patch we would have to do a small workaround to solve the fact that the tooltips in GTK when opened with Ctrl+F1 are rendered outside and under the widget area (makes sense when the widget is a line of text for instance). The best option would be probably to allow users to redefine the x and y properties in the query-tooltip, but it seems the API does not allow it; and anyway we need the position of the focused element of the webpage and we do not have it. We would have to wait for the DOM bindings, bug 16401, and check how to change the position of the tooltip if possible at all.
Joanmarie Diggs
Comment 2 2010-01-31 15:49:37 PST
Alex, since the bugs you mention seem to be fixed, is this something that can be tackled now?
Alejandro G. Castro
Comment 3 2010-02-09 23:45:51 PST
(In reply to comment #2) > Alex, since the bugs you mention seem to be fixed, is this something that can > be tackled now? I'm afraid we have to check how we could do fix a couple of things before because I think gtk+ positions the tooltip under the widget when you use Ctrl+F1 (in this case the widget is the webview) and there is not API to change it. And in webkit the tooltip is not associated with the focused element but with the mouse position, but I did not check it in detail. We still need to check those things in order to fix the bug, I'll try to check them.
Joanmarie Diggs
Comment 4 2010-02-09 23:54:22 PST
(In reply to comment #3) > (In reply to comment #2) > > Alex, since the bugs you mention seem to be fixed, is this something that can > > be tackled now? > > I'm afraid we have to check how we could do fix a couple of things before > because I think gtk+ positions the tooltip under the widget when you use > Ctrl+F1 (in this case the widget is the webview) and there is not API to change > it. And in webkit the tooltip is not associated with the focused element but > with the mouse position, but I did not check it in detail. We still need to > check those things in order to fix the bug, I'll try to check them. Thanks!! Depending on what you find.... I wonder if this could (should?) be broken into two separate bugs: 1. Ctrl+F1 should cause the tooltip to appear.... somewhere :-) 2. The tooltip should appear next to the focused element to be consistent with Gtk+ If item #1 is solved, users who are blind could then cause the tooltip to appear (which a screen reader could then speak and display in braille). This is the important thing IMHO.
Carlos Garcia Campos
Comment 5 2010-11-04 04:02:17 PDT
Created attachment 72925 [details] Patch to fix tooltip location and text when using the keyboard Attached patch fixed the text and position of the tooltip when using the keyboard. It uses the current focused element to get its location and title. I'm not sure the title is enough, but it fixes the test case attached to this report. There are two issues blocking this though: - GTK+ bug: https://bugzilla.gnome.org/show_bug.cgi?id=633974 - https://bugs.webkit.org/show_bug.cgi?id=48986
Carlos Garcia Campos
Comment 6 2010-11-16 10:08:48 PST
Comment on attachment 72925 [details] Patch to fix tooltip location and text when using the keyboard Now that bug #48986 is fixed this patch can be committed, it still won't work because of the gtk+ issue, but it's not more broken than current code, at least now it will show the right tooltip although wrongly positioned.
Xan Lopez
Comment 7 2010-11-24 05:32:06 PST
Comment on attachment 72925 [details] Patch to fix tooltip location and text when using the keyboard View in context: https://bugs.webkit.org/attachment.cgi?id=72925&action=review r- for always using main frame, which I think is wrong. > WebKit/gtk/webkit/webkitwebview.cpp:1634 > + I think you want to use focusedOrMainFrame()from FocusController here?
Carlos Garcia Campos
Comment 8 2010-11-25 00:17:26 PST
Created attachment 74836 [details] Updated patch using focusedOrMainFrame()
Xan Lopez
Comment 9 2010-11-25 00:36:00 PST
Comment on attachment 74836 [details] Updated patch using focusedOrMainFrame() r=me
Carlos Garcia Campos
Comment 10 2010-11-29 04:10:53 PST
Created attachment 75013 [details] New patch that converts tooltip area to window coords
WebKit Commit Bot
Comment 11 2010-11-29 04:35:26 PST
Comment on attachment 75013 [details] New patch that converts tooltip area to window coords Clearing flags on attachment: 75013 Committed r72789: <http://trac.webkit.org/changeset/72789>
WebKit Commit Bot
Comment 12 2010-11-29 04:35:33 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.