Review Request 124509: Use small button for session combo

[Kai Uwe Broulik]

> On Juli 28, 2015, 4:51 nachm., Martin Klapetek wrote:
> > lookandfeel/contents/loginmanager/Main.qml, lines 187-188
> > <https://git.reviewboard.kde.org/r/124509/diff/1/?file=388390#file388390line187>
> >
> >     I understand from the comment that it's a HACK, I don't understand from the comment how is the hack supposed to "fix" it. Might be worth adding like three words in the spirit of "becuase this property does that"?

The ToolButton uses some strange heuristic to determin when the button should be round and when not. When it's inside a ButtonRow it will stay square and ButtonRow has a property "checkedButton" but yeah I'll update the comment.


- Kai Uwe


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


On Juli 28, 2015, 1:41 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124509/
> -----------------------------------------------------------
> 
> (Updated Juli 28, 2015, 1:41 nachm.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> As discussed with the VDG since it's a more of a power user feature
> 
> 
> Diffs
> -----
> 
>   lookandfeel/contents/loginmanager/Main.qml 065a115 
> 
> Diff: https://git.reviewboard.kde.org/r/124509/diff/
> 
> 
> Testing
> -------
> 
> Work, I had to employ a hack to make the button stay square, and also ComboBox doesn't have a public API to open the popup. I tried using Plasma Menu but it doesn't support exclusive group and the QQC Menu cannot be told where it should open other than the mouse position (also why is the text field not the same height as the login button?)
> 
> 
> File Attachments
> ----------------
> 
> Login screen
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/07/28/cd65bb14-a5cd-4fce-b937-710b732aa0b4__sessioncombo1.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

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