Differential D1584

start struts timer in resizeEvent
ClosedPublic
Actions

Authored by davidedmundson on May 10 2016, 2:16 PM.

Details

Reviewers
mart
graesslin
Group Reviewers
Plasma
Commits
R871:6abc546d3017: start struts timer in resizeEvent
R120:6abc546d3017: start struts timer in resizeEvent
Summary

starting the stuts timer in resize/move events means we don't have to
litter it across any code that might move/resize the panel, also we get
event compression for free (so could probably remove the timer in
future).
Much neater and fixes a bug.

BUG: 362887

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson updated this revision to Diff 3747.May 10 2016, 2:16 PM
davidedmundson retitled this revision from to start struts timer in resizeEvent.
davidedmundson updated this object.
davidedmundson edited the test plan for this revision. (Show Details)
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptMay 10 2016, 2:16 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.May 10 2016, 2:44 PM
mart added a reviewer: mart.
This revision is now accepted and ready to land.May 10 2016, 2:44 PM
graesslin accepted this revision.May 10 2016, 3:06 PM
graesslin added a reviewer: graesslin.
graesslin added a subscriber: graesslin.

Just tested and confirm that it fixes the problem.

Closed by commit R120:6abc546d3017: start struts timer in resizeEvent (authored by davidedmundson). · Explain WhyMay 11 2016, 11:10 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mshell/panelview.cpp (9 lines)

Diff 3757

View Options

shell/panelview.cpp

Loading...