diff options
author | Alexander Tkachev | 2019-07-31 00:15:55 +0700 |
---|---|---|
committer | Matan Bareket | 2019-08-03 04:35:48 -0400 |
commit | c91bcbfb94de77d8aa1c168e904209efb7e2618b (patch) | |
tree | cf229260252746b656ff71c0870c758d2f782854 /gui/ThemeEngine.cpp | |
parent | 32a997d243f61f9b7003506417098d20205cba8c (diff) | |
download | scummvm-rg350-c91bcbfb94de77d8aa1c168e904209efb7e2618b.tar.gz scummvm-rg350-c91bcbfb94de77d8aa1c168e904209efb7e2618b.tar.bz2 scummvm-rg350-c91bcbfb94de77d8aa1c168e904209efb7e2618b.zip |
GUI: Fix scrollbars
- removed +1px in ListWidget, added in lordhoto's 2007 commit 68eb28a
(aka r29971 in svn) `Fix for bug #1670082 "GUI: Modern theme gfx glitch
in launcher".`, because it made clip this last line of scrollbar in all
themes, which doesn't look good. In 2007 theme was written in .ini,
which is not the case now. I don't see any glitches after removing this
"fix";
- fixed how scrollbar top and bottom scroll buttons are drawn in
ThemeEngine::drawScrollbar: there were these weird magic numbers, but in
reality extra space that buttons should occupy is hardcoded in
scrollbar.cpp (ScrollBarWidget) and is just +1px.
Diffstat (limited to 'gui/ThemeEngine.cpp')
-rw-r--r-- | gui/ThemeEngine.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gui/ThemeEngine.cpp b/gui/ThemeEngine.cpp index 7e42bc0ab3..598d9c2ab8 100644 --- a/gui/ThemeEngine.cpp +++ b/gui/ThemeEngine.cpp @@ -1030,7 +1030,7 @@ void ThemeEngine::drawScrollbar(const Common::Rect &r, int sliderY, int sliderHe drawDD(kDDScrollbarBase, r); Common::Rect r2 = r; - const int buttonExtra = (r.width() * 120) / 100; + const int buttonExtra = r.width() + 1; // scrollbar.cpp's UP_DOWN_BOX_HEIGHT r2.bottom = r2.top + buttonExtra; drawDD(scrollState == kScrollbarStateUp ? kDDScrollbarButtonHover : kDDScrollbarButtonIdle, r2, |