Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

Fix queue buildup on unchanged state#48

Open
brinkokevin wants to merge 1 commit intojsdotlua:main from
brinkokevin:main
Open

Fix queue buildup on unchanged state #48
brinkokevin wants to merge 1 commit intojsdotlua:main from
brinkokevin:main

Conversation

@brinkokevin
Copy link

@brinkokevin brinkokevin commented Jul 4, 2025

Removes update from queue when calling setState with equal value as current state. This would previously cause queue buildup until the next rerender.

Copy link
Collaborator

@jeparlefrancais jeparlefrancais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll let @grilme99 make the final merge, but it feels okay for me to merge as is since it's been merged in upstream (see Roblox/react-luau@d8d83cf). The flag has been tried and results are fine (see this message)

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

Reviewers

@jeparlefrancais jeparlefrancais jeparlefrancais approved these changes

@grilme99 grilme99 Awaiting requested review from grilme99

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /