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 373890 - font size and update interval are too power user
font size and update interval are too power user
Status: RESOLVED FIXED
Product: netspeed
Classification: Deprecated
Component: general
unspecified
Other All
: Normal minor
: ---
Assigned To: Netspeed maintainers
Netspeed maintainers
Depends on:
Blocks:
 
 
Reported: 2006-11-11 15:37 UTC by Pedro Villavicencio
Modified: 2011-01-07 11:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (10.70 KB, patch)
2006-11-14 13:45 UTC, Pedro Villavicencio
none Details | Review

Description Pedro Villavicencio 2006-11-11 15:37:36 UTC
The font size and update interval options located in the netspeed preferences are too power users for being in the GUI they may be accessed only by the gconf interface.
Comment 1 Jörgen Scheibengruber 2006-11-11 17:12:44 UTC
hmm, netspeed is a poweruser tool. maybe the settings are not too usefull and could be removed and set to sane defaults. however i kind of disagree with the gconf interface, since it is totally horrible for applets (have to find the right UID in /apps/panel/applets/ and hen kill the applet because the api does signal changes? - not so nice imho)
Comment 2 Pedro Villavicencio 2006-11-12 13:53:26 UTC
well yeah, but my concern was about the "too much", i mean who really uses that options, i don't really know anyone who likes to have the desktop with font 9 and the applets with font 4. About the gconf thing, yeah it's really awful for applets ... ok i'm gonna work on having these two options with defaults instead providing a gconf interface, thanks again.
  
Comment 3 Pedro Villavicencio 2006-11-14 13:45:57 UTC
Created attachment 76563 [details] [review]
patch

remove the label font size and update interval options from the settings dialog. may you please take a look to it and tell me if is ok to commit, thanks.
Comment 4 Jörgen Scheibengruber 2006-11-14 19:04:01 UTC
Hmm, no wait a second. This is not ok to commit. It might surprise you, but the font setting is actually there for a reason. The default panel font is to large for netspeed IMHO... This does not mean that there has to be a font setting, we can solve this somehow differently, but I am not going with the default panel font.
Comment 5 Jörgen Scheibengruber 2006-11-25 12:30:35 UTC
I have been testing your patch now for a while. Maybe it's a good idea anyways... Go ahead and commit if you like, if people complain we can revert it anyways ;-)
Comment 6 Pedro Villavicencio 2006-11-26 23:51:43 UTC
Yeah no problem, I think the users are not going to complaint (i hope hehe). Ok already committed thanks again for your time.

Regards.
Comment 7 Christof Krüger 2011-01-07 11:53:49 UTC
Sorry for digging up such an old bug.
I wonder whether it would be possible to export refresh_rate as an gconf option instead of using the currently hard coded value of 1000 ms. I'm using the netspeed applet for ages now and am really used to it. Now I'm trying to optimize the battery running time on my new netbook and realized that the netspeed applet eats 1-2 percent of cpu power on average. I'd like to change the refresh_rate to something along 5000 ms to reduce the load.