Bug 325899 - [Preferences] [accessibility] Preferences Dialog menu button needs a tooltip
Summary: [Preferences] [accessibility] Preferences Dialog menu button needs a tooltip
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M3   Edit
Assignee: Oleg Besedin CLA
QA Contact: Oleg Besedin CLA
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks: 333898
  Show dependency tree
 
Reported: 2010-09-21 15:53 EDT by Carolyn MacLeod CLA
Modified: 2011-10-06 13:35 EDT (History)
5 users (show)

See Also:


Attachments
screen snap of preferences dialog (26.22 KB, image/png)
2010-09-21 16:08 EDT, Carolyn MacLeod CLA
no flags Details
Patch (2.56 KB, patch)
2010-10-12 11:58 EDT, Oleg Besedin CLA
no flags Details | Diff
screenshot of properties dialog (13.63 KB, image/png)
2010-12-06 12:53 EST, Allan Godding CLA
no flags Details
Screenshot of 3.7M3 (83.70 KB, image/jpeg)
2010-12-17 15:02 EST, Oleg Besedin CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Carolyn MacLeod CLA 2010-09-21 15:53:10 EDT
Build id: I20100915-2024
org.eclipse.ui.internal.dialogs.FilteredPreferenceDialog

In method getContainerToolBar(Composite composite) of FilteredPreferenceDialog class, the popupMenuAction needs toolTipText (or it can perhaps just have "text", which the ToolBarManager will use for the tooltipText because the toolItem is showing an image).

This is needed for accessibility: screen readers need to have a name, and the ToolBarManager class provides a name based on the tooltipText.
Comment 1 Carolyn MacLeod CLA 2010-09-21 15:54:05 EDT
Would be nice if this could go into 3.6.2. It should be a very simple fix.
Comment 2 Carolyn MacLeod CLA 2010-09-21 16:08:03 EDT
Created attachment 179341 [details]
screen snap of preferences dialog

In case it is useful, here is a screen snapshot of the tool item that needs a tooltip. I can verify the fix, if you like, or if you have a copy of inspect32.exe, you can look to make sure the button has an accName when it gets focus.
Comment 3 Oleg Besedin CLA 2010-10-12 11:58:09 EDT
Created attachment 180684 [details]
Patch
Comment 4 Oleg Besedin CLA 2010-10-12 11:59:18 EDT
Patch applied to CVS Head.
Comment 5 Oleg Besedin CLA 2010-10-27 15:07:03 EDT
Verified in I20101026-2000.
Comment 6 Allan Godding CLA 2010-12-06 12:53:47 EST
Created attachment 184642 [details]
screenshot of properties dialog

Unfortunately, this bug also affects PropertyPage (see screenshot).  It would be fantastic if the fix for this could make it into 3.6.2.
Comment 7 Carolyn MacLeod CLA 2010-12-06 12:57:44 EST
Reopened to address comment 6.
Comment 8 Oleg Besedin CLA 2010-12-06 13:28:48 EST
(In reply to comment #6)
> Created an attachment (id=184642) [details]
> screenshot of properties dialog
> Unfortunately, this bug also affects PropertyPage (see screenshot).  It would
> be fantastic if the fix for this could make it into 3.6.2.


Where that dialog is coming from? The drop-down "triangle" button is from the dialog, not the property page. 

The only other place that adds that drop down in the SDK that I can find is the FilteredItemsSelectionDialog, and that adds a "Menu" tooltip. (Visisble in, say, "Open Resource" dialog, Ctrl+Shift+R.)
Comment 9 Carolyn MacLeod CLA 2010-12-17 14:49:19 EST
Select a project in the Package Explorer and choose "Properties" from the context menu. The menu button is on the top right.

If possible, please also release both fixes for 3.6.2 (i.e. the fix in comment 3, and the fix for this property page dialog). Thanks!
Comment 10 Oleg Besedin CLA 2010-12-17 15:02:37 EST
Created attachment 185456 [details]
Screenshot of 3.7M3

(In reply to comment #9)
> Select a project in the Package Explorer and choose "Properties" from the
> context menu. The menu button is on the top right.


That is fixed in 3.7M3, see the attached screenshot? That's why I am asking Allan which SDK dialog he is using in his screenshot.
Comment 11 Carolyn MacLeod CLA 2011-01-10 11:36:45 EST
Thanks, Oleg. I believe Allan is still on Eclipse 3.5 (He is using Jazz).
We were hoping that the fix could go into 3.6.2, so that Jazz based products can have it before Eclipse 3.7.
Comment 12 Oleg Besedin CLA 2011-01-10 12:09:35 EST
The screenshot that Allan attached does not look like the FilteredPreferenceDialog (which is shown in the Carolyn's screen snap). For starters, there is no filter field in there. 

I've searched through the Platform code and FilteredPreferenceDialog is the only dialog that I can find with the  drop-down triangle button that did not have a tooltip.

The 3.4 and 3.6 SDKs have property dialogs with filter fields so this does not seem to be something changed in 3.6+.

So, my suspicion is that the properties dialog that Allan sees does not come from the Platform UI. That's why I asked in the comment 8 what is the Eclipse dialog that needs to be fixed.
Comment 13 Oleg Besedin CLA 2011-01-10 14:10:31 EST
I had a chat with Allan and he confirmed that, indeed, it is the PropertyDialog that is used in his screenshot. 

I'll mark again this bug as fixed and will open a clone for 3.6.2 backport.
Comment 14 Oleg Besedin CLA 2011-01-10 14:13:31 EST
The 3.6 backport is in the bug 333898.
Comment 15 Oleg Besedin CLA 2011-01-10 16:04:28 EST
Updated tooltip text to be "Menu", see bug 333898 comment 3 .
Comment 16 Oleg Besedin CLA 2011-03-08 10:07:00 EST
Verified in I20110307-2110.
Comment 17 Dani Megert CLA 2011-05-16 04:34:16 EDT
*** Bug 345769 has been marked as a duplicate of this bug. ***
Comment 18 Dani Megert CLA 2011-10-06 08:20:29 EDT
This didn't make it into 4.x.

NOTE: This was requested to backport, hence it should probably also go into 4.1.1.
Comment 19 Dani Megert CLA 2011-10-06 08:20:44 EDT
.
Comment 20 Oleg Besedin CLA 2011-10-06 13:35:59 EDT
Copied changes into R4_development stream.