[Differential] [Request, 2 lines] D4477: [Icon Applet] More sensible minimum height

[Kai Uwe Broulik]

broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Take into account the label's height for minimum size. When we resize the applet super tiny, it's likely text wraps onto two lines, so use a two line text as minimum (it already does that for preferredHeight anyway).

TEST PLAN
  I can no longer resize the applet icon to a size where it's invisible or a single letter.
  
  (I find `small` iconSize questionable for this also, though)

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D4477

AFFECTED FILES
  applets/icon/package/contents/ui/main.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170207/669431ca/attachment.html>