Skip to content

Upstream compliant theming - #322

Merged
isantop merged 27 commits into
master_eoanfrom
upstream-compliant_eoan
Oct 2, 2019
Merged

isantop merged 27 commits into
master_eoanfrom
upstream-compliant_eoan

Conversation

@isantop

@isantop isantop commented Sep 23, 2019 •

Copy link
Copy Markdown
Contributor

It is expected that 19.10 will ship with the new Adwaita-based Pop theme to increase compatibility with GNOME applications and reduce bugs and breakages. This new codebase represents this work to produce a "compatible" theme which still presents a distinctive Pop identity to the OS.

Architectural changes:

  • This theme is based on the Ubuntu Yaru theme, which brings in the GNOME Shell and GTK Themes under one basket. It includes a light and a dark GNOME Shell theme which will be built automatically and can be set up to automatically switch when the dark theme is selected, pending this functionality in GNOME Shell.
  • Built with Meson instead of make.
  • Includes a Sound theme from Yaru (which may or may not be selected by default, and can be set as such in pop-default-settings
  • Retains the current separated Icon theme.

Visual Updates

  • A widely simpler appearance. Pop is still rather flatter than Adwaita, but retains much of its padding and sizing.
  • Reduced reliance on shadows for element separation, as this has a tendency to break many things.
  • More contrast-compliance in many respects, particularly with the dark theme, which helps ensure usability for all users.
  • Brighter, more visible colored backgrounds in the dark theme, as this helps aid contrast and reduce optical vibrations of heavily saturated colors against dark backgrounds.

Current TODOs with this Theme:

  • Documentation updates, including new or removed screenshots from the README.
  • Documentation updates with build instructions.
  • Creating a disco-compatible testing branch to allow employees to dogfood the new theme and provide feedback about visuals and design.
  • Moving some style changes from _common.scss to _tweaks.scss. This should not affect the appearance of the theme.
  • Dark theme list bar selected text backdrop is very low-contrast
  • Investigate Nautilus list view outline issue.

Ian Santopietro added 9 commits September 16, 2019 14:21
This is the result of making sure Pop does not break upstream GNOME apps. It's very
Pop-y, but with metrics and features that give it 100% feature parity with Adwaita
which helps ensure upstream apps will not break if a user is using this theme. It's
current incomplete, so it should be considered alpha-quailty for now.
@isantop
isantop requested review from a team, WatchMkr and mmstick September 23, 2019 20:43
@isantop isantop self-assigned this Sep 23, 2019
@isantop

isantop commented Sep 23, 2019 •

Copy link
Copy Markdown
Contributor Author

Please be aware that checkboxes in Repoman are too-close together with this theme. This is fixed in pop-os/repoman#20

Ian Santopietro added 7 commits September 23, 2019 15:49
These have been outdated for a long time anyway, and don't represent an accurate
image of the theme. Lots of images of the theme are available in screenshots online,
and these represent an undue maintenance burden.

Also point the logo towards master
@brs17

brs17 commented Sep 24, 2019

Copy link
Copy Markdown
Contributor

Documentation updates should also include replacing the build instructions involving make.

@brs17

brs17 commented Sep 24, 2019

Copy link
Copy Markdown
Contributor

Will there be slim variations for these themes?

@brs17

brs17 commented Sep 24, 2019

Copy link
Copy Markdown
Contributor

On 19.10 where I have installed the appropriate .debs as provided I am seeing odd overlay/padding on the system menu as well as the "notification menu"(not sure of proper name for it). I see this in both the normal theme and dark theme.

Screenshot from 2019-09-24 07-40-24
Screenshot from 2019-09-24 07-40-44
Screenshot from 2019-09-24 08-21-32
Screenshot from 2019-09-24 08-21-38

This is the configuration I have in gnome-tweaks:

Screenshot from 2019-09-24 08-15-24

@brs17

brs17 commented Sep 24, 2019

Copy link
Copy Markdown
Contributor

Not sure if it is intentional, but the column selections ("Name", "Size", "Modified", etc) in nautilus are the exact same color as the background. This may make it unclear that the headers are selectable.

Screenshot from 2019-09-24 07-48-20

This also appears to be the case in the dark theme as well.

@WatchMkr

Copy link
Copy Markdown
Contributor

Quick notes - stuff to cover with Kate:

  • The top bar feels too tight
  • Bad contrast on un-focus windows (side bars in particular but also select a file in nautilus and unfocus the window).
  • Shell calendar menu text hierarchy feels off (today's month, day, year feel too small). Would love to remove year here if possible. Otherwise, let's take a look at what we can do to improve the sizes.
  • Not keen on the arrow pointing to the clock and user menu.

@isantop

isantop commented Sep 24, 2019

Copy link
Copy Markdown
Contributor Author

@brs17 That is a side effect the the Pop-testing theme installation, and doesn't happen when the Pop theme is upgraded to the new version.

@WatchMkr WRT:

Not keen on the arrow pointing to the clock and user menu.

This is actually a lot more consistent with GTK, where popovers also use the arrow. I think a large part of what makes them not so great is the aforementioned bug with the -testing theme installation. On the actual installed version, the effect works a lot better and it helps provide a visual hierarchy for where the popup is coming from. I think we should wait until it's easier to test with the way that will actually look before we make a final decision on that.

One important note is that the visual glitch is caused by a bug in the current Pop shell theme directly related to removing those arrows. When the system theme doesn't use the arrows, and you select a shell theme that does, it creates that issue. Keeping the arrows prevents this sort of glitch and ensures that the Pop theme doesn't break other themes.

@isantop

isantop commented Sep 24, 2019

Copy link
Copy Markdown
Contributor Author

Will there be slim variations for these themes?

@brs17 That might be possible, although the Slim themes will cause breakage with apps because of padding changes.

@brs17

brs17 commented Sep 24, 2019

Copy link
Copy Markdown
Contributor

Will there be slim variations for these themes?

@brs17 That might be possible, although the Slim themes will cause breakage with apps because of padding changes.

πŸ‘ If that is the case, we will want to remove the Slim toggle from gnome-control-center.

@WatchMkr

Copy link
Copy Markdown
Contributor

Take a closer look at the orange used in the light theme with Kate. Dark theme colors come across as smooth and fitting. Taffy instead of hard candy. Light theme colors (especially orange) look saturated. Hard candy instead of taffy.

@isantop

isantop commented Sep 24, 2019

Copy link
Copy Markdown
Contributor Author

@WatchMkr updated Debs with the latest fixes:

pop-testing.zip

@brs17

brs17 commented Sep 29, 2019

Copy link
Copy Markdown
Contributor

dpkg errors still cause failure to add the new theme via the upstream-compliant staging repository.

Errors when trying to add the theme:

system76@pop-os:~$ sudo apt upgrade 
Reading package lists... Done
Building dependency tree       
Reading state information... Done
Calculating upgrade... Done
The following packages were automatically installed and are no longer required:
  gir1.2-mutter-4 libdouble-conversion1 libebook-contacts-1.2-2 libecal-1.2-19
  libgnome-desktop-3-17 libip6tc0 liblirc-client0 libmessaging-menu0
  libmutter-4-0 libmysqlclient20 libpoppler87
  libreoffice-avmedia-backend-gstreamer libtagc0 libvpx5 multiarch-support
Use 'sudo apt autoremove' to remove them.
The following packages will be upgraded:
  pop-gnome-shell-theme pop-gtk-theme
2 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Need to get 528 kB of archives.
After this operation, 2,043 kB disk space will be freed.
Do you want to continue? [Y/n] 
Get:1 http://apt.pop-os.org/staging/upstream-compliant eoan/main amd64 pop-gtk-theme all 5.0.0~1569615259~19.10~d3d09d6 [416 kB]
Get:2 http://apt.pop-os.org/staging/upstream-compliant eoan/main amd64 pop-gnome-shell-theme all 5.0.0~1569615259~19.10~d3d09d6 [113 kB]
Fetched 528 kB in 1s (378 kB/s)                 
(Reading database ... 234259 files and directories currently installed.)
Preparing to unpack .../pop-gtk-theme_5.0.0~1569615259~19.10~d3d09d6_all.deb ...
Unpacking pop-gtk-theme (5.0.0~1569615259~19.10~d3d09d6) over (4.1.4~1560290633~
19.10~f75e86a) ...
dpkg: error processing archive /var/cache/apt/archives/pop-gtk-theme_5.0.0~15696
15259~19.10~d3d09d6_all.deb (--unpack):
 trying to overwrite '/usr/share/themes/Pop/gnome-shell/gnome-shell.css', which 
is also in package pop-gnome-shell-theme 4.0.12~1559836915~19.10~d27d536
dpkg-deb: error: paste subprocess was killed by signal (Broken pipe)
Preparing to unpack .../pop-gnome-shell-theme_5.0.0~1569615259~19.10~d3d09d6_all
.deb ...
Unpacking pop-gnome-shell-theme (5.0.0~1569615259~19.10~d3d09d6) over (4.0.12~15
59836915~19.10~d27d536) ...
Errors were encountered while processing:
 /var/cache/apt/archives/pop-gtk-theme_5.0.0~1569615259~19.10~d3d09d6_all.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

@brs17

brs17 commented Sep 29, 2019

Copy link
Copy Markdown
Contributor

Installing it still seemed to work just fine with an apt insall pop-gtk-theme. πŸ˜•

@brs17

brs17 commented Sep 29, 2019

Copy link
Copy Markdown
Contributor

The Cancel button on the login screen is the exact same color as the background of the login page and therefore the only part of the button that is really visible is the text itself.

@brs17

brs17 commented Sep 30, 2019

Copy link
Copy Markdown
Contributor

In GNOME Calendar, when creating a new event, the label which shows which calendar the event will be added to, might be too low contrast.

Notice where it says "Personal" in the top bar.

19.10:
calendar-text-contrast

19.04:
calendar

@brs17

brs17 commented Sep 30, 2019

Copy link
Copy Markdown
Contributor

@brs17 Nautilus list-view header borders

Screenshot from 2019-09-24 10-48-06

This does look better.

I do notice that the text color of the headers looks very light, almost as if they are deactivated/not clickable. Might a slightly darker font be better here?

@brs17

brs17 commented Sep 30, 2019

Copy link
Copy Markdown
Contributor

Per offline discussion, let's make sure that updates on a system work well...let's potentially solve this issue #322 (comment) and then I'll give the πŸ‘ and we can merge it to master_eoan. From there, individual issues will be created for issues found.

This already looks really good and I think a lot of people will love it!

@isantop
isantop marked this pull request as ready for review September 30, 2019 20:46
@isantop

isantop commented Sep 30, 2019

Copy link
Copy Markdown
Contributor Author

#322 (comment) has been addressed, and everything appears to work. The theme is correctly updated and applied on both upgrades and new installations.

@isantop

isantop commented Sep 30, 2019

Copy link
Copy Markdown
Contributor Author

@brs17 I already created issues for the other outstanding notes.

@brs17

brs17 commented Oct 1, 2019

Copy link
Copy Markdown
Contributor

I still got a warning, but after a reboot, the theme seems to be working just fine.

Screenshot from 2019-09-30 18-41-43

@isantop

isantop commented Oct 1, 2019

Copy link
Copy Markdown
Contributor Author

@brs17 What do you get for output of ls -R /usr/share/themes/Pop/gtk-3.22

@isantop
isantop merged commit c88d6eb into master_eoan Oct 2, 2019
@isantop
isantop deleted the upstream-compliant_eoan branch October 2, 2019 16:02
isantop added a commit that referenced this pull request Jan 10, 2020
* Big change. This makes Pop more upstream-compliant

This is the result of making sure Pop does not break upstream GNOME apps. It's very
Pop-y, but with metrics and features that give it 100% feature parity with Adwaita
which helps ensure upstream apps will not break if a user is using this theme. It's
current incomplete, so it should be considered alpha-quailty for now.

* Install dark theme too

* Render assets, style scales

* Add elementary widgets styling

* Fix Search results width

* Ignore build obj folder

* Calendar fixes

* Menu fixes

* Use dark theme for GDM

* Fix broken GResource

* Remove screenshots from README

These have been outdated for a long time anyway, and don't represent an accurate
image of the theme. Lots of images of the theme are available in screenshots online,
and these represent an undue maintenance burden.

Also point the logo towards master

* Fix broken GResource for light theme too.

* Shell: Calendar day fixes

* Shell: Contrast improvement on close button

* Fix large user icon in system menu

* Fix lintian error with changelog

* Documentation updates

* Move slider styling out of _common and into Tweaks for easier maintenance

* Improve dark theme selected item backdrop foreground color

* Fix borders on Nautilus list-view headers

* Correct typo with color pallete

* Fix old color functions and replace with rgba

* Fix path issue with assets

* Attempt to fix upgrade errors with debian packaging.

* Fix theme installation error.

* Make sure we're hitting the correct GNOME Shell theme version

* Remove Extra files.
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.

4 participants