<table><tr><td style="">hein marked 5 inline comments as done.<br />hein added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11663">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11663#inline-58299">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mart</span> wrote in <span style="color: #4b4d51; font-weight: bold;">InlineMessage.qml:91</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">this would make opacity of contentitem 1.0 even when animationg to 0?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">No, because of <tt style="background: #ebebeb; font-size: 13px;">enabled: !root.visible</tt>. implicitHeight is bound to an expression that's recomputed when visible has changed, and returns 0 when visible is false. That means by the time the Behavior would kick off in response to the implicitHeight property changing, it's already disabled. In other words, under the only condition when implicitHeights goes to 0, this Behavior is known to be disabled and the ScriptAction will not run.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11663">https://phabricator.kde.org/D11663</a></div></div><br /><div><strong>To: </strong>hein, Kirigami, mart<br /><strong>Cc: </strong>davidedmundson, ngraham, broulik, plasma-devel, apol, mart, hein<br /></div>