linuxmint / blueberry

A Bluetooth configuration tool
GNU General Public License v3.0
182 stars 53 forks source link

Ui rework / hotplugging fixups #92

Closed itzexor closed 5 years ago

itzexor commented 5 years ago

Note: Hardware RF switch untested. I expect it works as usual though, it should show a page identical to bluetooth disabled in the screenshot below except with the additional "by hardware switch" text.

UI cleanup:

gnome-bluetooth BluetoothSettingsWidget override fix: Widget override code now loops through widgets to find them instead of using a hardcoded "path". It's still fragile. Adding version-specific code is probably the best approach if it breaks again.

Better support for hotpluggable devices:

Better rfkill error state presentation: Previously, when the user did not have permission to /dev/rfkill, the main status page was used to show an error state with a bluetooth disabled icon and the switch set to off and insensitive. This would likely be reached most often by a user attempting to disable bluetooth. This layout may be misleading because it could suggest that bluetooth is disabled when it's actually still enabled and discoverable, and that the application no longer worked because of rfkill permission issues.

The new layout places an rfkill-error-specific image which shows a dialog-error icon next to the toggle switch when an enable/disable operation fails. The error is explained by the image's tooltip which may have low discoverability. A new message was added that explains having a udev rule or being in the rfkill group can fix permissions issues.

This rfkill-specific error state no longer modifies the powered or present states for bluetooth adapter status, which allows the application to work as expected regardless of rfkill permissions.

rfkill-error Screenshot from 2019-10-31 17-37-17 Screenshot from 2019-10-31 17-37-07 Screenshot from 2019-10-31 17-34-51 Screenshot from 2019-10-31 17-34-46