Skip to content

Fix dashboard reminder card colors and host request title#8514

Merged
aapeliv merged 3 commits into
developfrom
web/fix/dashboard-reminder-card
May 10, 2026
Merged

Fix dashboard reminder card colors and host request title#8514
aapeliv merged 3 commits into
developfrom
web/fix/dashboard-reminder-card

Conversation

@aapeliv

@aapeliv aapeliv commented May 10, 2026

Copy link
Copy Markdown
Member

The reminder cards on the dashboard had hardcoded yellow backgrounds (#fff5e4 light / #716317 dark) that looked rough in both modes — particularly bad in dark mode. Also, the "respond to host request" reminder title was generic ("Respond to host request") rather than naming the surfer, which made the card less actionable at a glance.

This PR:

  • Replaces the hardcoded card backgrounds with a soft tint of the brand's secondary (orange) color via alpha(), plus a matching subtle border and 12px radius. Works in both light and dark mode without a useColorScheme branch.
  • Updates the host request reminder copy:
    • title: Respond to {{name}} (was: Respond to host request)
    • description: Don't forget to respond to the host request by {{name}}. (was: Don't forget to respond to {{name}} about their stay.)
  • Updates the test to interpolate name into the title assertion.
Screenshot 2026-05-09 at 23 49 55 Screenshot 2026-05-09 at 23 49 49

Testing

  • yarn test ReminderItem.test — all 5 tests pass.
  • yarn lint — clean for the touched files.
  • Visually confirm on the dashboard in both light and dark mode.

Web frontend checklist

  • There are no console warnings when running the app
  • Added tests where relevant
  • Clicked around my changes running locally and it works
  • Checked Desktop, Mobile and Tablet screen sizes

For maintainers

  • Maintainers can push commits to my branch
  • Maintainers can merge this PR for me

This PR was created with the Couchers PR skill.

Replace hardcoded card backgrounds with theme-aware secondary-tinted
colors that look acceptable in both light and dark mode, and include
the surfer's name in the 'respond to host request' reminder title.
@aapeliv
aapeliv requested a review from nabramow as a code owner May 10, 2026 02:33
@vercel

vercel Bot commented May 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
couchers Ready Ready Preview May 10, 2026 9:04am

Request Review

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@nabramow nabramow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was a bit more yellow in the design:

#5688

But I don't have especially string feelings about it.

tested and working fine

@aapeliv
aapeliv merged commit 71f6b77 into develop May 10, 2026
6 checks passed
@aapeliv
aapeliv deleted the web/fix/dashboard-reminder-card branch May 10, 2026 13:19
@github-actions

Copy link
Copy Markdown
Contributor

📝 Release Notes

This PR does not need to be included in release notes.

Reason: This PR makes a small UI polish to dashboard reminder card styling and slightly improves the wording of a host request reminder, but it is not a significant UX improvement or important bug fix that warrants release notes.

🤖 Bot Debug Information

Model: couchers.openai.gpt-5.4
Decision: exclude
Reasoning: This PR makes a small UI polish to dashboard reminder card styling and slightly improves the wording of a host request reminder, but it is not a significant UX improvement or important bug fix that warrants release notes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants