piroor / treestyletab

Tree Style Tab, Show tabs like a tree.
http://piro.sakura.ne.jp/xul/treestyletab/
Other
3.48k stars 279 forks source link

[Bug] photon sidebar backround is too light on linux #3525

Closed bughit closed 5 months ago

bughit commented 5 months ago

Abstract

On Ubuntu 20.04 the sidebar background with the Photon theme is much lighter than on windows. There's not enough contrast.

Steps to reproduce

  1. Start Firefox with clean profile on (win10 or win11) and Ubuntu 20.04
  2. Install TST.
  3. enable photon
  4. observe the sidebar background

Expected result

as on windows image

Actual result

on ubuntu 20.04

image

Environment

piroor commented 5 months ago

In short: Choosing the option will apply the BG color same to Windows: TST Options => "Development" => "Color scheme" => "Photon".

The color choice is intentional. There are various desktop themes on Linux distributions, and it is major on Linux community to change color scheme of the desktop environment - there is no "standard", so "Photon" color scheme may become unnatural on some environments easily. So I decided preferring system color on Linux. The BG color referes "ThreeDShadow" on the system color, it is "darkgray" on the "Photon" color scheme. It will become more darker color if you configure color definitions of your desktop environment system more darker. Or a user style sheet (under TST Options => Advanced) to override the color is also available, like:

:root { --tabbar-bg: darkgray; }
bughit commented 5 months ago

it is major on Linux community to change color scheme of the desktop environment

I'm fairly sure that Firefox chrome (whatever is under its control, within the window chrome which may be os specific) is intended to be the same (or as close as possible) across OSes.

image

This intent is pretty clear from the above win/linux comparison. You are violating this convention by choosing to style parts of TST differently on linux. Your different background sticks out as an unwelcome exception to this rule. By default there should not be a difference. If you believe anyone actually wants this, which is doubtful, it should be optional.

bughit commented 5 months ago

There is somewhat of a general color difference between linux and win, but I'm referring more to preserving the contrast difference between the foreground and the background, than the exact color values.

piroor commented 5 months ago

I'm fairly sure that Firefox chrome (whatever is under its control, within the window chrome which may be os specific) is intended to be the same (or as close as possible) across OSes.

Here is an anti-example: Firefox 124 on ArchLinux with the "Adapta" theme. image The dark background color of the tab bar is referred with the special system color name -moz-headerbar (or -moz-headerbarinactive, both special colors cannot be used from addon's resources) so Firefox just uses the color defined in the GNOME theme. As the result the contrast around tab face and tab background is quite different from Windows.

piroor commented 5 months ago

I think we cannot choose colors providing same contrast on all platforms, so I chose second-best: system colors which are expected to have enough contrast.

Anyway I agree that "ThreeDShadow" the current choice is little pale on this case so there is any other possible choice. I'm not negative to change the color I use for the background, but the new choice need to have contrast enough on various environments (e.g. ArchLinux + Adapta).

github-actions[bot] commented 5 months ago

This issue has been closed due to no response within 7 days after labeled as "wontfix", 7 days after last reopened, and 7 days after last commented.