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

[Eike Hein]

> On Nov. 21, 2015, 3:05 p.m., Eike Hein wrote:
> > This is intentional because in particular the root menu looks quite ugly and busy with all arrows ...
> 
> Pino Toscano wrote:
>     To me a menu with submenu entries not marked as such (if not when hovering them) is ugly, inconsistent with every other menu around, and lead me thinking "oh, there must be something wrong, no submenus?" when I looked at it at first.
>     
>     Can you please point me at discussions and/or design documents where this incoherency has been decided? Thanks.

There's been no discussions and/or design documents about it, it was a decision by me when I implemented the menu. I'm open to a discussion about it; the VDG should chime in.


(However please drop the childish passive aggressive act, it's vile and antagonistic, and has no place in our discussion venues.)


- Eike


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


On Nov. 21, 2015, 12:25 p.m., 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 p.m.)
> 
> 
> 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/20151121/72cc6594/attachment.html>