Review Request 126126: kicker: always show arrows for items with children

Kai Uwe Broulik kde at privat.broulik.de
Tue Dec 1 14:31:24 UTC 2015



> On Nov. 30, 2015, 1:50 nachm., Marco Martin wrote:
> > ping? what's the status of this?
> 
> Eike Hein wrote:
>     I was waiting for Pino, but here's a shot of the proposed variant:
>     
>     ![All arrows everywhere](http://i.imgur.com/FwAmgGZ.png)
> 
> Marco Martin wrote:
>     it looks aa bit more busy indeed, but may really help for submenus that have both categories and apps
> 
> Kai Uwe Broulik wrote:
>     I like it. It might help to add "usability" to such review requests.
> 
> Martin Klapetek wrote:
>     Should kickoff maybe get those arrows too? I think that similar case applies there to its submenus.

Plasma 4 kickoff had them.


- Kai Uwe


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


On Nov. 21, 2015, 12:25 nachm., Pino Toscano wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126126/
> -----------------------------------------------------------
> 
> (Updated Nov. 21, 2015, 12:25 nachm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Always show the arrow in menu items which represent a submenu, instead
> of only when hovering on them. This eases the discovery of which items
> are actually submenus as opposed to actual entries.
> 
> 
> Diffs
> -----
> 
>   applets/kicker/package/contents/ui/ItemListDelegate.qml becf13d278862b20f46eacae628364708abadfdc 
> 
> Diff: https://git.reviewboard.kde.org/r/126126/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Pino Toscano
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151201/c49a76cd/attachment-0001.html>


More information about the Plasma-devel mailing list