Merge lp:~brandontschaefer/unity/lp.1175666-fix into lp:unity

Proposed by Brandon Schaefer
Status: Merged
Approved by: Stephen M. Webb
Approved revision: no longer in the source branch.
Merged at revision: 3404
Proposed branch: lp:~brandontschaefer/unity/lp.1175666-fix
Merge into: lp:unity
Diff against target: 10 lines (+1/-0)
1 file modified
dash/ScopeView.cpp (+1/-0)
To merge this branch: bzr merge lp:~brandontschaefer/unity/lp.1175666-fix
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Stephen M. Webb (community) Approve
Review via email: mp+172921@code.launchpad.net

Commit message

When the ScopeView changes visibility make sure we update the filter scroll view as well. This way we don't get random overlay scroll bars appearing when it shouldn't!

Description of the change

When the ScopeView changes visibility make sure we update the filter scroll view as well. This way we don't get random overlay scroll bars appearing when it shouldn't!

To post a comment you must log in.
Revision history for this message
Stephen M. Webb (bregma) wrote :

Yep.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'dash/ScopeView.cpp'
2--- dash/ScopeView.cpp 2013-07-01 14:40:21 +0000
3+++ dash/ScopeView.cpp 2013-07-03 22:06:29 +0000
4@@ -225,6 +225,7 @@
5
6 OnVisibleChanged.connect([&] (nux::Area* area, bool visible) {
7 scroll_view_->SetVisible(visible);
8+ fscroll_view_->SetVisible(visible);
9 });
10 }
11