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

Make background image toggle more visible #480

Closed
ShrimpCryptid opened this issue Nov 21, 2024 · 1 comment · Fixed by #483
Closed

Make background image toggle more visible #480

ShrimpCryptid opened this issue Nov 21, 2024 · 1 comment · Fixed by #483
Assignees
Labels
needs-detail Needs UX or implementation details. research Needs follow-up/research with users. ux

Comments

@ShrimpCryptid
Copy link
Contributor

ShrimpCryptid commented Nov 21, 2024

Use Case

Requested by Chantelle! The background image toggle is currently buried in the Viewer settings tab. Can we make it more prominent so users who aren't familiar with the app are more likely to toggle it?

A few areas were suggested for this, either in the top section above the canvas (currently contains just the toggle for keeping range) or in the toolbar.

image

This is also related to #303, "redesign viewer settings!"

Acceptance Criteria

Please describe how you know this is done

Details

  • Do users just want to toggle it on and off from here? Will they need to switch between multiple background images?
  • Is selecting the first background image in the dataset okay as a default?
  • If TFE was a nucmorph-only app, this toggle might read, "Show original images" and also decrease the object opacity when toggled. Are there ideas from this that we can reuse?
    • What generalizes and what doesn't? (changing object transparency when background images are enabled?)
  • Semi-related also to Remove Collapse from SettingsTab. #472, where we could add a visibility toggle for background images
@ShrimpCryptid ShrimpCryptid added ux research Needs follow-up/research with users. needs-detail Needs UX or implementation details. labels Nov 21, 2024
@ShrimpCryptid
Copy link
Contributor Author

ShrimpCryptid commented Nov 22, 2024

Update:

  • Opacity should be filed under the Backdrop image settings
  • Opacity should only be applied when backdrops are visible
  • Backdrop visibility should now be a new state boolean and synced to the URL
    • This allows the currently selected backdrop to be saved to URL state even if it's not currently visible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-detail Needs UX or implementation details. research Needs follow-up/research with users. ux
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant