----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127120/#review92631 -----------------------------------------------------------
-1 I'm against complicating the effect as well. I don't want every time a small behavior change gets introduced, a config option popping up to "behave just like before". Especially because it won't. If the size is not animated, more logic would needed to be reintroduced, that "anchoring you did put. that's complexity hidden behind a config option, meaning effectively dead code for most developers, the road to unmaintained :p for me it's either having it or disabling animations completely by disabling the effect, especially because animate move without animating resizing was simply a wrong behavior, we just didn't have the means to do better. effects/morphingpopups/package/contents/code/morphingpopups.js <https://git.reviewboard.kde.org/r/127120/#comment63141> removing this skip small steps probably makes sense - Marco Martin On Feb. 21, 2016, 9:31 a.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127120/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2016, 9:31 a.m.) > > > Review request for Plasma. > > > Repository: kwin > > > Description > ------- > > Disabling both animations makes the effect look like the old Plasma tooltip > animation. > Also don't animate when geometry didn't change + don't skip small steps as > resize may happen in steps so we must animate all of them. > > > Diffs > ----- > > effects/morphingpopups/package/contents/code/morphingpopups.js ce6ad86 > effects/morphingpopups/package/contents/config/main.xml PRE-CREATION > effects/morphingpopups/package/contents/ui/config.ui PRE-CREATION > effects/morphingpopups/package/metadata.desktop f9b192c > > Diff: https://git.reviewboard.kde.org/r/127120/diff/ > > > Testing > ------- > > Changing config from KCM needs KWin restart because even though > `configChanged` is emitted, `readConfig` returns the old values (bug, or am I > doing something wrong?). > > > Thanks, > > David Rosca > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
