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

Redaction ux improvements #263

Merged
merged 8 commits into from
Sep 30, 2024
Merged

Redaction ux improvements #263

merged 8 commits into from
Sep 30, 2024

Conversation

marySalvi
Copy link
Collaborator

This PR contains several smaller UI/UX improvements:

  • Fixes sporadic white lines in the ImageData table header
  • ImageData table remains visible during redaction
  • Alters hover color on 'DE-PHI' button to not look disabled
  • Resets progress counter to 0 on new redaction
  • Redaction snackbar closes for new workflow (both new redaction and new input directory selection)
  • Resets tags when new input directory is selected

@marySalvi marySalvi requested a review from manthey September 23, 2024 17:07
@marySalvi marySalvi force-pushed the redaction-UX-improvements branch from 30f415c to a90c279 Compare September 23, 2024 18:01
@manthey
Copy link
Contributor

manthey commented Sep 30, 2024

One future change would be that when a new input directory is selected, we currently don't replace the table until it is parsed and there isn't any feedback that we are working on the new table. Maybe the extant table should become greyed out or vanish or there should be some other indicator.

@marySalvi marySalvi merged commit b76c9e8 into main Sep 30, 2024
4 checks passed
@marySalvi marySalvi deleted the redaction-UX-improvements branch September 30, 2024 21:45
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.

2 participants