wojnilowicz added a comment.
In https://phabricator.kde.org/D7787#162826, @volkov wrote:
> Could you try to test it without + 5 and with the following change applied?
> https://phabricator.kde.org/D8590 ?
Without +5 and with https://phabricator.kde.org/D8590 horizontal scrollbar
volkov added a comment.
Could you try to test it without + 5 and with the following change applied?
https://phabricator.kde.org/D8590 ?
REPOSITORY
R236 KWidgetsAddons
REVISION DETAIL
https://phabricator.kde.org/D7787
To: wojnilowicz, #frameworks
Cc: volkov, cfeck, tbaumgart, #kmymoney
This revision was automatically updated to reflect the committed changes.
Closed by commit R236:56c136985448: Set KPageListView width properly (authored
by wojnilowicz).
REPOSITORY
R236 KWidgetsAddons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7787?vs=19457&id=20807
REVISION DET
wojnilowicz added a comment.
In https://phabricator.kde.org/D7787#145222, @cfeck wrote:
> This is for KWidgetsAddons, right?
Yes.
REPOSITORY
R252 Framework Integration
REVISION DETAIL
https://phabricator.kde.org/D7787
To: wojnilowicz, #frameworks
Cc: cfeck, tbaumgart, #kmymo
cfeck added a comment.
This is for KWidgetsAddons, right?
REPOSITORY
R252 Framework Integration
REVISION DETAIL
https://phabricator.kde.org/D7787
To: wojnilowicz, #frameworks
Cc: cfeck, tbaumgart, #kmymoney
wojnilowicz created this revision.
wojnilowicz added a project: Frameworks.
REVISION SUMMARY
sizeHintForIndex returns lower value than sizeHintForColumn and because
of that horizontal scrollbar appears which is unnecessary.
Hardcoded 25 value has been changed to `vertical scrollbar width + 5
