Fix Minimap with QtCurve style
ClosedPublic

Authored by sars on Mar 14 2019, 5:00 PM.

Details

Summary

The handle of the scrollbar in QtCurve is much smaller than the scrollbar width when the minimap is on and does not fill the whole width.

This patch moves to always have the scrollbar-handle use the whole width (-2 pixels)

Test Plan

No visual difference with Breeze and with QtCurve the minimap scrollbar handle now uses the whole width of the scrollbar.

Diff Detail

Repository
R39 KTextEditor
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sars created this revision.Mar 14 2019, 5:00 PM
Restricted Application added projects: Kate, Frameworks. · View Herald TranscriptMar 14 2019, 5:00 PM
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel. · View Herald Transcript
sars requested review of this revision.Mar 14 2019, 5:00 PM
brauch accepted this revision.Mar 30 2019, 12:26 AM

Looks sensible, and I think I've even seen this bug already somewhere.

This revision is now accepted and ready to land.Mar 30 2019, 12:26 AM
This revision was automatically updated to reflect the committed changes.
sars added a comment.Mar 30 2019, 9:28 AM

Yes I also have a memory that there was a bug report about it, but I can't find it now....

Thanks for the fix.