Skip to content

Conversation

@zachchong
Copy link
Contributor

@zachchong zachchong commented Dec 10, 2025

Fix 1: Mobile: Fix the text overflow issue on calendar (days)
Before:
image

After:
image

Fix 2: As a mobile user, I want to select dates easily by tapping anywhere on the date cell, so that I don’t have to press exactly on the number for it to respond.

Copy link

Copilot AI left a comment

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 addresses two mobile usability issues in the booking calendar component: text overflow in calendar day headers and improved date cell clickability.

  • Changed day header format from "EEEE" (full day name) to "EEE" (abbreviated) to prevent text overflow
  • Added selectable={true} property and styling changes to make entire date cells clickable instead of just the date numbers

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
frontend/src/utils/calendar-utils.ts Updated date format constants to use abbreviated day names
frontend/src/components/booking-calendar/booking-calendar.tsx Added selectable prop and weekday format configuration
frontend/src/components/booking-calendar/booking-calendar.module.scss Added global styles to expand clickable area of date cells

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

export const CURRENT_LOCALE = "en-US";
export const DAY_HEADER_FORMAT = "EEEE dd MMMM";
export const DAY_HEADER_FORMAT = "EEE dd MMMM";
export const WEEKDAY_FORMAT = "EEE";
Copy link

Copilot AI Dec 10, 2025

Choose a reason for hiding this comment

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

[nitpick] The constant name WEEKDAY_FORMAT is ambiguous because both DAY_HEADER_FORMAT and WEEKDAY_FORMAT use the same format value 'EEE'. Consider renaming to WEEKDAY_COLUMN_FORMAT or WEEK_VIEW_DAY_FORMAT to clarify that this is specifically for the weekday column headers in month view, distinct from the day header used elsewhere.

Suggested change
export const WEEKDAY_FORMAT = "EEE";
export const WEEKDAY_COLUMN_FORMAT = "EEE";

Copilot uses AI. Check for mistakes.
@zachchong zachchong merged commit cbaf206 into master Dec 10, 2025
6 checks passed
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