Move system monitor entry to the top of the list

Bug #799946 reported by Christoph Buchner
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
System Load Indicator
Fix Released
Low
Michael Hofmann

Bug Description

In my opinion, it would make more sense from a UI standpoint, if the system monitor entry would be on top of the list.

Reason:
It would be easier to train muscle memory for opening the system monitor (click - one line down - click), as opposed to now, where one has to move past all the system readings (which are read-only to boot), and find and click the correct entry.

Furthermore, starting the system monitor is _by far_ my most-often used function of that menu.
Preferences are typically only set once, and most of the time the graphs deliver sufficient information that the numerical stats, while a really nice supplemental information(!), don't have to be checked upon by calling the menu.

Do you agree that this would be an improvement?

Revision history for this message
Michael Hofmann (mh21) wrote :

Hi Christoph,

convinced :-).

If you feel adventurous, you can try to provide a patch. What's needed I think is a change to the menu item order in preferences.ui, and some changes to the timeout callback multiloadindicator.vala.

Changed in indicator-multiload:
status: New → Confirmed
milestone: none → 0.3
importance: Undecided → Low
Revision history for this message
Christoph Buchner (bilderbuchi) wrote :

OK, I just took a look. It's a bit difficult, since I've never even touched vala or gtk before. :-P
I think it's not a totally trivial change in preferences.ui.in, because the system data are not GtkMenuItems (the menu begins with the separator above the sysmon), so I can't simply move the sysmon above those. In fact, I don't know how/where the system data gets added to the menu. I'll go dig some more...

Revision history for this message
Michael Hofmann (mh21) wrote :

This is now available in the beta versions at ppa:indicator-multiload/daily and will be in 0.3.

Thanks for reporting this!

Changed in indicator-multiload:
assignee: nobody → Michael Hofmann (mh21)
status: Confirmed → Fix Committed
Revision history for this message
Christoph Buchner (bilderbuchi) wrote :

thanks! sorry I couldn't help more... :-/

Revision history for this message
Michael Hofmann (mh21) wrote :

Oh no problem :-)

Michael Hofmann (mh21)
tags: removed: wishlist
Revision history for this message
Christoph Buchner (bilderbuchi) wrote :

The stable PPA already contains 0.3, so shouldn't this be fix-released?

Changed in indicator-multiload:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.