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

Weird outline arounds some apps #52

Closed
actionless opened this issue May 23, 2018 · 55 comments
Closed

Weird outline arounds some apps #52

actionless opened this issue May 23, 2018 · 55 comments

Comments

@actionless
Copy link
Member

From @jdamboeck on February 14, 2018 19:55

Outline despite of outline=0.. also what is this glow around the windows?
screenshot from 2018-02-14 20-52-11

oomox 1.5 arch

Copied from original issue: themix-project/themix-gui#123

@actionless
Copy link
Member Author

  1. outline is a border which is displayed over an item selected with keyboard (with <Tab> key)

  2. glow is inverted shadow (for dark themes)

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 13:6

Yeah... I know what an outline is. But I disabled the outline then why is it still there.
As for the shadow, there should be a switch to just keep it as a shadow instead of a glow with dark themes.

@actionless
Copy link
Member Author

  1. could you highlight the mistaken outline on your screenshot?

  2. i'll be happy to merge such change

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 15:37

  1. Okay.. I made a mistake.. there is an option focused outline width. i thought it relates to the window border but its the selected item in other elements.

but there is another error.

see the attached screens:
screenshot1
screenshot2

  1. That would be really great

@actionless
Copy link
Member Author

does the problem happens only with Telegram (and other google-chrome/electron-based apps) ?

if so -- that could be a duplicate of this issue:
themix-project/themix-gui#121 (Slack in that ticket AFAIK is using electron)

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 16:6

it happens with all windows that use a small window decoration(tool style window i think). gimp for example or the oomox popup when you export the theme.

@actionless
Copy link
Member Author

i still can't reproduce it but let's try to debug it:

  1. is it displayed fine if gtk theme set to oomox but WM decorations theme is different?
    and if the opposite?

  2. which WM do you use?

  3. create a file ~/.config/gtk-3.0/gtk.css and put there:

window decoration {
	margin: 0;
	border: none;
}

restar the app and see if it helped

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 17:53

  1. i cant change my WM decorations.
  2. i am using gnome 3.26.2
  3. stays the same

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 18:3

  1. Okay.. After login back the faulty outline is gnone... but the outline that stays is always the focused window border color.

screenshot from 18 02

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 18:30

and... when i change the border color... the faulty border on all tool style windows stays the same color as before. so i think there maybe an bug when applying the color to those windows.
also the courner rounding on those windows does not change.

before(retro/c64) :
screenshot2

after(retro/c64 modified border color and radius) :
screenshot

the square is the default border color and radius
the circle is after changing it from the default state

@actionless
Copy link
Member Author

i cant change my WM decorations.

if your WM not offering such option you can create a "fake" theme:

  1. generate oomox theme normally
  2. remove metacity-1 directory from the generated theme
  3. copy that directory from some other non-oomox theme

@actionless
Copy link
Member Author

and... when i change the border color... the faulty border on all tool style windows stays the same color as before. so i think there maybe an bug when applying the color to those windows.

are they applying after re-logging-in to the gnome?

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 18:50

are they applying after re-logging-in to the gnome?

ah.. yes they do but its still the wrong color... always the focused window corder even if unfocused

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 18:52

As for the metacity theme... i think gnome does not use that anymore?
most of my themes have no metacity folder. also if i delete that folder everything is still as intended

@actionless
Copy link
Member Author

so, just to confirm it's not using metacity:

when you're doing this sequence:

  1. change to some other theme
  2. remove metacity-1 from oomox theme
  3. choose back oomox theme
  4. relogin to gnome

you're still able to reproduce the issue or you're seeing some default Adwaita titlebars?

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 18:59

yes... but i can't resize the windows with csd anymore(okay.. this is because of your gtk.css changes) the buggy windows stay the same

screenshot from 2018-02-15 19-59-46

so indeed it is using a metacity theme.
ill try adding MetacityTheme=Arc to the index.theme file

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 19:7

okay.. no matter what i change it stays visually the same

@actionless
Copy link
Member Author

then i guess now gnome is reading them from gtk theme

does editing ./oomox/plugins/theme_oomox/gtk-theme/src/gtk-3.20/scss/widgets/_window.scss have any effect for the issue? because there i don't see any separate handling of CSD and non-CSD windows, while on your screenshots i see what CSD and non-CSD windows have slightly different decorations (so i've suggested what on non-CSD windows metacity being used)

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 19:55

for that i would have to compile it myself i think?

@actionless
Copy link
Member Author

you can just edit /opt/oomox/plugins/theme_oomox/gtk-theme/src/gtk-3.20/scss/widgets/_window.scss with sudo

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 19:58

nevermind... checked the source out. nothing to compile i think.

@actionless
Copy link
Member Author

i particularly recommend playing with the line 40 (.ssd) or you can do it in your ~/.config/gtk-3.0/gtk.css

i think that .ssd class is being used by gnome for non-CSD windows

