New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix:MultiSelect on:change triggered twice #1341
base: main
Are you sure you want to change the base?
Conversation
@jumtp is attempting to deploy a commit to the Themesberg Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughThe update to the Changes
Assessment against linked issues
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- src/lib/forms/MultiSelect.svelte (3 hunks)
Additional comments not posted (5)
src/lib/forms/MultiSelect.svelte (5)
6-8
: The setup of the dispatcher usingcreateEventDispatcher
is correctly implemented and follows Svelte's best practices.
43-45
: The refactoring of theselectOption
function to use the dispatcher for emitting the 'change' event is well-implemented and should help in preventing the double triggering issue.
52-52
: Using the dispatcher to emit the 'change' event in theclearAll
function is correctly implemented.
58-58
: The use of the dispatcher in theclearThisOption
function to emit the 'change' event is consistent with other changes and correctly implemented.
116-116
: The refactoring of the hidden select element to directly dispatch aninput
event on value change is a good improvement, aligning with the overall strategy to streamline event handling.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Closes #1337
📑 Description
where is the bug from ?
update
always triggered twiceflowbite-svelte/src/lib/forms/MultiSelect.svelte
Lines 64 to 71 in 6ff61d2
How does this PR fix the bug?
<select>
was not bind:value and that just<select {value}/>
.value is control by functionsclearAll、clearThisOption、selectOption
.always triggered twice through
clearAll
test.use:init just for update value so i use dispatcher to call on:change.
Status
✅ Checks
ℹ Additional Information
Summary by CodeRabbit