Review Request 118791: (Need opinions) KRunner: Avoid having a scroll area

[Marco Martin]

> On June 17, 2014, 11:31 a.m., Marco Martin wrote:
> > as gut feeling, i think results pages that show too many things at once could be overwhelming, but it can be tried and see how it works, just one thing tough..
> > I don't think you can be completely sure that the results will fit the screen. I think you can leave the scroll area, but try to expand it as needed, even until it goes to touch the bottom margin of the screen (or near to it) and the scrollarea would almost never show a scrollbar, except corner cases.
> 
> Vishesh Handa wrote:
>     Well, we're only showing 20 results, and based on the font we are using, it should always fit. No?
>     
>     I could modify the patch to show the scroll area if it goes near the edge of the screen. Though I don't see how that could happen.

i don't think such assumptions can ever be made.
just someone with a small laptop, but with accessibility settings (like huge fonts and high contrast because of eyesight impairment) 20 items would easily take almost double of screen height


- Marco


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


On June 17, 2014, 11:07 a.m., Vishesh Handa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118791/
> -----------------------------------------------------------
> 
> (Updated June 17, 2014, 11:07 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This is an experiment.
> 
> I've removed the scroll area in the KRunner, and now all of the results are shown. I'm not totally convinced if this is a good idea, hence the review request.
> 
> Advantages -
> * You can see all of the results
> 
> Disadvantages -
> * Consumes more space
> * Less consistent with the old runner as now it no longer has a scroll area
> 
> Currently the way the code is structured, the model only shows 20 results. We need to tell the model a maximum number as it decides how many results should be shown per category based on that number.
> 
> 
> Diffs
> -----
> 
>   lookandfeel/contents/runcommand/RunCommand.qml 7a0503d 
> 
> Diff: https://git.reviewboard.kde.org/r/118791/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> Krunner in its full view
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/06/17/a5743542-405a-4346-aa26-ac9158215555__krunner.png
> 
> 
> Thanks,
> 
> Vishesh Handa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140617/9a347076/attachment.html>