jnsh / arc-theme

A flat theme with transparent elements (actively maintained fork)
GNU General Public License v3.0
906 stars 78 forks source link

Latest changes have no window controls on some windows #175

Closed cvetan closed 2 years ago

cvetan commented 2 years ago

I have checked out and installed latest changes on Ubuntu 20.04.4 and I discoveded some issues.

For instance Sublime Text window has no window controls until you hover over them. This wasn't the case earlier, so I think some of the latest changes introduces this issue.

You can see screenshots bellow.

Screenshot from 2022-02-27 14-59-37 Screenshot from 2022-02-27 14-59-41

jnsh commented 2 years ago

Did you confirm that you don't have similar issues with default themes?

I didn't see arc-theme getting update from official package sources for Ubuntu 20.04, so this should not result from changes, unless you've installed the theme from some other sources. Either way, could you try building the latest version of the theme, to see if that solves your issue?

If that didn't help, please let me know what desktop environment or window manager you're using. And whether you're experiencing this same issue with other applications besides sublime text? Also, you're saying you've discovered "some issues" and the one with sublime text is only a single example, so can you detail what other issues you're experiencing.

cvetan commented 2 years ago

Do you mean deafult theme of Sublime Text, or default theme of window manager? I did build theme from this repository and with latest commit, as per instructions for install. I've used no other sources, beside this since you forker theme. :D

I did check with default adwaita theme, and it didn't have issue. You can see down screenshot with arc theme build from some of the older commits, and with default adwaita theme.

And I am using gnome on ubuntu 20.04 . arc theme - older commits build Screenshot from 2022-02-27 15-53-08

adwaita theme Screenshot from 2022-02-27 15-53-37

And yes, I found only this issue, sorry if plural made the confusion.

jnsh commented 2 years ago

Sorry, I think I misunderstood your first post, and thought you meant that an Ubuntu update broke the theme, If the theme was built yourself, and this started happening after some recent changes, then this is clearly something resulting from the theme itself :)

I'm unfamiliar with Sublime Text, but the screenshots look like it's not using decorations from GNOME's window manager, but does its own decorations based on the GTK theme. I need to investigate this further, but https://github.com/jnsh/arc-theme/commit/45b46b529984dad14d0ecc9999a9aec9752b36bd might have resulted in this.

Can you tell which version of sublime text you're using, so I can use the same version for testing?

cvetan commented 2 years ago

Sublime text 4126 with Agila theme

jnsh commented 2 years ago

Apparently this happens with some sublime text themes, although the default theme seemed unaffected.

Arc-theme draws its custom titlebar buttons with a background-image, and makes the standard button icons transparent. Looks like this results in somewhat odd behavior with how some of the sublime text themes are done, and forcibly hiding the icon with 45b46b529984dad14d0ecc9999a9aec9752b36bd made it even worse.

This should be fixed now, after reverting that commit.