themix-project / oomox-gtk-theme

Oomox-gtk-theme is a fork of Numix-gtk-theme with slightly different visual design choices and more customization capabilities. It can be used in Themix/Oomox app.
https://github.com/themix-project/oomox
GNU General Public License v3.0
46 stars 8 forks source link

<invalid> #51

Closed actionless closed 6 years ago

actionless commented 6 years ago

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/oomox#123

actionless commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago
  1. could you highlight the mistaken outline on your screenshot?

  2. i'll be happy to merge such change

actionless commented 6 years ago

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 commented 6 years ago

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

if so -- that could be a duplicate of this issue: https://github.com/actionless/oomox/issues/121 (Slack in that ticket AFAIK is using electron)

actionless commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

From @jdamboeck on February 15, 2018 19:7

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

actionless commented 6 years ago

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 commented 6 years ago

From @jdamboeck on February 15, 2018 19:55

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

actionless commented 6 years ago

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

actionless commented 6 years ago

From @jdamboeck on February 15, 2018 19:58

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

actionless commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

From @jdamboeck on February 15, 2018 22:27

but the issue #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 commented 6 years ago

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

actionless commented 6 years ago

From @jdamboeck on February 16, 2018 0:29

yes

actionless commented 6 years ago

and what about:

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

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

?

actionless commented 6 years ago

From @jdamboeck on February 16, 2018 0:34

The same as as without gtk.css

actionless commented 6 years ago

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

actionless commented 6 years ago

should look like that:

2018-02-16--1518741434_1912x719_scrot

actionless commented 6 years ago

From @jdamboeck on February 16, 2018 0:39

ah lol.. .. where do i put that?

actionless commented 6 years ago

i mean replace $roundness and and $spacing to real values and compute the math, so it will be css in the end, and put to gtk.css to try

actionless commented 6 years ago

From @jdamboeck on February 16, 2018 0:47

get's even worse screenshot from 18 02

actionless commented 6 years ago

weird, in my case the second border is the same color as titlebar