Bug 342121 - Unable to see what has focus in parts of Memory Analyzer
Summary: Unable to see what has focus in parts of Memory Analyzer
Status: RESOLVED FIXED
Alias: None
Product: MAT
Classification: Tools
Component: GUI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Andrew Johnson CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks: 300655
  Show dependency tree
 
Reported: 2011-04-07 05:20 EDT by Julie Stalley CLA
Modified: 2014-01-31 08:02 EST (History)
1 user (show)

See Also:


Attachments
Screen shot to show the problem (21.34 KB, image/png)
2011-04-07 05:32 EDT, Julie Stalley CLA
no flags Details
Proposed patch (548 bytes, patch)
2011-04-07 05:35 EDT, Julie Stalley CLA
andrew_johnson: iplog-
Details | Diff
Proposal for focus box (3.86 KB, image/png)
2012-11-29 07:05 EST, Andrew Johnson CLA
no flags Details
Second proposal for focus box (3.57 KB, image/png)
2013-03-21 08:11 EDT, Andrew Johnson CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Julie Stalley CLA 2011-04-07 05:20:32 EDT
Build Identifier: 

Report views in memory analyzer make it almost impossible to see what has focus. The blue highlighting of the significant lines prevent the user from seeing the focus. 

Reproducible: Always

Steps to Reproduce:
1. Open Memory Analyzer
2. Go to one of the report views where the line is highlighted in blue. 
3. Tab through the items and note that it is difficult to see which line has focus
Comment 1 Julie Stalley CLA 2011-04-07 05:32:37 EDT
Created attachment 192710 [details]
Screen shot to show the problem
Comment 2 Julie Stalley CLA 2011-04-07 05:35:34 EDT
Created attachment 192711 [details]
Proposed patch

To allow focus to be clearly seen, I have changes the style sheet so that the highlighted line is just shown in bold rather than having a blue background.
Comment 3 Krum Tsvetkov CLA 2011-04-29 04:45:20 EDT
Thanks for reporting the problem and sending a patch.
I applied the patch and did some tests locally, however I am not really convinced that it is easier to notice which line is selected with just the text in bold. This impression I had for a "normal" tree where the selected item was somewhere in the middle of the rest.
I saw also some worse cases, where the selected line was the first one. Then it is difficult to distinguish the line from the header of the table.
So, for the moment I wouldn't like to submit the patch. But I'd like to understand the problem a bit better. Is it that when the line is highlighted in blue, then it is difficult to see the dotted rectangle which marks the keyboard focus? If yes, then can we improve this by selecting a lighter or darker color, or in some other way? I've nothing against the bold, but I think it will make it more difficult for many users to easily notice the selection.
Comment 4 Julie Stalley CLA 2011-05-03 06:38:34 EDT
Yes, the problem was reported as "when the line is highlighted in blue, it is difficult to see the dotted rectangle".
Comment 5 Andrew Johnson CLA 2012-11-29 07:05:16 EST
Created attachment 224100 [details]
Proposal for focus box
Comment 6 Andrew Johnson CLA 2012-11-29 07:05:41 EST
I think the focus box is visible on the blue highlighting, particularly when it appears/disappears as the tab key moved the focus, and the blue highlighting is more important than the focus.

If this really is a serious problem we could add this to style.css:

.result tbody tr.selected td a:focus {
    outline-style: dotted;
    outline-color: #000000
}

as shown in the focus1.png attachment.
Comment 7 Andrew Johnson CLA 2013-03-21 08:11:56 EDT
Created attachment 228844 [details]
Second proposal for focus box
Comment 8 Andrew Johnson CLA 2013-03-21 08:15:29 EDT
I've committed a change which adds a thin white dotted box showing focus for a selected item in a table (which has a blue background).
Comment 9 Andrew Johnson CLA 2013-03-21 08:16:45 EDT
Comment on attachment 192711 [details]
Proposed patch

Patch not used.