D26655: show a thin separator between view and scrollbar
Hugo Pereira Da Costa
noreply at phabricator.kde.org
Tue Jan 14 21:16:55 GMT 2020
hpereiradacosta added a comment.
>> my complain here is that this argument was either not made or discarded, when the first switch to thin scrollbar was done.
>> This is my main concern about this change: the going back and forth using adhoc arguments each time to justify the change, often contradicting each other. It means essentially that we either don't know what we are doing, or dont think our changes enough. This is bad imo
>
> If I had been around back then or noticed the patch, I would have made this argument.
For the record, you were around for the original commit that made the scrollbar thiner (https://phabricator.kde.org/D9792)
REPOSITORY
R31 Breeze
BRANCH
phab/scrollbarseparator
REVISION DETAIL
https://phabricator.kde.org/D26655
To: mart, #plasma, #breeze, #vdg, ngraham
Cc: ahiemstra, hpereiradacosta, mthw, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200114/3859ea55/attachment.html>
More information about the Plasma-devel
mailing list