Bug 291708 - bluedevil do not use the inactive-bluetooth systray icon if it is inactive
Summary: bluedevil do not use the inactive-bluetooth systray icon if it is inactive
Status: RESOLVED FIXED
Alias: None
Product: solid
Classification: Frameworks and Libraries
Component: bluetooth-systray (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Alex Fiestas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-16 22:09 UTC by FabiB
Modified: 2012-04-28 00:27 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description FabiB 2012-01-16 22:09:18 UTC
Version:           unspecified (using Devel) 
OS:                Linux

Hi. in /usr/share/kde4/apps/desktoptheme/default/icons/preferences.svgz there are two icons for the bluetooth 'indicator' (systraything). one is for active bluetooth and one for inactive. but if i rightclick on the systray icon and choose 'deactivate' it use still the active icon. 

Reproducible: Always

Steps to Reproduce:
rightclick on bluetooth systray and deactivate

Actual Results:  
icon will be the same as the active one

Expected Results:  
should be the inactive one

Version 1.2.2
Comment 1 Alex Fiestas 2012-04-27 23:07:08 UTC
Inactive icon is not for this purpose but for when the icon is hidden (because the  bluetooth is deactiavted at a hardware level or the daemon is not pressent).

But you have a point, we are not indicating that "Bluetooth is off" in any way. What I'm going to do is set an overlay icon instead which imho will solve the issue.
Comment 2 Alex Fiestas 2012-04-28 00:22:47 UTC
Actually the overlay is already there, but because of a bug it is not shown. Going to fix the bug now !
Comment 3 Alex Fiestas 2012-04-28 00:27:06 UTC
Git commit 78aeadfcc68b84acbd96a6b71515e9aecf1877da by Alex Fiestas.
Committed on 28/04/2012 at 02:22.
Pushed by afiestas into branch '1.1'.

Only empty the Overlay if there are powered devices.

M  +1    -1    src/monolithic/monolithic.cpp

http://commits.kde.org/bluedevil/78aeadfcc68b84acbd96a6b71515e9aecf1877da