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 648048 - the bluetooth switch in the top right should tell you when bluetooth is turned off
the bluetooth switch in the top right should tell you when bluetooth is turne...
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-04-17 21:06 UTC by Josh
Modified: 2011-06-27 15:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
bluetooth menu behavior (75.02 KB, image/png)
2011-04-17 21:06 UTC, Josh
  Details
PopupMenu: move PopupStatusSwitchMenuItem out of network.js (7.00 KB, patch)
2011-04-21 15:04 UTC, Giovanni Campagna
reviewed Details | Review
BluetoothStatus: show "hardware disabled" when disabled by rfkill (3.87 KB, patch)
2011-04-21 15:04 UTC, Giovanni Campagna
reviewed Details | Review
PopupMenu: enhance PopupSwitchMenuItem to show status labels (6.78 KB, patch)
2011-05-09 17:31 UTC, Giovanni Campagna
committed Details | Review
BluetoothStatus: show "hardware disabled" when disabled by rfkill (3.74 KB, patch)
2011-05-09 17:32 UTC, Giovanni Campagna
none Details | Review
BluetoothStatus: show "hardware disabled" when disabled by rfkill (3.16 KB, patch)
2011-05-09 20:35 UTC, Giovanni Campagna
committed Details | Review
bluetooth discrepancy between system and applet (27.20 KB, image/png)
2011-06-06 14:16 UTC, bidossessi.sodonon
  Details

Description Josh 2011-04-17 21:06:51 UTC
Created attachment 186165 [details]
bluetooth menu behavior

If I go into bluetooth settings, it tells me bluetooth has been disabled by a switch on my computer (indeed it has).  However when I click on the icon in the upper right corner of the screen it has an on/off switch that cannot be toggled.  In other words when one (see attachment) knows about two it should behave like three
Comment 1 Giovanni Campagna 2011-04-21 15:04:38 UTC
Created attachment 186432 [details] [review]
PopupMenu: move PopupStatusSwitchMenuItem out of network.js

Moves and converts NMDeviceTitleMenuItem from network.js into
PopupStatusSwitchMenuItem, that is a menu item that can show both
a switch and a greyed-out status label. This will be soon used by
the Bluetooth menu.
Comment 2 Giovanni Campagna 2011-04-21 15:04:50 UTC
Created attachment 186433 [details] [review]
BluetoothStatus: show "hardware disabled" when disabled by rfkill

Use the new PopupStatusSwitchMenuItem and set it appropriately when
bluetooth cannot be enabled. Use it also for showing "connecting..."
when activating device items.
Comment 3 Dan Winship 2011-04-26 15:17:10 UTC
Comment on attachment 186432 [details] [review]
PopupMenu: move PopupStatusSwitchMenuItem out of network.js

>+PopupStatusSwitchMenuItem.prototype = {

would it make more sense to just merge this functionality into PopupSwitchMenuItem?

>+        PopupMenu.PopupStatusSwitchMenuItem.prototype._init.call(this, label, 'popup-subtitle-menu-item', params);

should it default to { style_class: 'popup-subtitle-menu-item' }, since you set that everywhere you use it?
Comment 4 Dan Winship 2011-04-26 15:18:08 UTC
Comment on attachment 186433 [details] [review]
BluetoothStatus: show "hardware disabled" when disabled by rfkill

looks right
Comment 5 Giovanni Campagna 2011-05-09 17:31:14 UTC
Created attachment 187519 [details] [review]
PopupMenu: enhance PopupSwitchMenuItem to show status labels

Moves and converts NMDeviceTitleMenuItem from network.js into
PopupSwitchMenuItem, so that it can show both a switch and a
greyed-out status label. This will be soon used by the Bluetooth menu.
Comment 6 Giovanni Campagna 2011-05-09 17:32:20 UTC
Created attachment 187520 [details] [review]
BluetoothStatus: show "hardware disabled" when disabled by rfkill

Use the new PopupSwitchMenuItem functionality when bluetooth cannot
be enabled. Use it also for showing "connecting..." when activating
device items.
Comment 7 Giovanni Campagna 2011-05-09 20:35:54 UTC
Created attachment 187529 [details] [review]
BluetoothStatus: show "hardware disabled" when disabled by rfkill

Use the new PopupSwitchMenuItem functionality when bluetooth cannot
be enabled. Use it also for showing "connecting..." when activating
device items.
Comment 8 bidossessi.sodonon 2011-06-06 14:16:20 UTC
Created attachment 189329 [details]
bluetooth discrepancy between system and applet
Comment 9 bidossessi.sodonon 2011-06-06 14:17:01 UTC
Comment on attachment 189329 [details]
bluetooth discrepancy between system and applet

I believe this is related to the current issue.
Comment 10 William Jon McCann 2011-06-07 20:05:45 UTC
The design for the network menu is to show "disabled by hardware switch" when rfkilled.
Comment 11 Giovanni Campagna 2011-06-08 12:19:14 UTC
(In reply to comment #10)
> The design for the network menu is to show "disabled by hardware switch" when
> rfkilled.

Which is too long - that's why I changed the description.
Comment 12 Giovanni Campagna 2011-06-27 15:22:10 UTC
Attachment 187519 [details] pushed as c867081 - PopupMenu: enhance PopupSwitchMenuItem to show status labels
Attachment 187529 [details] pushed as 9412ac2 - BluetoothStatus: show "hardware disabled" when disabled by rfkill