Skip to content

Commit

Permalink
fix watch multi builtin events not work
Browse files Browse the repository at this point in the history
  • Loading branch information
wjian23 committed Oct 22, 2024
1 parent f971765 commit d0fe271
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,10 @@ Please only add new entries below the [Unreleased](#unreleased---releasedate) he
- **core**: When merge multiple `MixBuiltin` widgets, there may be a premature dropping of the outer `MixBuiltin` before it should occur. (#639 @M-Adoo)
- **core**: `watch!` does not notify the initial value. (#640 @M-Adoo)

- **core**: fix watch multi builtin events not work (#pr @wjian23)

- **core**: fix widget layout when h_algin and v_align are embedded in each other (#pr @wjian23)

### Breaking

- **macros**: Using `@ $w { ... }` will no longer automatically wrap a `FatObj` for `w`. (#639 @M-Adoo)
Expand Down
2 changes: 1 addition & 1 deletion macros/src/symbol_process.rs
Original file line number Diff line number Diff line change
Expand Up @@ -557,7 +557,7 @@ impl DollarRefsCtx {
}
if let (Some(this), Some(other)) = (r.builtin.as_mut(), dollar_ref.builtin) {
for e in other.run_before_clone {
if this.run_before_clone.iter().any(|e2| &e == e2) {
if this.run_before_clone.iter().any(|e2| &e != e2) {
this.run_before_clone.push(e);
}
}
Expand Down

0 comments on commit d0fe271

Please sign in to comment.