Active sidebar tab should be marked with visuallyselected="true"

RESOLVED INVALID

Status

SeaMonkey
Sidebar
--
trivial
RESOLVED INVALID
3 years ago
3 years ago

People

(Reporter: exalm7659, Unassigned)

Tracking

({regression})

SeaMonkey 2.38 Branch
All
Linux
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Created attachment 8625152 [details]
screenshot.gif

bug 1066531 introduced "visuallyselected" attribute for tabs in tabs widget.
It is only useful for Firefox e10s builds, and on normal builds (e.g. in SeaMonkey) it always has the same value as "selected".

Along with that, tab native appearance (-moz-appearance: tab;) now uses the "visuallyselected" attribute to indicate selected tab, while it used "selected" earlier.
That has caused a regression in SM sidebar tabs.

As they still uses the old "selected" attribute, the active tab displays as an inactive one in custom themes that use tab appearance. This is something that cannot be fixed in themes themselves and should be fixed in SM sidebar instead.

It can be reproduced with this style:
.texttab-sidebar {
    -moz-appearance: tab;
}
See the attached screenshots.

The two attributes should probably be updated simultaneously, like in tabs.
(Reporter)

Updated

3 years ago
Keywords: regression
(Reporter)

Comment 2

3 years ago
Oh well, seems like themes can do it with wrapping tab content into a box that inherits visuallyselected=selected. I guess this can be closed, since it's kinda of no use for SM itself...
(Reporter)

Updated

3 years ago
Status: UNCONFIRMED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.