--c33cdab2f37643f7ad36655ede2027e1 Content-Type: text/plain; charset="ascii" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit 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 --c33cdab2f37643f7ad36655ede2027e1 Content-Type: text/html; charset="ascii" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable
subdiff added a comment. | View Revision=
Good find but why does it need to compare <= tt style=3D"background: #ebebeb; font-size: 13px;">_height with contentHeight in the beg= inning at all? Normally what should imo happen is simply:
So just: _height =3D= Math.min(realHeight, maxHeight)