Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Plasma Desktop Plasma Desktop
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 28
    • Issues 28
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 46
    • Merge requests 46
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • PlasmaPlasma
  • Plasma DesktopPlasma Desktop
  • Merge requests
  • !715

[containments/panel] Fix initial sizing

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged David Edmundson requested to merge work/panel_size into master Nov 24, 2021
  • Overview 5
  • Commits 2
  • Pipelines 8
  • Changes 1

On startup the panel's containment size is effectively undefined. It then stays that way for 150ms until a timer fires.

Ultimately the timer breaks layouts internal usage of QQmlParserStatus. For runtime changes layout internally most relayouting is buffered internally till the polish event which is once per frame.

Removing this makes startup faster, less glitchy and more declarative.

Testing done: apitrace trace plasmashell --replace starts with the system tray in the right place, only task manger icons animate, and even those look better.

Edited Nov 29, 2021 by David Edmundson
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: work/panel_size