Skip to content

Fix TodoMVC toggle-all control in dioxus-native - #755

Open
NexusAndromeda wants to merge 1 commit into
DioxusLabs:mainfrom
NexusAndromeda:fix/todomvc-toggle-all
Open

Fix TodoMVC toggle-all control in dioxus-native#755
NexusAndromeda wants to merge 1 commit into
DioxusLabs:mainfrom
NexusAndromeda:fix/todomvc-toggle-all

Conversation

@NexusAndromeda

Copy link
Copy Markdown

Summary

  • Fixes the TodoMVC toggle all control not working in dioxus-native (TodoMVC Example #145).
  • Replaces the hidden checkbox + label pattern with an explicit button and onclick handler.
  • Adds a headless regression test via blitz-test-harness.

Test plan

  • cargo test -p blitz-tests todomvc_toggle_all_has_visible_label_content --test todomvc_toggle_all
  • Manual: cargo run -p todomvc --bin todomvc_native, create todos, click toggle-all to mark/unmark all

Replace the hidden checkbox/label pattern with a clickable button and add a headless regression test so the control stays visible and functional in native rendering.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant