Skip to content

Code Quality: Reduce datetime update UI dispatch overhead#18467

Open
muko wants to merge 1 commit into
files-community:mainfrom
muko:reduce-update-datetime-ui-dispatch
Open

Code Quality: Reduce datetime update UI dispatch overhead#18467
muko wants to merge 1 commit into
files-community:mainfrom
muko:reduce-update-datetime-ui-dispatch

Conversation

@muko
Copy link
Copy Markdown
Contributor

@muko muko commented May 8, 2026

Resolved / Related Issues

This PR improves date/time display refresh performance by reducing excessive UI-thread queue buildup.
It introduces a unified UI dispatch path and consolidates date property updates into batched/differential updates.

  • Unified UI dispatch handling to simplify update paths
  • Replaced self-assignment style updates with differential updates (update only when the rendered string actually changes)

Steps used to test these changes

  • Confirmed that date display updates

@yair100 yair100 requested a review from hishitetsu May 13, 2026 16:46
@yair100 yair100 added the ready for review Pull requests that are ready for review label May 13, 2026
@yair100
Copy link
Copy Markdown
Member

yair100 commented May 13, 2026

I recall we used to have a lot of issues with this code, hopefully this change helps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants