neurobagel / annotation_tool

https://annotate.neurobagel.org/
MIT License
3 stars 7 forks source link

[ENH] Add shortcuts #444

Closed rmanaem closed 1 year ago

rmanaem commented 1 year ago

Implement the following shortcuts in the tool for users to have easy access to

github-actions[bot] commented 1 year ago

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. We have applied the stale-issue label to indicate that this issue should be reviewed again and then either prioritized or closed.

jarmoza commented 1 year ago

Following @surchs' comment on the query tool, what would we like the subtitle to be for the Annotator?

surchs commented 1 year ago

what would we like the subtitle to be for the Annotator

I think something like "harmonize phenotypic variables"?

jarmoza commented 1 year ago

Would "harmonize phenotypic data" be more comprehensive given the categorization and annotation steps?

jarmoza commented 1 year ago

My preference is to move the links specified by Arman to the lower left of the Annotation tool window. This would technically be below the next page button in a new component (tool-footer), but justified left. Screen Shot 2023-07-10 at 5 08 11 PM

Thoughts @rmanaem and @surchs ?

surchs commented 1 year ago

I'm good with either but if it's at the bottom, you might have to scroll to see it, yes? I think that'd be suboptimal. And I can imagine that making it "always visible" / sticky at the bottom might be more work / break other things.

wdyt @jarmoza?

jarmoza commented 1 year ago

Agreed. The footer would have to be sticky.

The other option would be to put the links beneath the title. That would be quicker and less prone to error, though maybe cluttered.

jarmoza commented 1 year ago

@surchs In looking over the possibility and given that the subtitle should probably be beneath the title, I think your solution of justifying these new links to the right separated by vertical bar is the best way to go.

Screen Shot 2023-07-10 at 5 38 16 PM

surchs commented 1 year ago

This might also have the added benefit of wrapping these new fields in the responsive wrapping of the navbar in a collapsing menu when the screen size gets too small