Skip to content

Conversation

@ugtthis
Copy link
Contributor

@ugtthis ugtthis commented Nov 23, 2025

  • found a past car photo I took and dropped in the comma four
  • wanted the aspect ratio to be more a landscape banner while still showing the steering wheel
  • banner image max width is 2560px for wider displays but stops stretching after that
  • there is extra space between the image and section when mobile, but gets fixed if PR UI: Fix extra top spacing #220 gets merged

New image preview

new-banner-img-demo.mp4

Comparing banners

before-after-banner.mp4

@github-actions
Copy link
Contributor

deployed preview: https://comma-web--pr221-xhff4d9n.web.app

Welcome! 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

@sshane
Copy link
Contributor

sshane commented Nov 24, 2025

looks a bit orange? i like it though!

@ugtthis
Copy link
Contributor Author

ugtthis commented Nov 24, 2025

My previous edit of the photo felt too cold and sterile imo. Wanted this edit to remind me of a warm, nostalgic road trip.

Road trips are way more chill because of the device and they make sunset drives a lot more fun so wanted to lean into those memories. At least thats the intention with this edit

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.

2 participants