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.

Sign up for GitHub

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

Jump to bottom

Make required age annotation more obvious #817

Closed
3 of 4 tasks
alyssadai opened this issue Nov 28, 2024 · 0 comments · Fixed by #830
Closed
3 of 4 tasks

Make required age annotation more obvious #817

alyssadai opened this issue Nov 28, 2024 · 0 comments · Fixed by #830
Assignees
@rmanaem

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Nov 28, 2024 •
edited by surchs
Loading

The age transformation annotation is easy to miss - even though I encountered the nice warning message on the download page about age missing "annotations", it took me several seconds to see on the age annotation page that what I was missing was the transformation.

Things to be done:

  • Make all sections of the annotation page/components except for the explanation and review annotation collapsable and collapsed by default
  • Change the phrasing to incomplete annotations: instead of missing annotations
    Image
  • Add a label above the transformation dropdown
  • Put a required star on the transformation dropdown
@surchs surchs added the flag:schedule Flag issue that should go on the roadmap or backlog. label Dec 6, 2024
@alyssadai alyssadai moved this to Backlog in Neurobagel Dec 6, 2024
@alyssadai alyssadai removed the flag:schedule Flag issue that should go on the roadmap or backlog. label Dec 6, 2024
@surchs surchs moved this from Backlog to Specify - Active in Neurobagel Dec 11, 2024
@surchs surchs moved this from Specify - Active to Specify - Done in Neurobagel Dec 12, 2024
@rmanaem rmanaem moved this from Specify - Done to Implement - Active in Neurobagel Dec 12, 2024
@rmanaem rmanaem self-assigned this Dec 12, 2024
@rmanaem rmanaem mentioned this issue Dec 12, 2024
Merged
6 tasks
@rmanaem rmanaem moved this from Implement - Active to Implement - Done in Neurobagel Dec 12, 2024
@surchs surchs moved this from Implement - Done to Review - Active in Neurobagel Dec 12, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmanaem rmanaem

Labels
None yet
Projects
Neurobagel
Status: Review - Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] Made requirement of age annotation more prominent neurobagel/legacy_annotation_tool
3 participants
@surchs @alyssadai @rmanaem