Skip to content

Conversation

@mirmirmirr
Copy link
Member

  • Added logic to support selecting a range of weekdays by clicking the start and end days, including state management for range selection in weekday-calendar.tsx.
  • Refined the UI for weekday selection buttons, increasing their size and improving selected/unselected styling for better user experience in weekday-calendar.tsx.

Why

Because the backend is structured to take only a weekday range

Copilot AI review requested due to automatic review settings October 10, 2025 20:03
Copy link
Contributor

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 implements weekday range selection functionality to support backend requirements for weekday ranges. The changes enable users to select a continuous range of weekdays by clicking start and end days, replacing individual day selection.

  • Added range selection state management with startDay state variable
  • Implemented handleRangeSelect function to manage range selection logic
  • Enhanced UI styling for weekday buttons with improved sizing and selected state appearance

Reviewed Changes

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

File Description
app/ui/components/weekday-calendar.tsx Added range selection logic, improved button styling, and updated type imports
app/_lib/schedule/types.tsx Extracted Weekday type for better type reusability

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mirmirmirr mirmirmirr requested a review from jzgom067 October 10, 2025 20:08
Copy link
Member

@jzgom067 jzgom067 left a comment

Choose a reason for hiding this comment

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

👍

@jzgom067 jzgom067 merged commit a866b80 into main-preview Oct 10, 2025
1 check passed
@jzgom067 jzgom067 deleted the weekday-range-select branch October 14, 2025 16:14
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