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

put meta-analyses on the same page as everything else? #788

Closed
nicoalee opened this issue Jul 17, 2024 · 1 comment · Fixed by #791
Closed

put meta-analyses on the same page as everything else? #788

nicoalee opened this issue Jul 17, 2024 · 1 comment · Fixed by #791

Comments

@nicoalee
Copy link
Collaborator

Originally, we moved meta-analyses to a separate page in order to give the user a feel that the meta-analysis was a result of the previous steps, and those previous steps are completed and relatively set in stone.

I feel like this gives a disjointed feeling to the UI and the button to move to meta-analyses is kinda weird. I think we should condense it into one single project page, and link the project to the snapshots of the curation/extraction phases. We can add a little UI element that indicates "You are looking at an older version etc etc etc. This may differ from the current state of the project"

@nicoalee
Copy link
Collaborator Author

we have discussed and like the idea of keeping the meta-analyses on a different page, but we can change the styles of the tabs to make it look more tab like (add the boundary and make it look like the selected tab is part of the page) as well as the button below (change the styling so its not a box, its a button)

@nicoalee nicoalee moved this to In Progress in Neurosynth 2.0 Project Map Jul 18, 2024
@nicoalee nicoalee linked a pull request Jul 22, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from In Progress to Done in Neurosynth 2.0 Project Map Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

1 participant