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 781164 - Try to follow HIG with ellipses
Try to follow HIG with ellipses
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Interface
unspecified
Other All
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-04-11 00:22 UTC by Jeremy Bicha
Modified: 2017-04-11 01:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Try to follow HIG with ellipses (4.56 KB, patch)
2017-04-11 00:22 UTC, Jeremy Bicha
committed Details | Review
Add destructive-action style to Remove Language button (1.07 KB, patch)
2017-04-11 00:25 UTC, Jeremy Bicha
rejected Details | Review

Description Jeremy Bicha 2017-04-11 00:22:47 UTC
Here's a patch to add ellipses where more action is needed and remove ellipses where the user might just want to view something.

The app menu's Import and Export Bookmarks probably could use ellipses too so I can submit a patch for that if you want. It's more rare to see ellipses in that menu so that's why I hesitated.
Comment 1 Jeremy Bicha 2017-04-11 00:22:51 UTC
Created attachment 349645 [details] [review]
Try to follow HIG with ellipses
Comment 2 Jeremy Bicha 2017-04-11 00:25:29 UTC
Created attachment 349646 [details] [review]
Add destructive-action style to Remove Language button

Since this is a permanent "destructive" action, style it accordingly.

If you don't like the bright red color here, you could re-do this dialog to use symbolic +, -, ^ and down icons at the bottom instead.

We so far don't use bright red coloring in that case (although maybe it's worth looking into whether we should?)
Comment 3 Michael Catanzaro 2017-04-11 01:19:40 UTC
Review of attachment 349645 [details] [review]:

Let's avoid ellipses in the app menu unless that becomes more popular with other apps first.
Comment 4 Michael Catanzaro 2017-04-11 01:23:23 UTC
Review of attachment 349646 [details] [review]:

Using this style here feels like overkill to me. Your suggestion about redoing the dialog to avoid the text buttons entirely seems good, but that's more work of course.
Comment 5 Jeremy Bicha 2017-04-11 01:29:19 UTC
Attachment 349645 [details] pushed as 27cc337 - Try to follow HIG with ellipses