readthedocs / readthedocs/ext-theme
Add double confirmation selection modal
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 12
- Forks
- 6
- Avg merge
- 13h 25m
- Merged PRs (30d)
- 1
Description
Raised in #123, there are a few places in our UI we might want a double confirmation pattern in the modal. This would probably be project deletion, but perhaps also user account removal -- anything that is destructive, critical, and irreversible might be a good candidate.
I raised issue with text double confirmation, but I'd love to use a multiple choice selection double confirmation in a modal if we can. It wouldn't be much harder, but we could always start with either.
For a text input UI I'd probably just use the "Type delete" version, as the other forms of "Type the name of your project" are tedious and we only need to interrupt the user. I'd go this way if we're only adding this in two places.
A selection UI could show a few options, and the selected option has to match the instance that triggered the modal in the first place. This provides us with better confirmation and is less work from the user than typing. I'd go this way if/once we want to bring this to admin UI list tables/etc. We don't have bulk delete operations in our UI, so this would at least be less tedious than typing.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing issue #123 and the linked screencast to understand the proposed double-confirmation interaction. Clarify whether the project wants text confirmation or multiple-choice selection, and which destructive actions are in scope; done requires an agreed modal pattern applied to those specified actions.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- html
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100