Code review comment for lp:~ken-vandine/ubuntu-system-settings/lp1542098

Revision history for this message
Ken VanDine (ken-vandine) wrote :

> Did you mean “it’s ever visible”? It's supposed to be visible when a device is
> pairing automatically, so removing it altogether isn’t right.
> <https://wiki.ubuntu.com/Bluetooth#Listing_devices>
>
> Anyway, we recorded this problem in the Bluetooth panel just because that was
> a panel we happened to go into while pairing a keyboard with the test tablet.
> If we’d gone into the Wi-Fi panel instead, we would have recorded the problem
> there. The dividers are wrong everywhere they appear, so amputating one
> callsite won’t fix the problem.
>
> Please remove the dividers in the toolkit instead.

AFAICT, that spinner isn't displayed because the device doesn't get put into the connected device model until it's already connected.

« Back to merge proposal