Skip to content

Conversation

@hugosaintemarie
Copy link
Contributor

This PR includes a bunch a minor UI tweaks

  • Visual design - removing unnecessary elements to simplify
  • UX patterns - placement of success messages, placeholder copy, labels
  • Typography - removing almost all use of text-xs (apart from profile email in dropdown)
  • Casing - consistent sentence casing on text
  • Color - adding muted colors to secondary elements for hierarchy
  • Spacing - tweaking spacing for improved grouping and alignment
  • Responsiveness - Tweaking some breakpoint issues
  • Logo - updated SVG of Laravel icon to be thicker

See review on Loom

@hugosaintemarie hugosaintemarie requested review from taylorotwell and tnylea and removed request for tnylea February 20, 2025 20:07
@tnylea
Copy link
Contributor

tnylea commented Feb 20, 2025

This looks fantastic. It's crazy how all those minor updates make it look so much nicer.

Thanks. I approve of merging if Taylor agrees 🫡

@tnylea
Copy link
Contributor

tnylea commented Feb 23, 2025

In my last commit, I fixed the card layout appearance in dark mode. Here is the before.

login-before

And here is the after.

login-after

Looks, much better.

@taylorotwell taylorotwell marked this pull request as draft February 23, 2025 14:22
@taylorotwell taylorotwell merged commit 01fe623 into main Feb 24, 2025
@taylorotwell taylorotwell deleted the ui-pass branch February 24, 2025 01:26
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