> 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.
> 
> Eike Hein wrote:
>     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.)

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

I'm not sure what you refer to, since there was nothing of what you describe in 
my message: I described what my impressions with the new kicker. Also, 
considering nowadays most of the design decision pass through a VDG, I asked 
whether there was a decision of it about this "look" behaviour. That's it.

Back to the actual issue: I'm not sure why the VDG should chime in this 
situation, as from what I see the situation is pretty simple: every menu I see  
- starting from application menus in different toolkits and different OSes - 
usually show arrows in any case for submenus, including previous versions of 
the menus (and I see kickoff in plasma 5.4 has the same issue...). Hence, as a 
matter of coherency, a menu should just display arrows, not just on hover.


- Pino


-----------------------------------------------------------
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
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to