Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BlackMATE-border Theme - Overlooked use of "bar_unfocused.png" for unfocused non-maximized windows #313

Open
ericmarceau-rogers opened this issue Aug 16, 2024 · 2 comments

Comments

@ericmarceau-rogers
Copy link

ericmarceau-rogers commented Aug 16, 2024

Expected behaviour

Un-focused windows should all be presented with the same colour tile "bar_unfocused.png", whether maximized or not.
Frame_style "normal_unfocused" should incorporate attributes for "titlebar_fill_unfocused".

Actual behaviour

Currently, metacity-theme-3.xml for BlackMATE-border only has directives to show different colour for unfocused windows that are maximized, namely frame_style "normal_max_unfocused"!
No similar directive exists for frame_style "normal_unfocused" windows.

Steps to reproduce the behaviour

  • Use UbuntuMATE 22.04.4 LTS distro installation.
  • Load published theme set for BlackMATE-border from mate-themes-3.22.26 build.
  • Open more than one terminal.
  • Click on one then the other, while looking at the titlebar to observe any colour change (none will appear).
  • Maximize one window, then click on the non-maximized terminal from the taskbar and compare the titlebar colours (the colour difference will be barely visible, but it is observable).

MATE general version

mate-themes-3.22.26

Package version

mate-themes-3.22.26

Linux Distribution

UbuntuMATE 22.04.4 LTS

Recommended fix

patch for working implementation in attached file:
metacity-theme-3.xml__ericmarceau.patch.txt

Related discussion.

Link to bug report of your Distribution (requirement)

Bug report #2077202 submitted on Launchpad.

@lukefromdc
Copy link
Member

lukefromdc commented Sep 5, 2024

Commit
23ac3fd
fixes this for the normal BlackMATE theme, where it also has become an issue. Will do the same for BlackMATE-border

With my eyes the difference is very hard to spot (you have to focus on it) but it is in fact there, proving the code is working with the before and after test. Tested in marco and even in Wayfire's 3ed party marco-decorator, worked in both.

@lukefromdc
Copy link
Member

df2b819

and

a85ad3e

fixes this for BlackMATE-border. I got the logic reversed on the first commit so had to fix that with a second.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants