Skip to content

implement single column format for cards on small displays (fixes #6)#7

Merged
RyanS4 merged 1 commit into
mainfrom
6-card-size-rework
Jun 29, 2026
Merged

implement single column format for cards on small displays (fixes #6)#7
RyanS4 merged 1 commit into
mainfrom
6-card-size-rework

Conversation

@RyanS4

@RyanS4 RyanS4 commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Fixes #6

  • Larger screens maintain a double-column card format
  • Smaller displays now combine columns for better readability
{0B4CB396-B8E4-48F8-A6A4-3E873F6BFC54}

@RyanS4 RyanS4 self-assigned this Jun 29, 2026
Copilot AI review requested due to automatic review settings June 29, 2026 22:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the card layout on the Services and Lawn Programs pages to be responsive: cards stack into a single column on small screens for readability, while preserving a multi-column layout on larger screens (addressing Issue #6).

Changes:

  • Replaced repeated inline Box sx props with a shared style.cardContainer per page.
  • Added responsive flexDirection (xs: 'column', md: 'row') to stack cards on small displays.
  • Applied the new container style consistently across all card rows in both pages.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
green-day-website/src/Pages/Services.jsx Introduces a responsive card container style and applies it to each card row to stack on small screens.
green-day-website/src/Pages/LawnPrograms.jsx Applies the same responsive card container pattern to improve mobile layout for program cards.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RyanS4 RyanS4 merged commit eec5aac into main Jun 29, 2026
3 checks passed
@RyanS4 RyanS4 deleted the 6-card-size-rework branch June 29, 2026 23:04
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.

Mobile card component stretching

2 participants