Review Request 123645: Force visible scrollbars in ScrollView

David Edmundson david at davidedmundson.co.uk
Tue May 5 19:48:46 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123645/#review79914
-----------------------------------------------------------

Ship it!


I think that Qt report might be waiting on us? Especially as it's a one liner.

Won't be in before Qt 5.5 anyway, so ship this now if you want.

- David Edmundson


On May 5, 2015, 7:43 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123645/
> -----------------------------------------------------------
> 
> (Updated May 5, 2015, 7:43 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Bugs: 347254
>     http://bugs.kde.org/show_bug.cgi?id=347254
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Since there hasn't been any attempts to fix this yet on the Qt side and our plasma stuff is primarily used on Desktop systems as of now, it makes sense to always force scrollbars on, to improve accessibility and consistency with widget applications.
> 
> I wish we could have it switch between transient and non-transient depending on whether you're currently interacting with it through your mouse or finger like Windows does quite nicely.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/plasmastyle/ScrollViewStyle.qml e5cb52e 
> 
> Diff: https://git.reviewboard.kde.org/r/123645/diff/
> 
> 
> Testing
> -------
> 
> I now get permanently visible scrollbars on Plasma, they no longer automatically disappear aftera short timeout. They still won't show when the viewport is large enough.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150505/8c4218cb/attachment.html>


More information about the Plasma-devel mailing list