(In reply to comment #0)

> * The battery menu item should be insensitive and not launch the power
> statistics dialog

From where should it be opened then? Should it end up being an app?

We can probably add some of that information to the system info panel.  The rest is just geek stuff and someone can launch it from the command line or something.

We do have a symbolic icon now, but it still launches the statistics

Created attachment 182230 [details] [review]
Do not launch power statistics app from battery items

Comment on attachment 182230 [details] [review]
Do not launch power statistics app from battery items

>+            this._batteryItem.actor.reactive = false;
>+            this._batteryItem.actor.can_focus = false;

This will never change now, so just set it when this._batteryItem is constructed in _init:

        this._batteryItem = new PopupMenu.PopupMenuItem('', { reactive: false });


>+                item.actor.reactive = false;
>+                item.actor.can_focus = false;

Likewise, change the super call in DeviceItem._init to:

        PopupMenu.PopupBaseMenuItem.prototype._init.call(this, { reactive: false });

Created attachment 182311 [details] [review]
Do not launch power statistics app from battery items

[Removing GNOME3.0 target as decided in release-team meeting on March 03, 2011. "nice-to-have" categorisation for GNOME3.0]