Skip to content

[WC-2892]: Calendar improvements #1628

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

rahmanunver
Copy link
Contributor

@rahmanunver rahmanunver commented Jun 17, 2025

Pull request type

New feature (non-breaking change which adds functionality)


Description

Added:

  • Config to select hours of days shown. (ex: from 6am to 6pm)
  • Config to select days of week shown (ex: Mon-Tue-Fri only)
  • Config to either hide the overflowing events with "Show more" or Show them all.
  • Handling the action variables properly with the ListActionValue's

@rahmanunver rahmanunver requested a review from a team as a code owner June 17, 2025 09:24
@rahmanunver rahmanunver force-pushed the spike/WC-2892-calendar-improvements branch from 729563e to 092f468 Compare June 19, 2025 07:54
@github-actions github-actions bot removed the shared label Jun 19, 2025
const weekdayList = range.map(d => loc.format(d, "EEE")).join(", ");

return weekdayList;
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we move out custom week from this function to a separate file?

Copy link
Collaborator

Choose a reason for hiding this comment

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

note: this custom view extraction will always gets recalculated everytime a simple props changed occurs.

@samuelreichert samuelreichert force-pushed the spike/WC-2892-calendar-improvements branch from ee24386 to dbb7293 Compare June 25, 2025 09:27
@samuelreichert samuelreichert force-pushed the spike/WC-2892-calendar-improvements branch from 439a6f9 to 77e1160 Compare June 26, 2025 09:26
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.

3 participants