Skip to content

rearrange device header#611

Open
stefpi wants to merge 12 commits into
masterfrom
device-header
Open

rearrange device header#611
stefpi wants to merge 12 commits into
masterfrom
device-header

Conversation

@stefpi
Copy link
Copy Markdown
Contributor

@stefpi stefpi commented Jun 3, 2026

mobile: before and after

imageimage

desktop: after and before (sorry for the position swap) (click on image to enlarge)
image

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Welcome to 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://611.connect-d5y.pages.dev

@adeebshihadeh
Copy link
Copy Markdown
Contributor

@nickorie all design changes are for you to approve

@stefpi stefpi requested a review from nickorie June 3, 2026 20:41
@nickorie
Copy link
Copy Markdown

nickorie commented Jun 3, 2026

After layout looks great.

  1. Increase the mileage stats text size to match whatever is used for the various location and time headers on the routes.
  2. Would recommend switching 'teleoperate" to "remote control".
  3. If possible, battery text to something more meaningfully like a percentage than just the battery voltage (but that can be done later when hardware is more finalized, since I know it's not as simple as some direct conversion or something)

@stefpi
Copy link
Copy Markdown
Contributor Author

stefpi commented Jun 3, 2026

Screenshot from 2026-06-03 16-03-40 Screenshot from 2026-06-03 16-07-35

increased mileage text size
remote control instead of teleop
gave filter button a background to match the drives and give the feeling that it is a button rather than just text

I think I can read a better battery from the body v1 but as you said it isn't super accurate. I'll do that in a seperate PR.

@stefpi
Copy link
Copy Markdown
Contributor Author

stefpi commented Jun 4, 2026

image

on very small screens I prefer this to not wrap the name of the device

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.

3 participants