Ubuntu
ubuntu-system-settings package

Switch always slides off->on when entering screen

Bug #1287293 reported by Matthew Paul Thomas
28
This bug affects 5 people
Affects Status Importance Assigned to Milestone
ubuntu-system-settings (Ubuntu)
Triaged
High
Unassigned

Bug Description

Ubuntu Trusty r188

1. Turn on Wi-Fi and Bluetooth, if they aren't on already.
2. Go to System Settings > "Wi-Fi" or "Bluetooth".

What happens: The main switch slides from off to on, implying that it was off a moment ago.

What should happen: The switch is already on when the screen becomes visible.

Tags: wifi

Related branches

lp:~cyphermox/ubuntu-system-settings/bluetooth-redesign
PS Jenkins bot: Approve (continuous-integration)
Manuel de la Peña (community): Approve
Sebastien Bacher (community): Needs Fixing
Brendan Donegan: Pending requested
Diff: 2028 lines (+1319/-115)
17 files modified
CMakeLists.txt (+2/-0)
debian/changelog (+19/-0)
debian/control (+2/-0)
plugins/bluetooth/PageComponent.qml (+147/-30)
plugins/bluetooth/bluetooth.cpp (+54/-27)
plugins/bluetooth/bluetooth.h (+15/-5)
plugins/bluetooth/device.cpp (+180/-26)
plugins/bluetooth/device.h (+22/-5)
plugins/bluetooth/devicemodel.cpp (+133/-19)
plugins/bluetooth/devicemodel.h (+15/-3)
tests/plugins/CMakeLists.txt (+1/-0)
tests/plugins/bluetooth/CMakeLists.txt (+61/-0)
tests/plugins/bluetooth/fakebluez.cpp (+118/-0)
tests/plugins/bluetooth/fakebluez.h (+84/-0)
tests/plugins/bluetooth/tst_bluetooth.cpp (+172/-0)
tests/plugins/bluetooth/tst_device.cpp (+197/-0)
tests/plugins/bluetooth/tst_devicemodel.cpp (+97/-0)
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, that's a known issue but one that is not easy to fix/workaround with the toolkit :/

Changed in ubuntu-system-settings (Ubuntu):
importance: Undecided → High
status: New → Triaged
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

So this is a toolkit bug, then?

Revision history for this message
Lars Karlitski (larsu) wrote :

We can hack around this by using Loaders (a qml component which delays loading another component). We've done that in other places of the settings, but it's quite some effort and leads to ugly code.

I've filed bug #1276066 for the toolkit a while ago asking for a way to temporarily stop animations. Let's wait until that's in before fixing system settings.

Removing ubuntu-ui-toolkit as that issue is tracked in the other bug.

no longer affects: ubuntu-ui-toolkit (Ubuntu)
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

I think this is fixed "by accident" via the redesign I'm working on. The bluetooth switch watches the state of the adapter, and it seems like it's behaving properly in my testing. Something worth noting as I get the branch ready...

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

This is ok now in bluetooth but still occurs in wifi

tags: added: wifi
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Not sure how this is a toolkit issue. The BT panel works fine, the difference is it implements its own components rather than using the menu model from the indicator.

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

I just realized I reported a duplicate of this for Bluetooth last week. So it's not the case that the Bluetooth panel works fine.

Revision history for this message
Jonas G. Drange (jonas-drange) wrote :

So, given the discussion in bug 1276066, we are to hide the control elements until we know its value. This seems to be the issue only for slow components, primarily components using a QMenuModel backend.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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