subdiff added inline comments.

INLINE COMMENTS

> DigitalClock.qml:211
>  
> -                height: Math.max(sizehelper.lineCount > 1 ?  2 * 
> Math.round(timeLabel.height / 6) : 2 * Math.round(timeLabel.height / 3), 
> theme.smallestFont.pixelSize)
> +                height: Math.min(Math.max(sizehelper.lineCount > 1 ?  2 * 
> Math.round(timeLabel.height / 6) : 2 * Math.round(timeLabel.height / 3), 
> theme.smallestFont.pointSize), Math.round(timeLabel.height))
>                  width: main.width

@mart : Should this also stay in pixelSize? I assume in general always when 
comparing heights and font sizes, right?

REPOSITORY
  R120 Plasma Workspace

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

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

To: subdiff, #plasma, mart
Cc: mart, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas

Reply via email to