ubuntu / yaru

All Ubuntu Yaru GNOME themes
https://community.ubuntu.com/c/desktop/theme-refresh
GNU General Public License v3.0
1.36k stars 182 forks source link

19.04 dailies: logs and system-monitor are suddenly upsteam Gnome icons #1290

Closed ubuntujaggers closed 5 years ago

ubuntujaggers commented 5 years ago

Think there might have been a package name change or something? I'll keep an eye on this and see if it changes back to Yaru.

The new Gnome icons are good and the effect isn't jarring, so that proves the wisdom of desquircling our icons (because they can co-exist with the new Gnome ones):

image

image

...but we do have nice theme icons for these apps, so maybe we need to update the symlinks or something?

ubuntujaggers commented 5 years ago

Argh, and calculator too! It blended so beautifully with our house icons that I didn't even notice this one, lol:

image

Feichtmeier commented 5 years ago

:thinking: could you check which icon is being used in the .desktop file?

Check with

nano /usr/share/applications/org.gnome.Calculator.desktop

somewhere there is a line with

icon=...
ubuntujaggers commented 5 years ago

Oh, weird. Nano creates a new file when I do that.

If I navigate to the folder in Files and search for "calc", the only file in there is the one for LibreOffice. Doesn't seem to be a file for Calculator.

I'm running Disco Dingo which I installed clean yesterday after knackering my laptop a bit.

EDIT: Anyone got the same on Disco Dingo?

Feichtmeier commented 5 years ago

maybe it is named gnome-calculator.desktop?

ls /usr/share/applications | grep gnome
ubuntujaggers commented 5 years ago

Nope, it's vanished!

gnome-applications-panel.desktop gnome-background-panel.desktop gnome-bluetooth-panel.desktop gnome-color-panel.desktop gnome-control-center.desktop gnome-datetime-panel.desktop gnome-default-apps-panel.desktop gnome-disk-image-mounter.desktop gnome-disk-image-writer.desktop gnome-display-panel.desktop gnome-info-overview-panel.desktop gnome-initial-setup.desktop gnome-keyboard-panel.desktop gnome-language-selector.desktop gnome-mouse-panel.desktop gnome-network-panel.desktop gnome-notifications-panel.desktop gnome-online-accounts-panel.desktop gnome-power-panel.desktop gnome-printers-panel.desktop gnome-privacy-panel.desktop gnome-region-panel.desktop gnome-removable-media-panel.desktop gnome-search-panel.desktop gnome-session-properties.desktop gnome-sharing-panel.desktop gnome-shell-extension-prefs.desktop gnome-software-local-file.desktop gnome-sound-panel.desktop gnome-thunderbolt-panel.desktop gnome-ubuntu-panel.desktop gnome-universal-access-panel.desktop gnome-user-accounts-panel.desktop gnome-wacom-panel.desktop gnome-wifi-panel.desktop org.gnome.baobab.desktop org.gnome.Calendar.desktop org.gnome.Cheese.desktop org.gnome.DejaDup.desktop org.gnome.DiskUtility.desktop org.gnome.Evince.desktop org.gnome.Evince-previewer.desktop org.gnome.Evolution-alarm-notify.desktop org.gnome.FileRoller.desktop org.gnome.font-viewer.desktop org.gnome.gedit.desktop org.gnome.Mahjongg.desktop org.gnome.Mines.desktop org.gnome.Nautilus.desktop org.gnome.PowerStats.desktop org.gnome.Screenshot.desktop org.gnome.seahorse.Application.desktop org.gnome.Shell.desktop org.gnome.Shell.PortalHelper.desktop org.gnome.Software.desktop org.gnome.Software.Editor.desktop org.gnome.Sudoku.desktop org.gnome.Terminal.desktop org.gnome.Todo.desktop org.gnome.Totem.desktop org.gnome.tweaks.desktop remmina-gnome.desktop

Feichtmeier commented 5 years ago

Ahhh it's a snap

ubuntujaggers commented 5 years ago

Good diagnosis.

@jhenstridge @popey: for core snaps that ship with Ubuntu by default, when Yaru has an icon for the app, would it be possible to build it into the snap?

ubuntujaggers commented 5 years ago

Ahhh, I believe ImageMagick is also a snap? I was trying to test this work in progress:

image

...but it didn't pick up the new icon when I did ninja install and I can't see ImageMagick in the folder above.

Feichtmeier commented 5 years ago

@kenvandine @jhenstridge is it possible to bake in the yaru icon for these gnome snaps?

jhenstridge commented 5 years ago

There's no easy automatic way to push these icons into the snaps, and doing so doesn't really fix the problem: they might look better on one set of desktops, but will now be out of place on a different set of desktops.

There's currently no way to properly use icon names (rather than full file names) in snap .desktop files, which would be the right way to make the icons themeable. I don't think we've got a good plan for that yet: it brings up similar namespacing issues to various other desktop related features (e.g. we don't want the gnome-calculator snap to be able to provide a themed icon for gedit).

Feichtmeier commented 5 years ago

@jhenstridge of that's sad news :( We, especially @ubuntujaggers, spent a lot of time creating these icons and now it looks like they won't be used. Couldn't ubuntu ship with the deb versions of these apps? I don't think it's that important for the user to receive updates via snap at these quiet common desktop tools. Alternatively: aren't those snaps only preinstalled on the Standart ubuntu? If yes, it would make more sense to me to use the yaru icons instead of the upstream icons ("hardcoded")

ubuntujaggers commented 5 years ago

@jhenstridge

There's no easy automatic way to push these icons into the snaps, and doing so doesn't really fix the problem: they might look better on one set of desktops, but will now be out of place on a different set of desktops.

But some of the desktops are Ubuntu ones :)

