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

dashboard ui design #21

Draft
wants to merge 48 commits into
base: master
Choose a base branch
from
Draft

Conversation

sarem-h
Copy link
Contributor

@sarem-h sarem-h commented Jun 12, 2024

No description provided.

Copy link

github-actions bot commented Jun 12, 2024

Welcome to new-connect! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

deployed preview: https://21.connect-d5y.pages.dev

@ugtthis
Copy link
Contributor

ugtthis commented Jun 13, 2024

Trip card UI

Screenshot 2024-06-13 at 7 31 47 AM

Im a fan of the proportions that resemble the golden ratio. Having a 60/40 split with the map taking up 60% and the drive-info taking 40% is not ideal for this context in my opinion.

My reasoning is that most users use the drive-info more frequently than the map since drive-info has more concrete info that can differentiate from the other drives(ex. time, duration, % engaged, etc). For the new route format where it does not include time for the 0.9.7 update, we will need to come up with a solution which will be a different convo than this message

Having the map smaller while keeping the drive-info side larger benefits the user to help find what they are looking for. Airports are complex but their signs are great at making sure people get where they need to, just by looking up. Text is large, icons/visualization support the text.

This is great when a user has a specific destination in mind. In the airport, the destination in mind is something like the terminal or Baggage Claim. When using comma connect its the specific past drive they want to get more info about.

Screenshot 2024-06-13 at 7 34 47 AM

Will post more comments soon

@ugtthis
Copy link
Contributor

ugtthis commented Jun 13, 2024

Align center for desktop-sized screens. For mobile, full-width.

I tested it on the devices below. Content should have equal padding on both the left and right sides. If on mobile, screen-sized device, the width can be 100%. For an iPhone it does do 100% width but for iPad in portrait mode it does not. But when iPad is in landscape mode the content should be aligned center.

device-tests

@ugtthis
Copy link
Contributor

ugtthis commented Jun 13, 2024

Right panel for trip details - Shane's idea

Shane's idea should work on most desktop screens. Like my laptop screen (16in MacBook) shown in previous comment has enough room to fit a right panel that would show the trip-details. Discord comment here for reference.


right-panel-idea

@incognitojam
Copy link
Member

incognitojam commented Jun 21, 2024

It would be better to create some smaller self-contaned PRs and we can get features merged 🙂
Some things to split out:

  • Route Card redesign
  • Route list "sort by"
  • Permanent drawer on wider screens

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.

4 participants