Indicators should listen to show/hide from GtkWidget

Bug #590920 reported by Neil J. Patel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Unity
Fix Released
Medium
Jay Taoko
unity (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

We currently just sync the properties without checking whether the widgets of the IndicatorEntry are actually showing or not (and changes to that).

Related branches

Revision history for this message
Neil J. Patel (njpatel) wrote :

As with the other properties, we need to link the "visible" property of the GtkImage and GtkLabel from Indicator.ObjectEntry to the "visible" property on the CtkText and CtkImage.

There isn't a bunch of indicators that would show/hide their label/image, so it'll probably just be easier to make a test entry that does that. We may need some fixes inside CtkBox to account for actors that have visible=false (i.e. skip them :).

Changed in unity:
assignee: Neil J. Patel (njpatel) → Jay Taoko (jaytaoko)
status: Confirmed → Triaged
Neil J. Patel (njpatel)
Changed in unity:
milestone: 2010-06-17 → 2010-06-24
Jay Taoko (jaytaoko)
Changed in unity:
status: Triaged → In Progress
Neil J. Patel (njpatel)
Changed in unity:
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity - 0.2.12-0ubuntu1

---------------
unity (0.2.12-0ubuntu1) maverick; urgency=low

  * New upstream release:
    - reordered applications launchers don't save their location (LP: #592087)
    - cannot close panel menus unless clicking on particular zones of the
      display (LP: #595880)
    - Add support for switching workspaces (LP: #594157)
    - Indicators should listen to show/hide from GtkWidget (LP: #590920)
    - Race in expose manager event processing (LP: #588299)
    - Files Place (View) - add support for file grouping (LP: #597256)
    - Apps Place (View) - add basic support (LP: #597257)
  * debian/patches/01_draw_background_with_nautilus_off.patch:
    - integrated upstream
  * debian/control:
    - add libclutk-dev as a dep to libunity-dev
    - bump clutk, dee and bamf dep to latest
 -- Didier Roche <email address hidden> Thu, 24 Jun 2010 21:12:40 +0200

Changed in unity (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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