remove boolean flip to prevent it being negated twice

[Polymeta]

this moves the boolean change into the else case, because when you call selectAll() with isAllSelected = true right now, it will then call clearSelected, which is fine. However, inside that method, the isAllSelected boolean will get set to false already, once its done and the flow goes back to the selectAll(), we can see that after if/else the boolean is flipped again causing the selectAll to not work once all have been selected

to reproduce the error try the following:

In playground:

1. go to multiselect
2. Enable "multiple"
3. Select "toggle all"
4. Trying selecting it again to unselect all and observe this not working

[Polymeta]

I dont think the workflow failing doesn't have to do with the code? Not sure

[TomSmith27]

I think its cause your not a contributor, ive added you, just need to accept

[Polymeta]

Hm, same error even after accepting the invite, maybe it needs time to like refresh or something?

[Polymeta]

Yeah, no dice, maybe it works if someone else starts the workflow?

[TomSmith27]

Looks like its cause your code is in a fork, not sure
marocchino/sticky-pull-request-comment#227

[TomSmith27]

Yeah, no dice, maybe it works if someone else starts the workflow?

Can you remake the request in this repo

[Polymeta]

moved to #40