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 766909 - Tooltip of the eject button is a bit confusing - or just missing
Tooltip of the eject button is a bit confusing - or just missing
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: GtkFileChooser
3.22.x
Other All
: Normal enhancement
: ---
Assigned To: gtk-bugs
Federico Mena Quintero
Depends on:
Blocks:
 
 
Reported: 2016-05-26 13:45 UTC by sworddragon2
Modified: 2017-08-23 20:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
SidebarRow: Add Unmount tooltip to eject_button (1.15 KB, patch)
2017-08-19 16:25 UTC, Daniel Boles
committed Details | Review

Description sworddragon2 2016-05-26 13:45:58 UTC
In the FileChooser on specific devices like sr0 the tooltip does show the mounted directory like /media/sr0 but the eject button at this entry has the same tooltip which could be a bit confusing. Maybe the tooltip of it should be prefixed with "Eject " or so to be more clear.
Comment 1 sworddragon2 2016-05-27 07:22:16 UTC
Edit: I'm noticing that under "Other Locations" the devices are using the tooltip "Unmount" for their buttons so maybe the same tooltip should be used too.
Comment 2 Daniel Boles 2017-08-15 00:19:06 UTC
gtk_places_view_row_set_is_network() does this

>       gtk_widget_set_tooltip_text (GTK_WIDGET (row->eject_button), is_network ? _("Disconnect") : _("Unmount"));

The problem I see is different but still bad: despite that, and the default Unmount being set in the .ui file to match the default property value - I never see any tooltip on the eject_button, and the Inspector says it's NULL.
Comment 3 Daniel Boles 2017-08-15 00:20:45 UTC
IOW, far as I can tell, your problem is caused not because the button has the same tooltip as the row - but because it gets _no_ tooltip, and therefore the one from the row overall (as normally seen when hovering its label) wins out.
Comment 4 Daniel Boles 2017-08-15 00:27:07 UTC
Needless to say, this occurs in Nautilus too.

Finally, I'm not certain if it's relevant, but it still seems worth noting explicitly that these strings - both in the .ui and .c files - are marked as translatable.
Comment 5 Daniel Boles 2017-08-19 16:25:15 UTC
Created attachment 357982 [details] [review]
SidebarRow: Add Unmount tooltip to eject_button

It was reported that the lack of a tooltip made its purpose unclear.
This can be solved by just copying PlacesViewRow’s eject_button tooltip.

--

I was looking in the wrong widget. PlacesViewRow is for the main area and does
have a tooltip, but SidebarRow does not, hence this bug.

The solution seems obvious to me: make SidebarRow have the same tooltip in the
equivalent place.

This of course does not conditionally show Disconnect for network locations as
PlacesViewRow does, because SidebarRow has no awareness or property for that
(and, honestly, I have no idea whether we can put such locations in the sidebar)
Comment 6 Daniel Boles 2017-08-19 16:28:22 UTC
(Personally, I would've preferred Eject, but I presume it makes more sense to copy terminology already used in equivalent situations elsewhere)