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

unify/style project listing #224

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

michaelkirk
Copy link
Contributor

@michaelkirk michaelkirk commented Mar 12, 2025

FIXES #148 (and beyond!)

Global

before:

Screenshot 2025-03-12 at 12 51 28

after:

I've lifted the styles from the neighbourhood picker list (and tweaked them a bit) and am now using them for the project picker list.

Returning users most likely care more about their existing projects, so I've moved those before the "create" flow. For new users (screenshots farther down) the won't have any existing projects, so the "create" flow will be at the top.

Note: I'm no longer displaying CNT projects on the main LTN page. The more I think about it, the more I think we should try to maintain that these are two different apps with two different databases. Other than us developers, I don't imagine there will be much cross over between the two. But if this is controversial, it's easy to revert this part 50e6a14

Screenshot 2025-03-12 at 16 34 58

cnt

before:

Screenshot 2025-03-12 at 12 51 57

after:

Same idea - project list is before "new projects". It's more drastic here since the "create flow" is so much bigger.

I also stripped "LAD" from the study area name, so it matches with the name from the "create" flow.

Screenshot 2025-03-12 at 16 34 41

Empty state before (global)

before: inexplicable "load a saved file."

Screenshot 2025-03-12 at 12 38 38

after: New project is a button

Screenshot 2025-03-12 at 12 58 54

Empty state before (CNT)

before: inexplicable "Or continue with a previously opened file."

Screenshot 2025-03-12 at 12 39 16

after:

Screenshot 2025-03-12 at 12 20 27

width: 35%;
min-width: 400px;
max-width: 700px;
}
Copy link
Contributor Author

@michaelkirk michaelkirk Mar 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Previously we had a little bit wider left column for CNT, I think to accommodate the three column LAD list, which makes sense.

But honestly, I think we could do a little wider in general, especially on narrower screens, so I've applied that here.

I've also added some min/max widths that I think offer reasonably consistent feeling. All the particular values are up for debate, but I think having some kind of min/max is desirable.

@michaelkirk michaelkirk force-pushed the mkirk/style-project-listing branch from 5482bd3 to f6faf0a Compare March 12, 2025 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Project picker screen empty state is confusing
1 participant