Skip to content

persist checklist state#112

Merged
weswc merged 1 commit into
mainfrom
comment_saves
May 7, 2026
Merged

persist checklist state#112
weswc merged 1 commit into
mainfrom
comment_saves

Conversation

@weswc
Copy link
Copy Markdown
Member

@weswc weswc commented May 6, 2026

Based on feedback from users, the checklist persistence was not as desired. A few key issues:

  • If a checklist was selected prior, then edited in another issue, the priorly selected checklist would also get updated
  • Selecting a different checklist while editing a checklist would clear all non-saved changes

To solve this we have the following semantics:

  • A checklist content persists as edited for the duration of a create pop-up modal
  • Saving a checklist persists the edit for the duration of the browser
  • Any queued checklist is considered unique from the other checklists and is marked as queued

@weswc
Copy link
Copy Markdown
Member Author

weswc commented May 6, 2026

Screen.Recording.2026-05-06.at.5.00.03.PM.mov

@weswc
Copy link
Copy Markdown
Member Author

weswc commented May 6, 2026

More evidence of changing selected checklist and content remains the same when coming back

Screen.Recording.2026-05-06.at.5.01.25.PM.mov

@weswc weswc merged commit 1e10f33 into main May 7, 2026
5 checks passed
@weswc weswc deleted the comment_saves branch May 7, 2026 12:49
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