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 172667 - Several HIG regressions
Several HIG regressions
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: cpufreq
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-04-05 01:08 UTC by Dennis Cranston
Modified: 2005-04-06 00:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A screenshot (19.23 KB, image/png)
2005-04-05 18:14 UTC, Carlos Garcia Campos
Details

Description Dennis Cranston 2005-04-05 01:08:02 UTC
The following commit introduced several HIG regressions in the preference
dialog.  Please see http://developer.gnome.org/projects/gup/hig/.

2005-04-04  Carlos Garcia Campos <carlosgc@gnome.org>

	* cpufreq-applet.schemas.in: Add selector_mode key for configuring
	the frequency selector drop down menu

	* cpufreq-preferences.glade: Add frequency selector configuration

	* src/cpufreq-applet.[ch]:
	* src/cpufreq-monitor-protected.h:
	* src/cpufreq-monitor-sysfs.[ch]:
	* src/cpufreq-monitor.[ch]:
	* src/cpufreq-popup.c:
	* src/cpufreq-prefs.c: Add support to change the governor in the
	frequency selector drop down menu. Fixes #164581
Comment 1 Dennis Cranston 2005-04-05 04:04:19 UTC
By the way, the original HIG changes are found in bug #157913.
Comment 2 Carlos Garcia Campos 2005-04-05 12:07:45 UTC
Why is better to use labels with spaces instead of alignments?
Comment 3 Dennis Cranston 2005-04-05 16:09:16 UTC
The short answer is spacing and alignment.  

Notice that in your redesign there is not 12 pixels of white space around the
border of the dialog.  There should also be 12 pixels between the action area
and the vbox of the dialog.  Now, the controls do not line up on the right and
left side of the dialog.  Also, there should be 18 pixels between each
categories.  And, a colon ":" should be appended to labels preceeding a control.  

You should read this link: 
http://developer.gnome.org/projects/gup/hig/2.0/design-window.html#window-layout-spacing

You might want to read this link, too:
http://bugzilla.gnome.org/show_bug.cgi?id=163850

Thanks.
Comment 4 Carlos Garcia Campos 2005-04-05 18:14:40 UTC
Created attachment 39719 [details]
A screenshot

Ok, here is the current design (not committed yet), I think it's now a HIG
compliant dialog. is it ok now?

Thank you for your comments.
Comment 5 Dennis Cranston 2005-04-05 19:17:40 UTC
This looks much better.  Unfortunately, glade frames add an extra pixel around
their border; and, the frame labels are displayed in a washed out bold typeface.
 But, these issues are not of cpufreq's concern.  Like I said, it looks much
better.  If you apply your UI changes, you can close this bug.  Thanks.
Comment 6 Dennis Cranston 2005-04-06 00:49:26 UTC
Thanks Carlos!