Skip to content

Fix: fix ui command exec order in dedicated thread mode.#540

Merged
andycall merged 21 commits intomainfrom
fix/ui_command_order
Jan 18, 2024
Merged

Fix: fix ui command exec order in dedicated thread mode.#540
andycall merged 21 commits intomainfrom
fix/ui_command_order

Conversation

@andycall
Copy link
Copy Markdown
Member

@andycall andycall commented Jan 3, 2024

The UICommands, generated from the JS thread, have the potential to be executed unexpected in the UI thread.

This patch simulates the UICommand execution behavior, as in single-thread mode, to maintain consistent rendering results.

@andycall andycall force-pushed the fix/ui_command_order branch from 2d8377f to c6a554a Compare January 10, 2024 13:21
@andycall andycall force-pushed the fix/ui_command_order branch from 0df2015 to b1c1315 Compare January 17, 2024 11:34
@ErosZy ErosZy added this pull request to the merge queue Jan 18, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Jan 18, 2024
@ErosZy ErosZy added this pull request to the merge queue Jan 18, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Jan 18, 2024
@andycall andycall force-pushed the fix/ui_command_order branch from 56631a8 to 3d490ea Compare January 18, 2024 06:36
@andycall andycall added this pull request to the merge queue Jan 18, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Jan 18, 2024
@andycall andycall force-pushed the fix/ui_command_order branch from 1644bbd to 98799ff Compare January 18, 2024 11:56
@andycall andycall added this pull request to the merge queue Jan 18, 2024
Merged via the queue into main with commit 4642465 Jan 18, 2024
@andycall andycall deleted the fix/ui_command_order branch January 18, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants