neurobagel / old-query-tool

User interface for searching across Neurobagel graph
https://query.neurobagel.org/
MIT License
2 stars 0 forks source link

Clarify query parameter names #285

Closed alyssadai closed 10 months ago

alyssadai commented 12 months ago

While we are still only handling non-session-level phenotypic information, we should update the names of the query parameters in the UI accordingly so there is no confusion about which timepoint users are querying for.

These include: "Min/max age" -> "Min/max age at baseline" "Diagnosis" -> "Diagnosis at baseline" "Minimum number of sessions" -> "Minimum number of imaging sessions"

TODO

rmanaem commented 11 months ago

@neurobagel/dev There is one issue with adding a tooltip for diagnosis. Since we already have a tooltip for healthy control and because the healthy control checkbox is implemented as part of the diagnosis categorical component, when we hover over the healthy control checkbox both tooltips get rendered

Image

Are we okay with merging the two?

rmanaem commented 11 months ago

@surchs Could you add a drawing of what you mentioned during stand-up?

surchs commented 11 months ago

something like this: image

  1. add horizontal dividers to visually differentiate two new sections (pheno and imaging)
  2. move all pheno things to pheno section, and all imaging things to imaging section (modality and # sessions)
  3. add one disclaimer at the top of the pheno session that all variables are taken at baseline session