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 735649 - Add tooltips to buttons on new dialogs
Add tooltips to buttons on new dialogs
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
git master
Other All
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-08-29 01:16 UTC by Michael Catanzaro
Modified: 2014-08-29 13:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
cookies-dialog: add tooltip to remove cookie button (1.06 KB, patch)
2014-08-29 01:16 UTC, Michael Catanzaro
reviewed Details | Review
history-dialog: add tooltips to buttons (1.78 KB, patch)
2014-08-29 01:16 UTC, Michael Catanzaro
reviewed Details | Review
passwords-dialog: add tooltips to buttons (1.78 KB, patch)
2014-08-29 01:16 UTC, Michael Catanzaro
none Details | Review
cookies-dialog: add tooltip to remove cookie button (1.06 KB, patch)
2014-08-29 12:15 UTC, Michael Catanzaro
committed Details | Review
history-dialog: add tooltips to buttons (1.78 KB, patch)
2014-08-29 12:15 UTC, Michael Catanzaro
committed Details | Review
passwords-dialog: add tooltips to buttons (1.78 KB, patch)
2014-08-29 12:15 UTC, Michael Catanzaro
committed Details | Review

Description Michael Catanzaro 2014-08-29 01:16:49 UTC
These icons are still unclear and probably should be replaced with text buttons, but this probably requires designer attention, so for now let's just add tooltips to these buttons instead. Would be nice to get these in before string freeze.
Comment 1 Michael Catanzaro 2014-08-29 01:16:50 UTC
Created attachment 284777 [details] [review]
cookies-dialog: add tooltip to remove cookie button
Comment 2 Michael Catanzaro 2014-08-29 01:16:54 UTC
Created attachment 284778 [details] [review]
history-dialog: add tooltips to buttons
Comment 3 Michael Catanzaro 2014-08-29 01:16:57 UTC
Created attachment 284779 [details] [review]
passwords-dialog: add tooltips to buttons
Comment 4 Carlos Garcia Campos 2014-08-29 06:26:18 UTC
Review of attachment 284777 [details] [review]:

::: src/resources/cookies-dialog.ui
@@ +137,3 @@
                     <property name="use_underline">True</property>
                     <property name="icon_name">list-remove-symbolic</property>
+                    <property name="tooltip_text" translatable="yes">Delete the selected cookie</property>

Note that multiple selection is allowed, so maybe this should be plural, the selected cookies.
Comment 5 Carlos Garcia Campos 2014-08-29 06:27:33 UTC
Review of attachment 284778 [details] [review]:

Same here, it seems the history dialog also allows multiple selections
Comment 6 Carlos Garcia Campos 2014-08-29 06:28:54 UTC
Review of attachment 284779 [details] [review]:

Not sure about this one, I don't have passwords, but I guess it also allows multiple selections.
Comment 7 Michael Catanzaro 2014-08-29 12:15:41 UTC
Created attachment 284810 [details] [review]
cookies-dialog: add tooltip to remove cookie button
Comment 8 Michael Catanzaro 2014-08-29 12:15:45 UTC
Created attachment 284811 [details] [review]
history-dialog: add tooltips to buttons
Comment 9 Michael Catanzaro 2014-08-29 12:15:48 UTC
Created attachment 284812 [details] [review]
passwords-dialog: add tooltips to buttons
Comment 10 Carlos Garcia Campos 2014-08-29 12:35:34 UTC
Comment on attachment 284810 [details] [review]
cookies-dialog: add tooltip to remove cookie button

Thanks
Comment 11 Michael Catanzaro 2014-08-29 13:05:45 UTC
Attachment 284810 [details] pushed as 2d1db96 - cookies-dialog: add tooltip to remove cookie button
Attachment 284811 [details] pushed as ca8ef0d - history-dialog: add tooltips to buttons
Attachment 284812 [details] pushed as 6430219 - passwords-dialog: add tooltips to buttons