(Need opinions) KRunner: Avoid having a scroll area

Review Request #118791 - Created June 17, 2014 and submitted

Information
Vishesh Handa
plasma-workspace
Reviewers
plasma
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.

  

Files


Marco Martin
Vishesh Handa
Marco Martin
Commit Hook
This review has been submitted with commit 0c834be25392b1fc41c515a02cd357e2f6a9b36f by Vishesh Handa to branch master.
Vishesh Handa
Review request changed

Status: Closed (submitted)

Loading...