This is a limitation of snaps currently, they come with their own non-themeable icon which means they can only match one theme at most. But in this case, if the snaps are made by Ubuntu collaborators and ship with Ubuntu as core apps, IMO it's better for them to match the default Ubuntu theme until they can adapt to the user's chosen one. It's like we get to choose just one theme for a snap but have chosen a non-Ubuntu one in these cases, which seems strange.

I fully accept it would be unwise to put our original icons in these snaps, but the removal of squircles makes Yaru icons more suitable for general use IMO. When an app doesn't match its neighbours because it's a snap, I don't think a current Yaru icon is worse than a Gnome one. And in this scenario, at least the subset of desktops where the snaps do match perfectly, is our own userbase. I would say one advantage of the new non-squircle icons (which weren't my idea) is that they work better outside their intended context, and that makes them suitable contenders for snaps. For instance: we have a new Yaru icon for Remmina, and upstream have actually asked us to submit a PR for it to be the upstream icon too. I don't think our design would have worked as a non-theme icon if we were still using squircles, but now it does.

Alternatively, is there a halfway house where a snap can ship with just two icons: the Yaru one for users of Yaru, and the default one for everyone else?

jhenstridge commented 5 years ago

@Feichtmeier, @ubuntujaggers: I'd prefer to get the proper icon theme support working. We're not just missing out on Yaru icons with the current status quo: we're also missing out on icon choice based on size.

Feichtmeier commented 5 years ago

@jhenstridge also symbolic icons are missing, upstream and in icon themes, Which looks strange in gnome shell and many gtk apps

clobrano commented 5 years ago

@jhenstridge is there an ongoing discussion about this topic we can follow and, hopefully, contribute to in order to complete the theme support?

ubuntujaggers commented 5 years ago

Hi guys,

Unfortunately, these icons landing on Ubuntu desktop does seem to confirm the fact that they don't shine on the Ubuntu launcher (this isn't a fault of the new Gnome icons - they're simply not designed for 48px and don't claim to be):

image

It's a bit of an anti-aliasing festival at this size:

image

Is there a lot of pressure to include these for 19.04, and will any more be added at this late stage? It's not just us in Yaru... a lot of Ubuntu users prefer icon themes like Papirus, Moka, etc., and these sore-thumb snaps are going to be an annoyance for anyone who uses a theme.

ubuntujaggers commented 5 years ago

@jhenstridge @clobrano @madsrh @Feichtmeier

I've just been using tweaks tool to look at the Adwaita icon set now that lots of the new icons are part of it. I would say my personal position has hardened. If we don't want a big, visible drop-off in the quality of the desktop, I don't think it's a sensible option to use icons that are optimised for 32px and 64px only for core default apps, when a 48px launcher plays such a large part in Ubuntu's workflow and first impression on new users.

This isn't a mockup, I've applied Adwaita as my icon theme.

image

These are well-executed designs but appear very fuzzy when rendered at 48px, which wouldn't matter too much for some styles, but is a deal-breaker for this one. Chunky, geometric icons with high contrast on the background need to be sharp. These are not sharp on the Ubuntu launcher :(

For comparison, here's how Yaru (which is optimised for 48px too) looks in the same context:

image

Sharp! Here's another comparison. Gnome calendar:

image

Yaru calendar:

image

IMO, at 48px, there's no comparison. Every time we lose a Yaru icon to a Gnome icon because Ubuntu ships a snap before the icon thing is resolved, there will be a loss of quality. Not just in terms of visual consistency but also in terms of objective image quality.

If Ubuntu makes a political decision to retire house icons and use Gnome icons, that's fine, but then there's an important job to either a) rethink the Ubuntu desktop to not use 48px icons by default, or b) pixel fit the Gnome svgs so we have a version for each that works at 48px (which, tbh, I could do if Ubuntu chose that path). But using Gnome icons isn't something we get for free. It's something we would have to prepare for and adapt to. Their icons weren't designed for Ubuntu and, worse, were actually designed with assumptions that Ubuntu doesn't share. So we can't just throw 'em in without hurting the visual quality of the desktop.

EDIT: For further illustration, here's a comparison of the current Gnome icon for gallery, and then the same icon where I've edited the svg to optimise it for our launcher size of 48px:

Current:

image

Optimised for Ubuntu launcher:

image

Hopefully you can see the clear advantage (no pun intended, lol) of using icons optimised for the size they will be shown at. But this approach requires Ubuntu to fork Gnome icons and edit each of them manually. It's definitely doable, but if we're going to have and maintain our own icon set, why not use our own (already almost finished) set of designs?

Edit 2: I used a slightly different SVG I found on Gitlab which is why the lens looks different, that's not a design choice. But you can see it's sharper.

ubuntujaggers commented 5 years ago

Here's a quick side-by-side showing the current Gnome icons I screenshotted above, next to tweaked versions of the Gnome svgs tweaked for 48px (noting that the svg I used for EoG isn't identical to the one in the screenshot - I don't know whether the one I found was earlier or later but the lens is different). Current icons are on the left, optimised are on the right:

image

jhenstridge commented 5 years ago

@jhenstridge is there an ongoing discussion about this topic we can follow and, hopefully, contribute to in order to complete the theme support?

I had some thoughts about this, and put together a proposal on the Snapcraft forum here:

https://forum.snapcraft.io/t/proposal-support-the-icon-theme-spec-for-desktop-icons/10676?u=jamesh

I am not sure when I'll get a chance to work on this, but I think it would cover the use cases mentioned here:

  1. use icon names that can be overriden by themes like Yaru
  2. install all the icons provided by the app: multiple sizes, symbolic version, etc.
clobrano commented 5 years ago

@jhenstridge, thanks!

clobrano commented 5 years ago

Closing this since it's up to the snap to allow the theme icon to be used