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

UI: Refine trip planner spacing and typography #399

Merged
merged 1 commit into from
Nov 30, 2024
Merged

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Nov 30, 2024

TL;DR

Refined the visual design of the trip planner UI components and updated the Usual Ride screen layout.

What changed?

  • Adjusted padding and spacing in the Origin/Destination component
  • Modified the vertical spacing between stops in the SavedTripCard
  • Updated the Usual Ride screen text styling and content
  • Changed bottom button positioning to use navigation bar padding
  • Removed redundant spacer in the bottom button section
  • Updated "Let's set the vibe." to "Let's set the vibe!"
  • Enhanced typography for the "Select a color" text

How to test?

  1. Navigate to the Usual Ride screen
  2. Verify the updated text and styling changes
  3. Check the bottom button alignment with the navigation bar
  4. Confirm the Origin/Destination component spacing looks correct
  5. Review the SavedTripCard vertical spacing between stops

Why make this change?

These UI refinements improve the visual consistency and user experience across the app. The adjusted spacing and padding create a more polished look, while the updated text styling and navigation bar handling ensure better compatibility across different devices.

Copy link
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ksharma-xyz ksharma-xyz changed the title Fix paddings UI: Refine trip planner spacing and typography Nov 30, 2024
@ksharma-xyz ksharma-xyz marked this pull request as ready for review November 30, 2024 00:58
@ksharma-xyz ksharma-xyz merged commit decf626 into main Nov 30, 2024
2 checks passed
Copy link
Owner Author

Merge activity

  • Nov 29, 8:06 PM EST: A user merged this pull request with Graphite.

@ksharma-xyz ksharma-xyz deleted the 11-30-fix_paddings branch November 30, 2024 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant