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

Release Yaru 19.04 as third variant "Yaru-Ambiance" #1532

Closed
wants to merge 21 commits into from

Conversation

clobrano
Copy link
Member

@clobrano clobrano commented Sep 16, 2019

After the change of the light variant, in order to ease the transition out, release the Yaru 19.04 (sometimes called "mixed" theme) as third variant.

  • update build scripts to install a third them Yaru-ambiance
  • replace light variant tweaks with headerbar sass file
  • add a dark variant (which is the same as Yaru-dark) for apps that prefers dark variant
  • fix different titlebutton padding between ambiance and -dark

closes #1494

clobrano and others added 11 commits September 11, 2019 09:56
- create an ambiance folder, copy of the dark one
- modify ambiance build script to generate a third variant named Yaru-ambiance
- add _headerbar.scss stylesheet from lighter-headerbar-v8:_tweaks.scss
- append headerbar stylesheet to Ambiance gtk.scss
Nautilus floating-bar background is transparent, so the label is hard to
see if there's some text behind it.

Fixing it setting the background to solid, using $bg_color value

Also fix text formatting, removing empty spaces.
Firefox was using a unfocused tabs label color for light headerbar.

Fixing it re-appling the apps specific code from lighter-headerbar-v8
branch.
Currently tweaks is a light variant of headerbar sass, so it is useless
to include both. Keeping headerbar only now.
- add $ambiance variable, that default to null
- set Firefox menubar style only if $ambiance is true
- adaptive transparency stopped to work with gnome shell 3.32
- a permanently opaque dock looks quiet heavy in "window not maximized" situations
- adding a soft transparency lowers the heavyness of the dock/panel combination a bit
The text-button.toggle.checked did not have the same background color as
the other button in the headerbar.

Moreover, removed unnecessary double definition of *.toggle widgets
in bothj menu/context-menu and in menubar use fg_color in place of a
too strong selection color
This variant is equal to yaru-dark, but it the name yaru-ambiance-dark
is neede for apps configured to prefer dark.
@Feichtmeier
Copy link
Member

Feichtmeier commented Sep 16, 2019

Some stuff I've noticed:

  • Headerbar buttons look unproportionality strong pressed
    grafik

  • Titlebuttons padding changes between dark and light variant, because for the dark theme you get the titlebutton styling from _tweaks.scss and for the ambiance version from headerbar.scss were I made the mistake to remove the padding from all titlebuttons and not only min/max/close
    --> I would recommend to remove all the code from _headerbar.scss that is not ambiance only code

  • media applications that prefer the -dark suffix do not change to the dark variant like "videos" or "photos"

  • entries in linked boxes are bigger than buttons
    grafik
    grafik

some entries where missing from 3.0 folder and in gresource for the dark
variant of ambiance (which is equal to yaru-dark)
@Feichtmeier
Copy link
Member

Could you please drop those additional box-shadows and just set 1 color for the buttons and then use the button mixins?
It looks horribly inconsistent with the rest of the buttons

@clobrano
Copy link
Member Author

Could you please drop those additional box-shadows and just set 1 color for the buttons and then use the button mixins?
It looks horribly inconsistent with the rest of the buttons

Ok

@Feichtmeier
Copy link
Member

Could you please drop those additional box-shadows and just set 1 color for the buttons and then use the button mixins?
It looks horribly inconsistent with the rest of the buttons

Ok

There is $_headerbar_button_bg at the top of the file - just use this for all the @include button(normal ... ) @include button(active ...) @include button(.....) parts, because the mixins expect one color as input no matter of the state

@clobrano
Copy link
Member Author

image
image

@clobrano
Copy link
Member Author

entries in linked boxes are bigger than buttons

this is Image viewer, right? The app is not taking the right varian here. Applying the dark one manually it works fine

Feichtmeier added a commit that referenced this pull request Sep 17, 2019
- lighten up the headerbar like in #1532
- import changes from master that were made in the meantime to titlebutton fix, app notifications and gtk 3.28 switches
- remove double imports and double definitions
@Feichtmeier
Copy link
Member

I think the reason for the dark variant not working for videos and eye of gnome and so on... media apps... is that it searches for "gtk-dark.css" but there is none. Instead there is "gtk-ambiance-dark.css"

@clobrano
Copy link
Member Author

clobrano commented Sep 17, 2019

this branch started with gtk-dark only and didn't work, this is why I added later gtk-ambiance-dark, which is still not working of course.

Also, the gtk-dark.css is still in. The goal would be to release only ambiance and keep using only gtk-dark for both themes

@Feichtmeier
Copy link
Member

@clobrano maybe it's something with that gresource file names

@clobrano
Copy link
Member Author

clobrano commented Sep 17, 2019

Maybe. I expect gtk to look for the same name with -dark postfix. I think I replicated already all the gresource entries, but there must be something missing. @8none1 should have already talked with @didrocks about that, I'll ping them tomorrow

- lighten up the headerbar like in #1532
- import changes from master that were made in the meantime to titlebutton fix, app notifications and gtk 3.28 switches
- remove double imports and double definitions
- revert aubergine to perma, like in master
- use the same border-color as in "yaru_headerbar" mixin
@clobrano clobrano mentioned this pull request Sep 17, 2019
In order to keep the right color combination for dark variant let's use
the original tweaks sass file in place of headerbar for the ambiance
dark variant
…biance

Dark variant uses a dark backdrop widget border in headerbar. Use the
same in Ambiance.

This also adjust the look and feel of entries linked to buttons (see
Image viewer). Not perfect, to be reviewed eventually.
@clobrano clobrano mentioned this pull request Sep 20, 2019
clobrano pushed a commit that referenced this pull request Sep 20, 2019
- lighten up the headerbar like in #1532
- import changes from master that were made in the meantime to titlebutton fix, app notifications and gtk 3.28 switches
- remove double imports and double definitions
@clobrano clobrano closed this Sep 23, 2019
@clobrano clobrano mentioned this pull request Oct 3, 2019
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

Successfully merging this pull request may close these issues.

Provide a mixed theme
2 participants