Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify deleting a low value chain, CTRL-z style (advanced) #1454

Open
6 tasks
johnwdubois opened this issue Aug 4, 2024 · 0 comments
Open
6 tasks

Simplify deleting a low value chain, CTRL-z style (advanced) #1454

johnwdubois opened this issue Aug 4, 2024 · 0 comments
Labels
polish This issue is about a small item of polishing, mainly aesthetics.

Comments

@johnwdubois
Copy link
Owner

johnwdubois commented Aug 4, 2024

Goal

  • "Accidental" Trails (e.g. a Trail with just one Track in it, that the user just made) should be very easy to delete, without any confirmation dialog.
  • The same goal is relevant for all chains (Trail, Resonance, Stack, Clique)

What to do (quick)

  • The desired behavior already seems to exist for the ESC key (just try it).
  • Let the DEL key just copy the ESC key behavior?

What to do (fancy)

  1. When any new Trail is created, set LowValueChain = TRUE.
  2. Change LowValueChain to FALSE whenever...
  • a 2nd Track is added to the Trail (i.e. Trail has ever had more than 1 Track in it)
  • a tag is added (i.e. if the Trail, or any of its Tracks, has ever had any tags attached to it)
  1. If a Trail has LowValueChain = TRUE, let the user delete it immediately, without confirmation, just by pressing any of the following...
  • "Delete" key
  • "Esc" key
  • "CTRL-z"
  1. Do the same for low value (e.g. "accidental")...
  • Resonance chains
  • Stacks
  • Cliques?

See also
Simplify the warning for deleting any Trail #1465

Fancy

  1. If a deletion warning must be shown, make clear what it applies to. Do not let the warning dialog box be drawn on top of the trail being referenced, where the user cannot see what it being deleted. Instead:
  • show the warning directly below the most recently clicked Track token in the Trail
  • if there is no room below the Track token, show the warning directly above it
  • alternatively, show it to left or right

Screenshot
This shows the current alert, which is both verbose and unclear (because it covers up the relevant Track/Trail).

image
@johnwdubois johnwdubois added the min label Aug 4, 2024
@johnwdubois johnwdubois added this to Core Aug 4, 2024
@github-project-automation github-project-automation bot moved this to To do in Core Aug 4, 2024
@johnwdubois johnwdubois added the polish This issue is about a small item of polishing, mainly aesthetics. label Aug 6, 2024
@johnwdubois johnwdubois changed the title Simplify the warning for deleting a Trail Simplify deleting a Trail Aug 6, 2024
@johnwdubois johnwdubois added enhancement New feature or request and removed min enhancement New feature or request labels Aug 11, 2024
@johnwdubois johnwdubois changed the title Simplify deleting a Trail Simplify deleting a "low-value" chain Aug 19, 2024
@johnwdubois johnwdubois changed the title Simplify deleting a "low-value" chain Simplify deleting a chain (advanced) Aug 19, 2024
@johnwdubois johnwdubois changed the title Simplify deleting a chain (advanced) Simplify deleting a low value chain, CTRL-z style (advanced) Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
polish This issue is about a small item of polishing, mainly aesthetics.
Projects
Status: To do
Development

No branches or pull requests

1 participant