Last Comment Bug 338033 - No visible focus indicator on image map areas with black background
: No visible focus indicator on image map areas with black background
Status: RESOLVED FIXED
: access, sec508
Product: Core
Classification: Components
Component: Layout (show other bugs)
: Trunk
: x86 Windows XP
: -- normal with 1 vote (vote)
: mozilla23
Assigned To: Takeshi Kurosawa
:
Mentors:
http://www.w3schools.com/html/tryit.a...
Depends on:
Blocks: focusnav
  Show dependency treegraph
 
Reported: 2006-05-15 11:50 PDT by Dan Kinnunen
Modified: 2013-04-27 18:31 PDT (History)
9 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch rv. 1.0 (1.03 KB, patch)
2013-02-21 06:07 PST, Takeshi Kurosawa
roc: review+
Details | Diff | Review
Screenshot with Patch rv. 1.0 (22.60 KB, image/png)
2013-02-21 06:08 PST, Takeshi Kurosawa
no flags Details
Refreshed Patch rv. 1.0 (using mercurial queues) (1.35 KB, patch)
2013-04-27 01:48 PDT, Takeshi Kurosawa
no flags Details | Diff | Review

Description Dan Kinnunen 2006-05-15 11:50:50 PDT
Env:Bon Echo 20060515 nightly build

Description:
When you TAB to a portion of an image map that has a black background, or a black image on a black background, there is no visible focus indicator.  There should be a dotted line around the image or image map area.

Firefox displays the visual focus indicator (dotted line) fine on areas with non-black background.  It appears to use a black dotted line, which is invisible on an image with a black background.  To contrast, Internet Explorer appears to "sense" the dark background of the image, and in that case uses a white dotted line instead of black to produce a visible keyboard focus indicator.

Recreate:
1. Load this URL in Firefox (Bon Echo):
     http://www.w3schools.com/html/tryit.asp?filename=tryhtml_areamap
    There will be a picture of the sun with two planets (mercury and venus)
     in the right pane of the window.
2. Press TAB until you see "sun.htm", "mercur.htm",  and "venus.htm" cycle through
    the status bar.  (It helps if you click the mouse on the text directly above
    the image, then press TAB.  

Error 1:  You should see a rectangular visual focus indicator (dotted line) around the Sun,
   then a circular dotted line around the 1st planet (mercury),  then a circular
   dotted line around the 2nd planet (venus) as you tab through the links.
   I assume the (black) dotted lines are actually there, but because of the black background, 
   they are invisible.  Firefox should sense the darkness of the background color,
   and use a white dotted line instead of black in this case.

Error 2: If you tab through the image map, watch the status line at the bottom of the
   Firefox window.  You see the status text cycle from ...sun.htm, to ...mercur.htm, 
   and finally to ...venus.htm.  If you keep pressing TAB past the venus link, notice
   that the status bar DOES NOT CHANGE as you continue to tab.  
   When you tab away from the link to venus.htm, the status line should be 
  blanked out, so it does not imply that the venus.htm link is still available
  at the new keyboard focus position.

You can find other images with this same problem.
For example, on the IBM.COM home page ( http://www.ibm.com/us/ )
The IBM logo image at the upper left corner has a black background.
If you tab to that image, you also do not see the visual focus indicator
(dotted line).
Comment 1 Mark Pilgrim (inactive) 2006-12-07 14:21:44 PST
Confirmed on nightly trunk.
Comment 2 David Adam 2009-06-30 00:57:34 PDT
This is still an issue in Firefox 3.0.11, and also manifests in <input type="submit"> elements with a black background colour.
Comment 3 Takeshi Kurosawa 2013-02-21 06:07:25 PST
Created attachment 716506 [details] [diff] [review]
Patch rv. 1.0

Patch

Draw focus indicator twice to make it visible.
At first, draw white solid indicator, then draw black dotted one.
Comment 4 Takeshi Kurosawa 2013-02-21 06:08:34 PST
Created attachment 716507 [details]
Screenshot with Patch rv. 1.0
Comment 5 Matt Woodrow (:mattwoodrow) (PTO until 27 June) 2013-02-25 13:19:47 PST
Comment on attachment 716506 [details] [diff] [review]
Patch rv. 1.0

I don't think I'm the right person to review this. The code changes look fine, but I'm not sure if we need UI review for the change.
Comment 6 Takeshi Kurosawa 2013-03-04 05:50:37 PST
Comment on attachment 716506 [details] [diff] [review]
Patch rv. 1.0

Thanks comment and review!
Thus I flag ui-review.
Comment 7 Takeshi Kurosawa 2013-03-31 07:18:27 PDT
Comment on attachment 716506 [details] [diff] [review]
Patch rv. 1.0

Hmm... I wondered ui-review is needed and there was no response.
So I canceled the request.
# but if it is needed, please set it again

Are there any required works or steps before requesting checkin-needed?
Comment 8 Takeshi Kurosawa 2013-04-27 01:48:47 PDT
Created attachment 742685 [details] [diff] [review]
Refreshed Patch rv. 1.0 (using mercurial queues)

Patch rv. 1.0 against current mozilla-central
Comment 9 Masayuki Nakano [:masayuki] (Mozilla Japan) 2013-04-27 01:55:00 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/beb08ab90a3d
Comment 10 Ryan VanderMeulen [:RyanVM] 2013-04-27 18:31:50 PDT
https://hg.mozilla.org/mozilla-central/rev/beb08ab90a3d

Note You need to log in before you can comment on or make changes to this bug.