310884 - Focused state stay on treechildren::-moz-tree-row when tree doesn't have -moz-appearance: listbox.

Status: NEW

(Core :: XUL, defect)

Description

[AKIHIRO Misaki (a.k.a Kuden)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; ja-JP; rv:1.4) Gecko/20030624 Netscape/7.1
Build Identifier: Mozilla/5.0 (Windows; U; Win98; ja; rv:1.8b5) Gecko/20051002 Firefox/1.4.1

Build Id: 2005100206

This becomes a problem by the no native appearance theme. 

chrome://global/skin/tree.css
Line 47: tree
Line 86: treechildren::-moz-tree-row(current, focus)

Reproducible: Always

Steps to Reproduce:
1. Write this code in userChrome.css.
tree {-moz-appearance: none !important;}
2. Run Firefox and open the Bookmarks sidebar (Press Ctrl+B) .
3. Focused state stay Bookmarks searchbox. Press Tab key.
Focus change: Bookmarks searchbox -> Bookmark item on top of Bookmarks tree ->
Active tab in tab bar or content area
Actual Results:  
Focused state stay on Bookmark item on top of Bookmarks tree.

Expected Results:  
Focused state doesn't stay on.

Comment 1

[AKIHIRO Misaki (a.k.a Kuden)]

Attachment: screenshot

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Kuden, still a problem?

Comment 3

[AKIHIRO Misaki (a.k.a Kuden)]

Attachment: screenshot on Faststripe

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090719 Minefield/3.6a1pre {Build ID: 20090719231624}

Reproducible with trunk.
Reproducible on Faststripe (default theme of Windows CE).
Reproducible on the All non-native appearance theme.

Comment 4

[AKIHIRO Misaki (a.k.a Kuden)]

Attachment: Faststripe for trunk/Windows

Faststripe for trunk/Windows

Steps to Reproduce:
1. make a new profile
2. install this extension
3. run Firefox and open the Bookmarks sidebar (Press Ctrl+B)
4. Press Tab key

Focus change: Bookmarks searchbox -> Bookmark item on top of Bookmarks tree
-> Active tab in tab bar or content area

Actual Results:
Focused state stay on Bookmark item on top of Bookmarks tree.

Does anyone reproducible this?

Comment 5

[Makoto]

Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090719 Minefield/3.6a1pre

Reproducible with trunk/Windows Vista.

Comment 6

[AKIHIRO Misaki (a.k.a Kuden)]

Well... this is reproduced by the normal default theme now. 

On Firefox 2:
default theme -> not reproducible
non-native appearance theme -> reproducible

This state is treechildren::-moz-tree-row(current, focus).
Isn't it a bug that the focus ring doesn't move?

Comment 7

[AKIHIRO Misaki (a.k.a Kuden)]

This is regression by Bug 251910. Dão Gottwald makes this patch. 

chrome://browser/skin/places/places.css

.sidebar-placesTree {
  -moz-appearance: none;
}

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to comment #7)
> This is regression by Bug 251910. Dão Gottwald makes this patch. 
> 
> chrome://browser/skin/places/places.css
> 
> .sidebar-placesTree {
>   -moz-appearance: none;
> }

CC'ing Dao.

Comment 9

[Dão Gottwald [:dao]]

The fact that the focus ring stays in trees with -moz-appearance:none is not a regression from bug 251910.

Comment 10

[Firefox Bug Husbandry Bot]

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