Pedantic Nitpicking Issues in KRunner

[David Edmundson]

On Sun, Mar 18, 2012 at 7:47 PM, Aaron J. Seigo <aseigo at kde.org> wrote:
> On Sunday, March 18, 2012 01:33:09 David Edmundson wrote:
>> Below is a list of really minor "issues" with the default setup of
>> KRunner. Some possibly aren't in KRunner code.
>
> i agree with the points made other than those which i comment on below.
>
> patches are welcome, as that will definitely ensure that fixes happen quicker
> than hoping others will jump on a bunch of problems recorded in a pdf file :)

Done some. Will sort out everything I said that's not been commented on \o/.
This weekend somehow turned into "hack plasma weekend".

Check RB next Saturday, and it'll be there.

> if you will not be working on these things, then i recommend you record the
> findings under community.kde.org/Plasma so they don't get lost in the mail
> archives.
>
>>     3) Make the delegate taller and word wrap.
>
> this is the best solution.
>
>> * The help view is scrollable, but there is no indication of this.
>>
>>     On the desktop, it's not natural to try to scroll on areas without
>> scrollbars. All scroll areas should have scrollbars.
>
> this has been discussed before. krunner's view area will not have scroll bars.
>
> the help info is shown in a very quick-n-dirty way: re-using the results view
> method. a nice help info specific presentation would be nice and is probably
> the best solution here.
>
Fair enough then, will leave this.

>> * Recent Documents has no description
>>
>>     This makes it stand out as it is the only one. Though I can't
>> think of anything better than "Documents which have been recently
>> opened" which is useless. So I'm not sure.
>
> yes, i don't think a description is required. but it also can't hurt :)
>
>> * Inconsistent padding height above and below line edit.
>>
>>     Makes the whole thing look not particularly aligned. True as a
>> floating window (10px above, 11px under).
>>     (and yes, 1 pixel differences are hugely important!)
>
> this is only the case in the floating window. do this in the top-edge mode (the
> default) and it looks worse.
>
> --
> Aaron Seigo
> Make Play Live
> http://makeplaylive.com
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>