diff options
| author | Lars Uebernickel <lars.uebernickel@canonical.com> | 2012-02-13 20:07:33 +0100 |
|---|---|---|
| committer | Lars Uebernickel <lars.uebernickel@canonical.com> | 2012-02-13 20:07:33 +0100 |
| commit | 2a6c388d5f5dc2d298322e46fec71839bf41ab58 (patch) | |
| tree | d37bb254ba29ca7b59c016b353e7b04ce4b17b62 /tools/80indicator-debugging | |
| parent | 233fdd70cdb48ae8818baf416399989841333fcf (diff) | |
| parent | b2ca0dbe7704098144b6ccd103dd760ad7dd3636 (diff) | |
| download | libayatana-indicator-2a6c388d5f5dc2d298322e46fec71839bf41ab58.tar.gz libayatana-indicator-2a6c388d5f5dc2d298322e46fec71839bf41ab58.tar.bz2 libayatana-indicator-2a6c388d5f5dc2d298322e46fec71839bf41ab58.zip | |
indicator_object_set_visible: only emit entry-{added,removed} when visibility actually changed
Emitting those signals is semantically incorrect. It also confused
indicator-loader and the panel, because they are not checking whether an entry
is already present in their entry-added handlers. This led to gtk warnings
("widget already has a parent") and strange effects, such as the same menu
being added multiple times into the panel.
This patch also makes indicator_object_dispose use indicator_object_set_visible
(it was already doing something similar).
Diffstat (limited to 'tools/80indicator-debugging')
0 files changed, 0 insertions, 0 deletions