[Differential] [Commented On] D2975: [shell] Event compress resizePanel

[bshah (Bhushan Shah)]

bshah added a comment.


  In https://phabricator.kde.org/D2975#55706, @mart wrote:
  
  > adding an event compression timer (that by the way, it's a perfectly fine, many times even desired thing) and solving the useless restore() calls at startup are two ortogonal pissues.
  
  
  Double restore() issue was fixed by other commits..
  
  > an event compression there *may* be a good thing if during the normal use (like, dragging panels around) things like setLength() still get called too much.
  
  In my testing, moving panel from bottom edge to right edge calls resizePanel 78 times.. and moving between two screens it calls it 179 times, IMO this are high numbers to require event compression.
  
  > but right now as the main problem is the startup issue, is too soon to put this in, as it hides the real problem
  
  I agree that we will need to solve startup issue, probably using QQmlParserStatus at proper place..

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D2975

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: bshah, #plasma
Cc: mart, hein, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161010/38b4389c/attachment.html>