----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119701/#review64224 -----------------------------------------------------------
dolphin/src/statusbar/dolphinstatusbar.cpp <https://git.reviewboard.kde.org/r/119701/#comment44878> Please keep the setFixedHeight for m_spaceInfo and m_progressBar. You should only remove it for m_zoomSlider, because the zoom slider does add an empty area underneath and so the vertical alignment is incorrect in high dpi environments + we also guarantee that the height of the status bar is always equal or bigger then the zoom slider height (line 110), so it is safe to remove the setMaximumHeight for the zoom slider. Maybe the qtcurve problem will be fixed with these changes ;) - Emmanuel Pescosta On Aug. 10, 2014, 11:34 nachm., Mathieu Tarral wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119701/ > ----------------------------------------------------------- > > (Updated Aug. 10, 2014, 11:34 nachm.) > > > Review request for KDE Base Apps. > > > Repository: kde-baseapps > > > Description > ------- > > Fixes display issues of dolphin statusbar widgets when a high DPI value is > used (>= 196 for example). > > Changes : > - remove setFixedHeight to fix vertical alignement > - changes fixed values in setMaximumWidth from 150 to averageCharWidth of > m_label * 15 > > So, the font width now respects the dpi settings because averageCharWidth > will get it's value. > But there is still an hardcoded value (15). > > Note : QtCurve seemed to be bugged (see screenshots) > > > Diffs > ----- > > dolphin/src/statusbar/dolphinstatusbar.cpp 169395e > > Diff: https://git.reviewboard.kde.org/r/119701/diff/ > > > Testing > ------- > > set DPI to 196 (System Settings -> Appearance -> Fonts) > run dolphin > look at widgets width in statusbar > > > File Attachments > ---------------- > > show-bug > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/858ff768-864d-4adf-9824-e38e82c7cf66__show-bug.png > fix-cleanlooks > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/e8484513-446d-4805-b1e8-1315b54f98f3__cleanlooks.png > fix-oxygen > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/7bc31f0b-fe7b-4ed2-8d53-ca00a994e70a__oxygen.png > fix-plastique > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/ecf5d4db-4b60-411a-8da9-40d143d9a0e0__plastique.png > fix-qtcurve-agua > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/599d0de4-f85b-46f1-a33a-465837cdbe78__qtcurve-agua.png > fix-qcurve-breeze > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/0daa1663-db15-4797-9ba7-1a0d0e471d7f__qtcurve-breeze.png > fix-qtcurve-default > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/10/ae97cb7b-2d6b-493d-bba1-4a083e5f7ad7__qtcurve-default.png > > > Thanks, > > Mathieu Tarral > >
