Merge lp://staging/~malaperle/overlay-scrollbar/default into lp://staging/overlay-scrollbar

Proposed by Marc-Andre Laperle
Status: Needs review
Proposed branch: lp://staging/~malaperle/overlay-scrollbar/default
Merge into: lp://staging/overlay-scrollbar
Diff against target: 64 lines (+23/-2)
1 file modified
os/os-scrollbar.c (+23/-2)
To merge this branch: bzr merge lp://staging/~malaperle/overlay-scrollbar/default
Reviewer Review Type Date Requested Status
Adolfo Jayme Barrientos (community) Abstain
Ayatana Scrollbar Team Pending
Review via email: mp+175867@code.staging.launchpad.net

This proposal supersedes a proposal from 2013-07-18.

Description of the change

Fix scrollbar not visible when it's the only child in a container.

To post a comment you must log in.
Revision history for this message
Adolfo Jayme Barrientos (fitojb) wrote : Posted in a previous version of this proposal

Please rebase your branch and resolve conflicts :-)

review: Needs Fixing
Revision history for this message
Adolfo Jayme Barrientos (fitojb) wrote :

Thank you very much for sending this fix, and for keeping up with trunk! Looks sensible to me, but someone from the team must review this :-)

review: Abstain
Revision history for this message
Marc-Andre Laperle (malaperle) wrote :

Is there any way I can improve my fix or help with the review process? Should I add more comments to explain the issue?

Thanks!

Unmerged revisions

379. By Marc-Andre Laperle

Scrollbar not visible when it's the only child in a container (LP: #1188831)

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