so _window.scss is clearly missing something for unfocused .ssd windows, i am wondering if gnome understands something like .ssd:backdrop

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 20:36

changing .ssd does indeed change the windows in question... now im on the hunt for any documentation on that.. but good luck searching for searches with ssd as a term :D

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 20:57

that did the trick

.ssd & {
    box-shadow: 0 0 0 1px $wm_border_focused;
    &:backdrop {
        box-shadow: 0 0 0 1px $wm_border_unfocused;
     }
}

screenshot from 18 02

@actionless
Copy link
Member Author

From @jdamboeck on February 15, 2018 22:27

but the issue numixproject#121 is still there.. it applies the right color but there is a double border. the one from the ssd and inside that another one which does not include the titlebar.
with csd there are also 2 borders... but they include the window title too.

screenshot from 18 02

screenshot from 18 02 02

@actionless
Copy link
Member Author

if you set window decoration { border-width: 0; } it looks fine but mouse not resizing the edges?

@actionless
Copy link
Member Author

From @jdamboeck on February 16, 2018 0:29

yes

@actionless
Copy link
Member Author

and what about:

window decoration {
border-width: $spacing + 2px;
}

window {
border-radius: $roundness / 2;
}

?

@actionless
Copy link
Member Author

From @jdamboeck on February 16, 2018 0:34

The same as as without gtk.css

@actionless
Copy link
Member Author

you need to compile in your head that scss snippet into css using the real values from your preset :-)

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

i have a spacing of 3 and a roundness of 0
so my gtk.css is now:

window` decoration {
	margin: 0;
	border-width: 5px;
}

window {
	border-radius: 0;
}

@actionless
Copy link
Member Author

hm, mb i'll have a look how that part implemented in other themes

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

if i pick a default color presets and export it, i too get the same color as the titlebar... when i edit "focused window border" "unfocused window border" and "background" the strange double border is back again.

somehow the wrong color gets added if you edit one of the default presets. but unedited the default color presets generate themes as they should be.

edit: bit its not consistent either. sometimes it works sometimes the error is back. might depend on the colors?

maybe it helps if i send you the preset where its broken so you can test with that?

named it .txt so i can upload it to github
brainiac.txt

@actionless
Copy link
Member Author

that black color is coming from WM_BORDER_UNFOCUS=000000

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

Yeah but why.

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

https://www.reddit.com/r/archlinux/comments/6duktu/adding_colored_window_borders_in_gnome/di9htfj/

if i put that part in my gtk.css it works as intended. maybe it helps finding a solution.

@actionless
Copy link
Member Author

actionless commented May 23, 2018

could you please check if the same theme still works fine in openbox?
last time when i was playing with i remember what it was working either in one place or in another.
so if it will be like that again -- mb we can have to _window.scss files and choose one for gnome or for the rest as oomox export option

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

in openbox all is as intended

@actionless
Copy link
Member Author

splendid! could you prepare a pull request please?

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

what for? i did not do anything... maybe we did missunderstand each other? the theme works in openbox without me doing any changes.

@actionless
Copy link
Member Author

if i put that part in my gtk.css it works as intended
in openbox all is as intended

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

yeah.. but i don't really know where to apply the gtk.css changes to the oomox -gtk-theme code. never done any real gtk theming before. :D I am just trying do sherlock my way through it.

@actionless
Copy link
Member Author

actionless commented May 23, 2018

you can see similar class names in _window.scss which i mentioned few times above if you'll not manage to do so, i'll have some time mb on sunday to give it a try

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

I will try to sherlock some more... Maybe I'll get my feet in it.

@actionless
Copy link
Member Author

after that you could add SCSS to your CV ;P

@actionless
Copy link
Member Author

From @jdamboeck on Feb 16

i got scss partially... i am an webdev after all but done mostly plain css and less the last years. but i don't really know the gtk specific parts in all of this.

@actionless
Copy link
Member Author

i am not sure if it's solved in all the setups or not so better will leave it open

@actionless
Copy link
Member Author

@pandasauce please try if any of the solutions proposed above are working in your setup

if so, please prepare a PR or at least share a patch or so

nana-4 added a commit to nana-4/oomox-gtk-theme that referenced this issue Nov 10, 2018
- Expand window resize area
- Fix double border around windows
- Fix .solid-csd styling to properly display windows in tiling window managers

Fixes themix-project#52
Fixes themix-project/themix-gui#170
actionless pushed a commit that referenced this issue Dec 28, 2018
- Expand window resize area
- Fix double border around windows
- Fix .solid-csd styling to properly display windows in tiling window managers

Fixes #52
Fixes themix-project/themix-gui#170
actionless pushed a commit that referenced this issue Jun 16, 2024
- Expand window resize area
- Fix double border around windows
- Fix .solid-csd styling to properly display windows in tiling window managers

Fixes #52
Fixes themix-project/themix-gui#170
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant