NeurodataWithoutBorders / nwb-guide

NWB GUIDE is a desktop app that provides a no-code user interface for converting neurophysiology data to NWB.
https://nwb-guide.readthedocs.io/
MIT License
22 stars 4 forks source link

The dashes at the top of the page allow users to skip ahead #749

Closed rly closed 1 month ago

rly commented 3 months ago

Describe the issue

Near the top of each page, there are a few clickable colored dashes that represent the subsections in the current section, just like in the left sidebar.

image

These dashes are clickable and allow the user to skip ahead and enter weird states

image

The subsections in the left sidebar are not clickable until you do the subsections before. That makes sense.

Given that we have the left sidebar tracking pipeline progress, I think the top dashes do not add value, add clutter/questions, and can be removed.

The top menu is also redundant with the left sidebar. I could go either way on whether we should remove that, but for now I lean toward suggesting we keep it.

Steps to Reproduce

n/a

Operating System

Mac M1

GUIDE Version

main

Code of Conduct

Yes

Did you confirm this issue was not already reported?

Yes

CodyCBakerPhD commented 3 months ago

Given that we have the left sidebar tracking pipeline progress, I think the top dashes do not add value, add clutter/questions, and can be removed.

Many user testers requested specifically that those pills be clickable

CodyCBakerPhD commented 3 months ago

The top menu is also redundant with the left sidebar. I could go either way on whether we should remove that, but for now I lean toward suggesting we keep it.

Pretty sure I remember TurboTax also has a similar nav for reference through a workflow, though that also is for the case where a single step of the left sidebar has multiple 'inner' steps

garrettmflynn commented 3 months ago

I agree that these are redundant. They're all holdover SODA components, so we might just want to remove and double-down on improvements to the tree representation—which would also remove synchronization overhead.

CodyCBakerPhD commented 2 months ago

Have these been removed yet?

garrettmflynn commented 2 months ago

Not yet, though it's very simple. Just wanted confirmation that we agreed on this change

CodyCBakerPhD commented 2 months ago

Users saw the pills and wanted them to be clickable; the pills are there because TurboTax has them sometimes; but I think I was outvoted on even having the pills in the first place since our left pane spans the entire progression of guided mode

So fine to remove

CodyCBakerPhD commented 1 month ago

removed in #781