Skip to content

rearrange device header#611

Merged
stefpi merged 16 commits into
masterfrom
device-header
Jun 8, 2026
Merged

rearrange device header#611
stefpi merged 16 commits into
masterfrom
device-header

Conversation

@stefpi

@stefpi stefpi commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

mobile: before and after

imageimage

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

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

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

nickorie commented Jun 3, 2026

Copy link
Copy Markdown

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

stefpi commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author
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

stefpi commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author
image

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

@stefpi stefpi merged commit 8bc9e22 into master Jun 8, 2026
4 checks passed
@stefpi stefpi deleted the device-header branch June 8, 2026 21:12
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