libindicator

Merge lp://qastaging/~larsu/libindicator/entry-visibility-fix into lp://qastaging/libindicator/0.5

entry-visibility-fix
Merge into trunk.0.5

Proposed by Lars Karlitski on 2012-02-12

Status:	Merged
Merge reported by:	Lars Karlitski
Merged at revision:	not available
Proposed branch:	lp://qastaging/~larsu/libindicator/entry-visibility-fix
Merge into:	lp://qastaging/libindicator/0.5
Diff against target:	44 lines (+8/-17) 1 file modified libindicator/indicator-object.c (+8/-17)
To merge this branch:	bzr merge lp://qastaging/~larsu/libindicator/entry-visibility-fix
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Charles Kerr (community)	2012-02-12	Approve on 2012-02-13
Indicator Applet Developers	2012-02-12	Pending
Review via email: mp+92666@code.qastaging.launchpad.net

Description of the change

See commit message of r447.

Noticed this while developing indicator-printers. It can be worked around by not calling indicator_object_set_visible (TRUE) when the indicator is already visible. But I think the correct place for the check is in libindicator.

lp://qastaging/~larsu/libindicator/entry-visibility-fix updated on 2012-02-13

448. By Lars Karlitski on 2012-02-13: indicator_object_set_visible: no need to check for ENTRY_INIT
449. By Lars Karlitski on 2012-02-13: indicator_object_dispose: use _set_visible instead of reimplementing it
450. By Lars Karlitski on 2012-02-13: indicator_object_dispose: remove unneeded list of entries