Merge lp://qastaging/~charlesk/libindicator/add-visibility into lp://qastaging/libindicator/0.4

Proposed by Charles Kerr
Status: Superseded
Proposed branch: lp://qastaging/~charlesk/libindicator/add-visibility
Merge into: lp://qastaging/libindicator/0.4
Diff against target: 522 lines (+383/-15)
3 files modified
libindicator/indicator-desktop-shortcuts.c (+1/-1)
libindicator/indicator-object.c (+375/-14)
libindicator/indicator-object.h (+7/-0)
To merge this branch: bzr merge lp://qastaging/~charlesk/libindicator/add-visibility
Reviewer Review Type Date Requested Status
Ted Gould Pending
Review via email: mp+88459@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2012-01-13.

To post a comment you must log in.
438. By Charles Kerr

fix bugs in IndicatorObject's visibility support.

439. By Charles Kerr

another iteration of the indicator-object visibility support patch, incorporating ideas from discussion with ted

 - some functions were public when they should have been private
 - the hide/show handler is now a virtual function & is documented in indicator-object.h
 - added unit tests
 - the GSettings monitor has been removed

440. By Charles Kerr

"bzr merge lp:indicator" + conflict resolution in tests suite

Unmerged revisions

440. By Charles Kerr

"bzr merge lp:indicator" + conflict resolution in tests suite

439. By Charles Kerr

another iteration of the indicator-object visibility support patch, incorporating ideas from discussion with ted

 - some functions were public when they should have been private
 - the hide/show handler is now a virtual function & is documented in indicator-object.h
 - added unit tests
 - the GSettings monitor has been removed

438. By Charles Kerr

fix bugs in IndicatorObject's visibility support.

437. By Charles Kerr

Better implementation.

This version handles the bootstrapping conditions better, and also refs+detaches the GtkMenu for safekeeping between removed->added events s.t. it can keep being reused.

436. By Charles Kerr

initial revision for GSettings/GtkMenu visibility support

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches