[Differential] [Commented On] D3885: Minor fix for plasma folder view filename editor
subdiff (Roman Gilg)
noreply at phabricator.kde.org
Sat Dec 31 11:51:45 UTC 2016
subdiff added a comment.
Good find but why does it need to compare `_height` with `contentHeight` in the beginning at all? Normally what should imo happen is simply:
- Compare `realHeight` with max possible height `maxHeight`
- If larger, set `_height = realHeight`, else `_height = maxHeight`
So just: `_height = Math.min(realHeight, maxHeight)`
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D3885
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: chinmoyr, #plasma
Cc: subdiff, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161231/0100b85b/attachment.html>
More information about the Plasma-devel
mailing list