Bug 198501 - Lack of scrollbar in the "?" feature
Summary: Lack of scrollbar in the "?" feature
Status: RESOLVED FIXED
Alias: None
Product: krunner
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-07-01 09:38 UTC by S. Burmeister
Modified: 2009-08-05 10:21 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
spot the arrow (44.68 KB, image/png)
2009-08-05 00:49 UTC, S. Burmeister
Details

Note You need to log in before you can comment on or make changes to this bug.
Description S. Burmeister 2009-07-01 09:38:00 UTC
Version:            (using KDE 4.2.90)
OS:                Linux
Installed from:    SuSE RPMs

If one clicks on the "?" in krunner a list of help items gets displayed. Since there is no scrollbar it is no possible to scroll without a mouse-wheel and thus most of the items stay invisible for the user.

This is with RC1 packages.
Comment 1 FiNeX 2009-07-01 10:21:20 UTC
It is possible to use TAB to scroll items... is it enough?
Comment 2 S. Burmeister 2009-07-01 20:31:23 UTC
what if you do not have a keyboard? Are there any other lists without scrollbar? If not, why should this be an exception?
Comment 3 Jacopo De Simoi 2009-07-18 00:43:52 UTC
SVN commit 998548 by jacopods:

Fix scrolling of results using the mouse by adding two page buttons to the view 
As a bonus, ws cleanup and make sure the configWidget disappears when items are re-used
BUG: 198501
This will be backported to 4.3 


 M  +1 -0      CMakeLists.txt  
 M  +22 -28    interfaces/default/interface.cpp  
 M  +6 -1      interfaces/default/resultitem.cpp  
 M  +9 -8      interfaces/default/resultscene.cpp  
 A             interfaces/default/resultsview.cpp   [License: GPL (v2+)]
 A             interfaces/default/resultsview.h   [License: GPL (v2+)]


WebSVN link: http://websvn.kde.org/?view=rev&revision=998548
Comment 4 Jacopo De Simoi 2009-07-18 00:44:23 UTC
SVN commit 998549 by jacopods:

backport page buttons + cleanup (r998548)  
CCBUG: 198501



 M  +1 -0      CMakeLists.txt  
 M  +23 -29    interfaces/default/interface.cpp  
 M  +6 -1      interfaces/default/resultitem.cpp  
 M  +9 -8      interfaces/default/resultscene.cpp  
 A             interfaces/default/resultsview.cpp   [License: GPL (v2+)]
 A             interfaces/default/resultsview.h   [License: GPL (v2+)]


WebSVN link: http://websvn.kde.org/?view=rev&revision=998549
Comment 5 Jacopo De Simoi 2009-07-18 00:46:53 UTC
We did not implement a scrollbar for design reasons; 
for a screenshot of the actual implementation please check:

http://reviewboard.kde.org/r/948/s/135/

Thanks
Comment 6 S. Burmeister 2009-08-05 00:30:14 UTC
The arrow is hardly visible. This is still an usability issue. Should I open a new report for that?
Comment 7 Jacopo De Simoi 2009-08-05 00:39:33 UTC
That is mainly a theme issue (works perfectly fine with the oxygen theme). In fact please open a new report so that we can CC the oxygen people just for this issue.
Thanks
Comment 8 S. Burmeister 2009-08-05 00:47:29 UTC
Sorry, but oxygen is not working well. See attached screenshot, the arrow is nearly invisible.
Comment 9 S. Burmeister 2009-08-05 00:49:02 UTC
Created attachment 35847 [details]
spot the arrow
Comment 10 Aaron J. Seigo 2009-08-05 01:14:55 UTC
it would help if the font you were using wasn't so horrible :/ in any case, we'll probably eventually also implement click-hold-drag scrolling.
Comment 11 Jacopo De Simoi 2009-08-05 08:20:47 UTC
(In reply to comment #8)
> Sorry, but oxygen is not working well. See attached screenshot, the arrow is
> nearly invisible.

btw the theme you are referring to is called "air" not "oxygen" (which is the old black default one).
Comment 12 S. Burmeister 2009-08-05 09:26:25 UTC
(In reply to comment #10)
> it would help if the font you were using wasn't so horrible :/ in any case,
> we'll probably eventually also implement click-hold-drag scrolling.

What does the font have to do with the arrow's size being too small to be spotted easily? Or the "scroll action" being at a place the user is not used to?

The scrolling is not only about the ability to scroll (without a mousewheel or without keyboard) but also about showing that there is more. As you can see in the screenshot, there is no indication that there are more than those three entries, because the last one ends where the window ends.

And the font is Arial truetype with bytecode enabled and anti-alias disabled, it does not get sharper than that. Why it is used in italic I don't know, as I did not set that up, I guess it's default.

@Jacopo: Regarding the theme, it's true, I thought you meant the windeco, which does not make sense since this obviously depends on the plasma theme. But hey, air is only default in KDE 4.3.
Comment 13 Jacopo De Simoi 2009-08-05 09:38:16 UTC
> As you can see in
> the screenshot, there is no indication that there are more than those three
> entries, because the last one ends where the window ends.

Actually the results fade close to the edge, that should hint you that "there is more to scroll down there".. indeed it's hardly noticeable with the font you are using.

> @Jacopo: Regarding the theme, it's true, I thought you meant the windeco, which
> does not make sense since this obviously depends on the plasma theme. But hey,
> air is only default in KDE 4.3.

I know that air is the default; what I meant is that the fact that it is gray on gray it is a theme issue and not a krunner issue and it has to be addressed as such; I'll talk with the theme coordinator about that, but if you open another bug about that, it will be easier to track things as they evolve. 
Thanks
Comment 14 S. Burmeister 2009-08-05 09:58:59 UTC
(In reply to comment #13)
> Actually the results fade close to the edge, that should hint you that "there
> is more to scroll down there".. indeed it's hardly noticeable with the font you
> are using.

As I mentioned, this is a high quality Sans Serif font, so that is no excuse. I also fail to see how fading another would be different with another font.

The fading out IMHO is not suitable as a signal either because it is used in notification, menus etc. if the content of the shown item does not fit the window (you are right on that one) but none of those offer a scrolling in the fading direction. Neither do they indicate that there are more items, just that the shown item has more text. So the fading does not indicate "scroll to see more items". If there are more items, a scrollbar is used.

> I know that air is the default; what I meant is that the fact that it is gray
> on gray it is a theme issue and not a krunner issue and it has to be addressed
> as such; I'll talk with the theme coordinator about that, but if you open
> another bug about that, it will be easier to track things as they evolve. 

I hestitated because I do not think that it would fix the issue in the right place, i.e. scrollbars are the common signal and tool for scrolling and should be used to mediate "more items" in a list. Further the size is too smal, if the user does not hover it by accident.

Anyway, bug 202587 is about the visibility in hope that the other issues are  addressed as well.
Comment 15 Jacopo De Simoi 2009-08-05 10:21:18 UTC
> So the fading does not indicate "scroll to see more items". 

Granted, in fact that's why we also show arrows.

> Anyway, bug 202587 is about the visibility in hope that the other issues are 
> addressed as well.

Thanks