D12944: [weather applet] Show a note why "Show temperature" is disabled

[Nathaniel Graham]

ngraham added a comment.


  +1 for redoing the checkboxes to follow the HIG! There seems to be a bit too much padding between them, though.
  
  As for the message, how about "Temperature cannot be shown because this widget is a part of the system tray, so there would not be enough room."
  
  However, a nicer UI might be a way to actually show the temperature when it's a tray widget instead of the current image. Something like this:
  
    Display today's: (o) Weather condition image
                     ( ) High temperature
                     ( ) Low temperature
                     ( ) Current temperature
  
  That way there would actually be some options for people using the tray widget, instead of a disabled option sitting there taunting them that we have to explain or else we get bug reports.

REPOSITORY
  R114 Plasma Addons

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

To: kossebau, #plasma
Cc: mart, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180518/3349739d/attachment.html>