Skip to content

CmdPal: Reduce DockWindow backdrop switching and visual artifacts#46309

Merged
zateutsch merged 1 commit intomainfrom
dev/jpolasek/b/46308-cmdpal-update-dock-backdrop-only-when-actually-changes
Mar 24, 2026
Merged

CmdPal: Reduce DockWindow backdrop switching and visual artifacts#46309
zateutsch merged 1 commit intomainfrom
dev/jpolasek/b/46308-cmdpal-update-dock-backdrop-only-when-actually-changes

Conversation

@jiripolasek
Copy link
Copy Markdown
Collaborator

Summary of the Pull Request

This PR reduces "blinking" of dock when (any) CmdPal settings changes. It handles only backdrop, not icons.

  • Avoids recreating the acrylic controller when the effective backdrop parameters have not changed.
  • Reuses the transparent backdrop instead of reassigning it during dock refreshes.
  • Cleans up backdrop controllers only when switching backdrop modes or disposing the window.
  • Removes obsolete dock-specific backdrop helper logic now handled directly in DockWindow.

PR Checklist

  • Communication: I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected
  • Tests: Added/updated and all pass
  • Localization: All end-user-facing strings can be localized
  • Dev docs: Added/updated
  • New binaries: Added on the required places
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

- Avoids recreating the acrylic controller when the effective backdrop parameters have not changed.
- Reuses the transparent backdrop instead of reassigning it during dock refreshes.
- Cleans up backdrop controllers only when switching backdrop modes or disposing the window.
- Removes obsolete dock-specific backdrop helper logic now handled directly in DockWindow.
@jiripolasek jiripolasek self-assigned this Mar 19, 2026
@jiripolasek jiripolasek added Product-Command Palette Refers to the Command Palette utility CmdPal - Dock Issues related to the Command Palette Dock labels Mar 19, 2026
@niels9001
Copy link
Copy Markdown
Collaborator

🔥🔥🔥

@michaeljolley
Copy link
Copy Markdown
Contributor

@niels9001, definitely not hot, but want to include this in the hotfix?

@jiripolasek
Copy link
Copy Markdown
Collaborator Author

definitely not hot

MeganSooDogMemeGIF

@michaeljolley
Copy link
Copy Markdown
Contributor

michaeljolley commented Mar 20, 2026

definitely not hot

In the sense that we're getting complaints. Definitely hot in that it's fixing something. 😄

@jiripolasek
Copy link
Copy Markdown
Collaborator Author

definitely not hot

In the since that we're getting complaints. Definitely hot in that it's fixing something. 😄

image

@niels9001
Copy link
Copy Markdown
Collaborator

niels9001 commented Mar 21, 2026

@niels9001, definitely not hot, but want to include this in the hotfix?

Agreed.

@zateutsch zateutsch merged commit 93f80f5 into main Mar 24, 2026
15 checks passed
@michaeljolley michaeljolley deleted the dev/jpolasek/b/46308-cmdpal-update-dock-backdrop-only-when-actually-changes branch March 24, 2026 19:51
zateutsch added a commit that referenced this pull request Mar 25, 2026
Cherry-picked commits:
- Make KBM Editor pinnable (#46482)
- CmdPal: Fix missing primary context command for late-bound items (#46131)
- CmdPal: Ensure DockWindow property cleans up after itself (#46303)
- CmdPal: Hotfix commonCallbacks array initial count to prevent negative number (#46215)
- CmdPal: Fix missing app context menu actions on the main page (#46293)
- CmdPal: Fix dock popup XamlRoot handling on DockControl (#46305)
- CmdPal: Reduce DockWindow backdrop switching and visual artifacts (#46309)
- Always On Top: The opacity should be able to configure the hotkey individually (#46410)
- [OOBE] Ensure the Settings button on the SCOOBE page opens Home, not a blank page (#46203)
- CmdPal: Fix scroller scrolling and down glyph (#46447)
- [Settings] Decouple Settings.UI.Library from PowerDisplay.Lib to fix (#46325)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.98.1 CmdPal - Dock Issues related to the Command Palette Dock Product-Command Palette Refers to the Command Palette utility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dock: Changing any settings causes Dock to refresh backdrop and blink

5 participants