diff options
author | Merov Linden <merov@lindenlab.com> | 2012-03-16 13:43:42 -0700 |
---|---|---|
committer | Merov Linden <merov@lindenlab.com> | 2012-03-16 13:43:42 -0700 |
commit | 356c29c888e6220915a0f050c393c7e91d922318 (patch) | |
tree | bfc9061cf3e66d232a9b7bc5a940afce02e6d7c1 /indra/llui/llscrollcontainer.cpp | |
parent | f4722686d4bd125ebbecac5da511390252a7084e (diff) | |
parent | 34c599858bd64b7fbda767ff9c9f7d8bcd2c2c25 (diff) |
EXP-1841 : pull from current viewer-experience.
Diffstat (limited to 'indra/llui/llscrollcontainer.cpp')
-rw-r--r-- | indra/llui/llscrollcontainer.cpp | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/indra/llui/llscrollcontainer.cpp b/indra/llui/llscrollcontainer.cpp index 20bed050ad..9b7e30bb04 100644 --- a/indra/llui/llscrollcontainer.cpp +++ b/indra/llui/llscrollcontainer.cpp @@ -389,10 +389,17 @@ void LLScrollContainer::calcVisibleSize( S32 *visible_width, S32 *visible_height { *show_h_scrollbar = TRUE; *visible_height -= scrollbar_size; - // Note: Do *not* recompute *show_v_scrollbar here because with - // the (- scrollbar_size) we just did we will always add a vertical scrollbar - // even if the height of the items is actually less than the visible size. - // Fear not though: there's enough calcVisibleSize() calls to add a vertical slider later. + + // The view inside the scroll container should not be extended + // to container's full height to ensure the correct computation + // of *show_v_scrollbar after subtracting horizontal scrollbar_size. + + // Must retest now that visible_height has changed + if( !*show_v_scrollbar && ((doc_height - *visible_height) > 1) ) + { + *show_v_scrollbar = TRUE; + *visible_width -= scrollbar_size; + } } } } |