349053 - Visual Focus with Scroll Bars in Web Forgery popup

Status: RESOLVED DUPLICATE of bug 348368

(Firefox :: Disability Access, defect)

Description

[Wayne DeAngelo]

Test environment:
   Windows XP, Firefox 2 (Bon Echo) build 20060811, 
   Screen resolution: 1024x768, High Contrast mode: High Contrast #1 (extra
large)

Description:
On the "Suspected Web Forgery" popup, when scroll bars are present in popup window (when large fonts are used) the visual focus does not change to follow the program focus, when tabbing to the buttons in the popup.


Steps to recreate:
1. Set High Contrast #1 (extra large) in Control Panel, accessibility options. 
(Note that this results in large fonts.)

2. Make sure the Google anti-phishing security option is set in the Firefox
security options.
    a. Open Firefox Tools, Options, and open Security page.
    b. Select "Tell me if the site I'm using is a suspected forgery" checkbox
    c. Select "Check by asking [Google] about each site I visit" radio button.
3. Open this web page URL in Firefox browser: 
http://www.google.com/tools/firefox/safebrowsing/phish-o-rama.html
- This results in the display of the "Suspected Web Forgery" pop-up.  (Scroll
bars are present when large fonts are selected).

4. Press the Tab key.  This moves program focus to the "Read More >>" button. 
Note that visual focus does not change... with large fonts and scroll bars
present, the user has no idea where the keyboard focus is at this point.

Expected results:

- The popup window text should visually shift to move the text with program
focus into view.  The visual focus should be clear at all times when tabbing through the
dialog.

Comment 1

[Mark Pilgrim (inactive)]

Confirmed that this is a problem, but I don't know how to fix it.  XUL elements don't have a .scrollIntoView method (like HTML elements do).  The .boxObject property of a <scrollbox> has a method called .ensureElementIsVisible ( http://www.xulplanet.com/references/xpcomref/ifaces/nsIScrollBoxObject.html#method_ensureElementIsVisible ) but we're not using a <scrollbox>, we're using a <vbox>.  I tried changing the <vbox> to a <scrollbox> and removing the content.style.overflow="auto" line in maybeAddScrollbars_(), but it didn't work.  I tried document.getElementById('safebrowsing-palm-message-content').boxObject.QueryInterface(Components.interfaces.nsIScrollBoxObject).ensureElementIsVisible but that doesn't work either.  Bumping over to Tony.

Comment 2

[Aaron Leventhal]

High contrast mode is required in the steps.

Is it a dupe of bug 348368?

Comment 3

[Wayne DeAngelo]

Yes, will mark it as dup of 348368

Comment 4

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1527381