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 684029 - universal access: missing colon in zoom options ui
universal access: missing colon in zoom options ui
Status: RESOLVED OBSOLETE
Product: gnome-control-center
Classification: Core
Component: Universal Access
unspecified
Other All
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-14 15:04 UTC by Joseph Scheuhammer
Modified: 2021-06-09 16:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Adds a colon to the gray scale slider label. (1.47 KB, patch)
2012-09-14 15:09 UTC, Joseph Scheuhammer
none Details | Review
Adds colon, ':', to 'Color' label on gray scale slider (1.42 KB, patch)
2012-10-15 04:12 UTC, Joseph Scheuhammer
needs-work Details | Review

Description Joseph Scheuhammer 2012-09-14 15:04:39 UTC
The grayscale slider in the "Color effects" tab is labelled "Color".  Other sliders in the dialog append a colon after their label, e.g., "Brightness:".  For consistency, the label on the grayscale slider should be "Color:".
Comment 1 Joseph Scheuhammer 2012-09-14 15:09:57 UTC
Created attachment 224337 [details] [review]
Adds a colon to the gray scale slider label.
Comment 2 Joseph Scheuhammer 2012-10-15 04:12:00 UTC
Created attachment 226438 [details] [review]
Adds colon, ':', to 'Color' label on gray scale slider

There have been modifications to the zoom-options.ui file since the previous patch was uploaded.  This patch takes those changes into account.
Comment 3 Bastien Nocera 2012-10-15 07:10:12 UTC
Review of attachment 226438 [details] [review]:

This isn't right. There's a number of places where you're using colons in the dialogue and you shouldn't.

Headings shouldn't have colons for example. I'll let Allan review this though.
Comment 4 Allan Day 2012-11-01 18:04:56 UTC
All the other labels have colons there, so this seems good to me.

Maybe we need to review the use of colons more generally, but that doesn't need to block this.
Comment 5 Bastien Nocera 2012-11-02 14:13:12 UTC
(In reply to comment #4)
> All the other labels have colons there, so this seems good to me.
> 
> Maybe we need to review the use of colons more generally, but that doesn't need
> to block this.

That would make it look very inconsistent, IMO, and I'd like this fixed properly.
Comment 6 Alexandre Franke 2017-07-16 16:15:24 UTC
In 3.24 Brightness doesn’t have a colon either, but Contrast does. Labels for sliders in the crosshairs tab of the zoom settings have colons. It seems labels for sliders everywhere else don't have colons. I failed to find anything relevant in the HIG. Allan, can you advise?
Comment 7 André Klapper 2021-06-09 16:31:34 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, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new bug report at
  https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/

Thank you for your understanding and your help.