cds-snc / platform-forms-client

NextJS application that serves the public-facing website for Forms
https://forms-staging.cdssandbox.xyz/
MIT License
31 stars 12 forks source link

Logic: View gets very large very quick #3811

Closed srtalbot closed 1 week ago

srtalbot commented 2 weeks ago

Logic view

When the form gets sufficiently complex there is a lot of scrolling to see where a flow goes, and it's not easy to see where a line is coming from when they intersect / overlap because the screen is so big.

image.png image.png image.png image.png

The above images are all one vertical pane, lined up. One of the issues for this one in particular is that I'm pointing all of my "off boarding" selections to "review" right now." So sending off boarding sections to a section of the logic grid that is less used could be helpful. Right now we treat each fork equally from a visual perspective, like each path could have the same number of sections after it. It might be that a diversion from the main line of questions is short lived or is meant for off boarding.

timarney commented 2 weeks ago

This is at least in part caused by the varying heights of the nodes

Given a more fixed height and using Top down we can see something more like this.

Screenshot 2024-06-12 at 11 29 44 AM
timarney commented 2 weeks ago

Adjustments made in https://github.com/cds-snc/platform-forms-client/pull/3833

We'll need more sample forms to see where else we need to adjust but I think this is a much better base to start with.

timarney commented 2 weeks ago

@srtalbot @samsadasivan

Going to marked this a ready for testing --- and we can iterate.

Changes can be viewed on staging.

samsadasivan commented 1 week ago

I think this works great for now. As Tim noted, there are a lot of paths we can test (darker background, moving things around so the mapping visual is bigger) but lets wait until power user testing.

LOVE the dotted edges, it oddly helps a lot. It opens the scope of how we can iterate.

Closing this issue for now.