http://bugzilla.opensuse.org/show_bug.cgi?id=1043604 Oliver Kurz changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1043616 -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1043604 http://bugzilla.opensuse.org/show_bug.cgi?id=1043604#c2 Yifan Jiang changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fcrozat@suse.com, | |lnussel@suse.com, | |sreeves@suse.com, | |yfjiang@suse.com --- Comment #2 from Yifan Jiang --- For the icon size, maybe it's better to make it consistent with SLE: https://build.opensuse.org/package/rdiff/home:yfjiang:branches:openSUSE:Leap:42.3/glib2-branding-openSUSE?opackage=glib2-branding-openSUSE&oproject=openSUSE%3ALeap%3A42.3&rev=2 I scratched a demo that we can see the effect by installing gio-branding-openSUSE here: http://download.opensuse.org/repositories/home:/yfjiang:/branches:/openSUSE:... The thing I am not sure is - this will change the overall icon size including those in nautilus file browser. Though SLE users have no issue with that, I am not sure if the change is acceptable for the coming Leap42.3? For example, there might be Leap automation testcases rely on the current large icon? Do we have further any opinions and input if we should push the change by now? -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1043604 http://bugzilla.opensuse.org/show_bug.cgi?id=1043604#c4 --- Comment #4 from Ludwig Nussel --- does this change also affect the default gnome shell? -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1043604 http://bugzilla.opensuse.org/show_bug.cgi?id=1043604#c6 --- Comment #6 from Yifan Jiang --- Hi Ludwig, Dimstar, Thanks for the comments! Yes, it is not only about Classic session. The setting will impact nautilus icon in GNOME default session as well. For the particular case:

https://openqa.opensuse.org/tests/449351#step/nautilus/4

There is a nautilus-1-sle-20160520 needles seems matching small icons, probably the case will not fail after patching :-) If there is a need, either chingkai/grace/I could update the nautilus test case once it fails. I'll push today if nobody is against it. -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1043604 http://bugzilla.opensuse.org/show_bug.cgi?id=1043604#c7 Yifan Jiang changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |okurz@suse.com Flags| |needinfo?(okurz@suse.com) --- Comment #7 from Yifan Jiang ---

Starting SLE classic desktop .. plus a redundant entry "Home.desktop" which should be hidden.

For the Home.desktop, it's possible created by plasma5-workspace, if the user has had a login to "KDE Plasma Workspace" session before, 2 extra desktop files are created Home.desktop and trash.desktop. Subsequent logging in of GNOME will have those two files on the desktop. Hi Oliver, does it happen when a new user logs into GNOME in the first place? -- You are receiving this mail because: You are on the CC list for the bug.

http://bugzilla.opensuse.org/show_bug.cgi?id=1043604 http://bugzilla.opensuse.org/show_bug.cgi?id=1043604#c9 Yifan Jiang changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED Flags|needinfo?(okurz@suse.com) | --- Comment #9 from Yifan Jiang --- Hi Oliver, everyone, I am closing this bug since it seems ended to me: 1. the icon size change has been updated in GNOME:Factory, next release of Leap will have it. 2. 42.3 seems not prone to have it fixed to break consistency, for details please see the comment in the SR#511159. 3. It looks nobody is against Home.desktop is brought by plasma5-workspace, I'll treat it as this :-) Please feel free to reopen if anything needs further discussion. -- You are receiving this mail because: You are on the CC list for the bug.