ubuntu / yaru

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

Inconsistent rounded corners for file icons #866

Closed matthewpaulthomas closed 5 years ago

matthewpaulthomas commented 6 years ago

yaru-theme-* 18.10.4, Ubuntu Cosmic

The generic icons for various file types have rounded corners.

icon for text file icon for mp3

When a file has no single visual representation, such as a sound file, a font, or a database, this doesn’t really matter one way or another. But when a file does have a well-known visual representation, such as an image, movie, or PDF, it’s inconsistent.

One example of this inconsistency is that the shape of an image icon differs depending merely on whether Files is able to show a thumbnail of it.

icons for mona lisa image

Another example is that a LibreOffice Writer document has an icon with rounded corners, but if you save the document as a PDF, suddenly its corners become square — when really they were square all along.

icons for test document

App icons in this theme have rounded corners. But that is, if anything, another reason for document icons not to have rounded corners, to help distinguish between apps and documents.

[Originally reported in the Yaru forum.]

Feichtmeier commented 6 years ago

I really don't see where the problem is. These rounded corners are everywhere in our theme and look very sexy to me. Thumbnails/previews of the real file like with PDF and images are just that a preview and that's why they don't have a round corner because the original neither has rounded corners

Thus I would strongly disagree to change this, since the round corners fit perfectly to the rest of this theme

matthewpaulthomas commented 6 years ago

A common issue with consistency is being consistent with the wrong things. In general, it’s better to be consistent with things nearby or more closely related, than with things far away or more distantly related, because it reduces mental load.

For example, it’s more important for document icons of the same type to be consistent with each other than with other document icons. And it’s more important for document icons in general to be consistent with each other than to be consistent with app icons or other parts of the theme.

If no document icons had thumbnails, this would be less of an issue. But given that many do have thumbnails (which is a good thing!), it’s better for non-thumbnail icons to be consistent in style with thumbnail icons than to be consistent with app icons.

For example, in iOS Files, document icons have very slightly rounded corners, the same radius regardless of whether they have thumbnails or not. It’s much smaller than the rounded corners for app icons.

Similarly in Android 7.1 and later, circular icons are used specifically for apps, not for documents.

Feichtmeier commented 6 years ago

I have to admit that I would really prefer the current styling :see_no_evil: icons are symbols anyways and not a copy of the IRL document. So I don't think this argument really fits I don't know how others feel though @madsrh @clobrano @godlyranchdressing

Feichtmeier commented 5 years ago

I'd like to add something ...

The content view in nautilus shows thumbnails for PDFs, videos and pictures (because its a content view and thats cool) But only after a certain zoom threshold is passed. Before it has the regular mime-type icon.

Imho having a thumbnail is a valid exception for the round icons, since it is a thumbnail and you don't want it to be cut. A symbol on the other side needs brain work anyways since your brain needs to decode that symbols. :haircut_man:

I think those different looking icon shapes of the OSX icons look far worse and chaotic. Almost like you mixed icon themes.

peek 2018-11-07 13-03 peek 2018-11-07 13-02

clobrano commented 5 years ago

App icons in this theme have rounded corners. But that is, if anything, another reason for document icons not to have rounded corners, to help distinguish between apps and documents.

document icons are taller than app icons, so I don't see the problem here.

I agree on the preview file issue, I think it would be better to have the same size and format for both mime types and preview, but trying to keep the current rounded format

madsrh commented 5 years ago

I think it would be better to have the same size and format for both mime types and preview, but trying to keep the current rounded format

So shrink the preview or make the mime icons larger?

image

Feichtmeier commented 5 years ago

I agree that the thumbnails are prbly too big, but cutting the edges might not even be possible (not with CSS I think). It also looks a bit weird to me. Also, I don't know if shrinking the thumbnails is possible from our side. This is adwaita by the way:

image Edit: and this is from my Samsung android phone just for reference, that it seems like it is not that uncommon to have that kind of mime-type icons:

photo_2018-11-07_13-51-26 photo_2018-11-07_13-54-50

clobrano commented 5 years ago

So shrink the preview or make the mime icons larger?

yes, that would be cool

I agree that the thumbnails are prbly too big, but cutting the edges might not even be possible (not with CSS I think).

I agree, I think it's a Nautilus thing. If so, it would be complex, since Nautilus should work accordingly to the Theme, which is not supposed to do, I believe.

A simpler solution would be to just adapt the size of the preview, instead of the full shape. It's probably a Nautilus thing as well, but easier to fix

Finally, I always though Android file managers are not good enough, compared to desktop ones, I would mind OSX better :)

Feichtmeier commented 5 years ago

Yeah from the upstream/downstream perspective this is not a good idea since hopefully Ubuntu uses 100% of the up-to-date gnome apps in the future without any modifications.

(I don't like that Android file manager neither, I just wanted to point out that Suru mime types are not the only ones with rounded icon backgrounds)

So what shall we do?

clobrano commented 5 years ago

I'll ask Nautilus team about it

clobrano commented 5 years ago

IRC #nautilus channel was very helpful :+1:.

Thumbnails generation is up to thumbnailers

$ tree /usr/share/thumbnailers/
/usr/share/thumbnailers/
├── evince.thumbnailer
├── gnome-font-viewer.thumbnailer
├── gsf-office.thumbnailer
└── totem.thumbnailer

Nautilus only asks gnome-desktop which one to rely on for each file type.

Thumbnailer spec requires thumbnailers to manage a --size flag (range 128:256), then Nautilus resizes the created picture if needed. This size is only the picture resolution though, not the shape of the picture (i.e. not whether the preview will have rounded corners or not).

We should modify each thumbnailer then, but I believe this means that the issue is an "upstream one", in the sense that we should work on it upstream and Yaru is not the right place. I propose to keep in mind this ticket and eventually discuss what to do, but to close it here.