After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 753394 - "Select Custom Icon" should propose an icon view
"Select Custom Icon" should propose an icon view
Status: RESOLVED OBSOLETE
Product: nautilus
Classification: Core
Component: File Properties Dialog
3.24.x
Other Linux
: Normal enhancement
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 611233 (view as bug list)
Depends on: 141154 611233
Blocks:
 
 
Reported: 2015-08-08 15:20 UTC by Jehan
Modified: 2021-06-18 15:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Select custom icon dialog (115.58 KB, image/png)
2015-08-08 15:20 UTC, Jehan
  Details
properties-window: make icon chooser dialog modal (1.01 KB, patch)
2016-05-09 16:29 UTC, Ernestas Kulik
none Details | Review
properties-window: make icon chooser dialog modal (1.23 KB, patch)
2016-05-11 09:24 UTC, Ernestas Kulik
committed Details | Review

Description Jehan 2015-08-08 15:20:40 UTC
Created attachment 308951 [details]
Select custom icon dialog

When selecting icons, the dialog shows a List view, which is not adapted. Every image is extra small, so you have to click on these to so a small preview.

Since the whole point of selecting an icon (likely searching it in a directory which would be full of pictures or icons) is to choose an image, wouldn't it make more sense to have this file selection dialog appear in an Icon view format, where images are at least previewable without having to click them?

Nautilus 3.16.2
Comment 1 rusins 2016-05-07 15:52:00 UTC
I'd like to add, that the "select custom icon" window is unresponsive to mouse clicks or any keyboard keys, except the escape key, which is the only way to close the dialogue, in the current version (3.20.1) (Yes, I know that this should be a new bug, but why not kill 2 birds with one stone?)

In addition to what Jehan said, I'd like to mention that other apps (such as Gnome Pie https://simmesimme.github.io/gnome-pie.html) have much better icon choosers, and they all seem to use the same underlying code/template.
Comment 2 Ernestas Kulik 2016-05-09 16:29:39 UTC
Created attachment 327531 [details] [review]
properties-window: make icon chooser dialog modal

The custom icon selection GtkFileChooserDialog is unresponsive when
opened. This can be fixed by making the dialog modal.
Comment 3 Carlos Soriano 2016-05-10 07:30:57 UTC
Review of attachment 327531 [details] [review]:

The fact of getting the focus solely on the new window is indeed a side effect of making it modal, but not the fix for this if we want to keep the previous behaviour of not making it modal (but we want to make it modal). So could you write something in the commit message along the lines of:
The custom icon selection GtkFileChooserDialog doesn't respond to keyboard keys when opened. The fix would be to grab the focus on the new window.
However, lately we are making all the dialogs modal following the design guidelines, which implicitly grabs the focus. So we can just make the dialog modal and as a side effect fix this.

Feel free to commit with the commit message fixed.

Thanks for your work Ernestas!
Comment 4 Ernestas Kulik 2016-05-10 18:35:48 UTC
Huh, I thought gtk_window_present() grabbed focus.

Sure, I’ll rewrite the message.
Comment 5 Carlos Soriano 2016-05-11 08:29:49 UTC
(In reply to Ernestas Kulik from comment #4)
> Huh, I thought gtk_window_present() grabbed focus.
> 
> Sure, I’ll rewrite the message.

wait...of course, it should. But then maybe we steal it somewhere else?
If not I understood wrong the bug...
The thing is that, once we present the window and the focus is in the new window, the widget should behave as expected.
Comment 6 Ernestas Kulik 2016-05-11 09:24:34 UTC
Created attachment 327632 [details] [review]
properties-window: make icon chooser dialog modal

The custom icon selection GtkFileChooserDialog is unresponsive when
opened. This is caused by the properties dialog window being modal,
since it is up to the window manager how to process input in such
situations.

One of the ways to ensure that the child dialog receives and processes
user input is by making it modal. This commit makes it such.
Comment 7 Carlos Soriano 2016-05-11 09:26:11 UTC
Review of attachment 327632 [details] [review]:

Sounds good, thanks!
Comment 8 Ernestas Kulik 2016-05-11 09:31:30 UTC
Comment on attachment 327632 [details] [review]
properties-window: make icon chooser dialog modal

Attachment 327632 [details] pushed as 6df5972 - properties-window: make icon chooser dialog modal
Comment 9 Stephen 2016-08-08 23:36:27 UTC
Is there any workaround for the unresponsive icon chooser in the meantime? (E.g. editing a config. file somewhere to set the icon?)

Can someone retitle this bug to relate to the unresponsiveness issue, and split the original bug off to a new one? Difficult to find that this has been fixed otherwise.
Comment 10 Stephen 2016-08-09 00:09:17 UTC
Sorry, answered my own question about a workaround. For others' reference:

gvfs-set-attribute /path/to/file metadata::custom-icon-name some-icon-name
Comment 11 rusins 2016-08-09 12:43:08 UTC
I'd like to mention that the the "select custom icon" window has been fixed and is now responsive to mouse clicks again in the latest bersion of nautilus (3.20.2). Jehan's issue is still topical however.

Just though I'd mention this, because I found it funny how all Stephen had to do was update his installation of nautilus.
Comment 12 Stephen 2016-08-09 12:52:21 UTC
No it isn't fixed in 3.20.2.
Comment 13 rusins 2016-08-09 12:54:45 UTC
It's fixed for me, so I guess either the bug is caused from something else of which I have a newer version than you, or the bug only affects some systems, perhaps periodically. ¯\_(ツ)_/¯
Comment 14 Carlos Soriano 2016-08-09 12:59:02 UTC
This bug is not about the unresponsive icon chooser, fixed in comment #8.
We should had make the reporter file a new bug report about this or file ourselves and attached the patch there.
So sorry for the confusion.
Comment 15 Ernestas Kulik 2016-08-09 13:09:27 UTC
(In reply to Stephen from comment #9)
> Is there any workaround for the unresponsive icon chooser in the meantime?
> (E.g. editing a config. file somewhere to set the icon?)
> 
> Can someone retitle this bug to relate to the unresponsiveness issue, and
> split the original bug off to a new one? Difficult to find that this has
> been fixed otherwise.

Bug 767229 is the designated tracking bug.

(In reply to Stephen from comment #12)
> No it isn't fixed in 3.20.2.

Reply to bug 767229 or file a new report. Make sure you have 3.20.2 installed first.
Comment 16 António Fernandes 2017-08-23 08:27:04 UTC
Comment 0 suggests file chooser to use an icon grid. Unfortunately, file chooser does not support such icon grid view: bug 141154. Adding dependency.

Comment 1 suggests a special purpose icon chooser, which is the subject of bug 611233.
Comment 17 António Fernandes 2018-06-14 23:07:42 UTC
*** Bug 611233 has been marked as a duplicate of this bug. ***
Comment 18 André Klapper 2021-06-18 15:53:08 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version of Files (nautilus), then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new ticket at
  https://gitlab.gnome.org/GNOME/nautilus/-/issues/

Thank you for your understanding and your help.