> On Ян. 24, 2016, 4:43 след обяд, David Edmundson wrote: > > >Continous usage, read as suspend/resume/use and so on, cause *cool* > > >tooltip animation transition between item to decrease it permormance > > > > Why does QPropertyAnimation decrease in performance after suspend/restore? > > > > Other than the KWindowSystem change I don't see what this is going to > > alter. You're removing a QPropertyAnimation when it's unused, which won't > > make a difference as it'll be unused anyway.
It's not only 1 circle suspend/resume, it's couple of days. I cannot say QPropertyAnimation is buggy, but i see it allocated once and lifetime is same as panel, read fps are dropped in some summy day :) At same sunny day if you add new panel, animation is great on newly panel with fresh QPropertyAnimation allocated pointer, so let *refresh* the pointer :) - Anthony ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126870/#review91521 ----------------------------------------------------------- On Ян. 24, 2016, 4:08 след обяд, Anthony Fieroni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126870/ > ----------------------------------------------------------- > > (Updated Ян. 24, 2016, 4:08 след обяд) > > > Review request for KDE Frameworks, Plasma and David Edmundson. > > > Bugs: 354353 > http://bugs.kde.org/show_bug.cgi?id=354353 > > > Repository: plasma-framework > > > Description > ------- > > Continous usage, read as suspend/resume/use and so on, cause *cool* tooltip > animation transition between item to decrease it permormance and looks quite > ugly. If add new panel animation is *cool* as is. The perfermonce decrease > can be cause due to swap use, or QPropertyAnimation issue, however we (you > KDE defs and i *like a user*) wants plasmashell and it's components (like > panel) to works *forever* > > > Diffs > ----- > > src/declarativeimports/core/tooltip.cpp a5e223b > src/declarativeimports/core/tooltipdialog.h 2ea8af9 > src/declarativeimports/core/tooltipdialog.cpp 6c3712e > > Diff: https://git.reviewboard.kde.org/r/126870/diff/ > > > Testing > ------- > > Workaround is quite simple: > Animation is refreshed as is, pointer hasn't same lifetime as panel > + Animation is disabled if compositing is not active > > > Thanks, > > Anthony Fieroni > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
