[Differential] [Commented On] D2595: Fix comic applet appearance
davidedmundson (David Edmundson)
noreply at phabricator.kde.org
Sat Aug 27 13:59:50 UTC 2016
davidedmundson added a comment.
Edit:
I was wrong about this being a regression. It seems this has always been here.
the switch between showing compact and full is:
Math.max(minimumWidth, Math.min(centerLayout.comicData.image.nativeWidth * 0.6, implicitWidth)); or implicitWidth
both minimumWidth and implicitWidth are fricking enormous.
Which means by default we'll show the compact on startup, and until the plasmoid is made even bigger than the already huge default size.
(and in fact dragging it larger in both directions does reveal the components correctly)
Your patch works by breaking the binding on minimumWidth used in the switchWidth, so that it now always shows the full version.
Which is replacing it by a different bug.
INLINE COMMENTS
> main.qml:33
> if (centerLayout.comicData.image) {
> return Math.max(minimumWidth, Math.min(centerLayout.comicData.image.nativeWidth * 0.6, implicitWidth));
> } else {
you've broken this binding. minimumWidth is no longer declared within this scope.
though this whole line makes little sense anwyay - see comment below.
REPOSITORY
rKDEPLASMAADDONS Plasma Addons
REVISION DETAIL
https://phabricator.kde.org/D2595
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: gladhorn
Cc: davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160827/9e7ae30b/attachment.html>
More information about the Plasma-devel
mailing list