Merge lp://qastaging/~unity-team/unity/unity.fix-676928 into lp://qastaging/unity

Proposed by Mirco Müller
Status: Merged
Approved by: Mirco Müller
Approved revision: no longer in the source branch.
Merged at revision: 950
Proposed branch: lp://qastaging/~unity-team/unity/unity.fix-676928
Merge into: lp://qastaging/unity
Diff against target: 349 lines (+286/-4)
6 files modified
src/QuicklistMenuItemCheckmark.cpp (+1/-1)
src/QuicklistMenuItemLabel.cpp (+1/-1)
src/QuicklistMenuItemRadio.cpp (+1/-1)
tests/CMakeLists.txt (+22/-0)
tests/ui/TestQuicklist.cpp (+1/-1)
tests/ui/TestQuicklistVisuals.cpp (+260/-0)
To merge this branch: bzr merge lp://qastaging/~unity-team/unity/unity.fix-676928
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Kalle Valo (community) Approve
Gord Allott Pending
Review via email: mp+53025@code.qastaging.launchpad.net

Description of the change

Fixed rendering of disabled Quicklist-items (Label, Checkmark, Radio). Added new visual test-case to ui-tests. Fixes LP: #676928

To post a comment you must log in.
Revision history for this message
Mirco Müller (macslow) wrote :
Revision history for this message
Kalle Valo (kvalo) wrote :

Looks good to me.

review: Approve
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

approved! Some tests, yeah \o/

review: Approve

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.