Review Request 124509: Use small button for session combo

Kai Uwe Broulik kde at privat.broulik.de
Fri Aug 14 22:56:16 UTC 2015


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

(Updated Aug. 14, 2015, 10:56 nachm.)


Review request for Plasma and KDE Usability.


Changes
-------

Use application-menu icon (sizing in the screenshot is messed up due to sddm test mode picking up my QT_DEVICE_PIXEL_RATIO but not the font size...)


Repository: plasma-workspace


Description
-------

As discussed with the VDG since it's a more of a power user feature


Diffs (updated)
-----

  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 (updated)
----------------

Login screen
  https://git.reviewboard.kde.org/media/uploaded/files/2015/07/28/cd65bb14-a5cd-4fce-b937-710b732aa0b4__sessioncombo1.png
Latest
  https://git.reviewboard.kde.org/media/uploaded/files/2015/08/14/ce4eccf1-dd4e-4946-86af-47474433315a__sessioncombo2.png


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150814/5a1cc384/attachment.html>


More information about the Plasma-devel mailing list