Skip to content

feat(web): add repeat icon to recurring events#1506

Open
tibisabau wants to merge 2 commits intoSwitchbackTech:mainfrom
tibisabau:feature/add-repeat-icon
Open

feat(web): add repeat icon to recurring events#1506
tibisabau wants to merge 2 commits intoSwitchbackTech:mainfrom
tibisabau:feature/add-repeat-icon

Conversation

@tibisabau
Copy link

Description

Added a repeat icon from @phosphor-icons/react to visually indicate recurring events across all event display components. The icon appears inline before the event title for events with recurrence rules.

Closes #954.

The code in this pull request was generated by GitHub Copilot with the Claude Sonnet 4.5 model.

Checklist if Applicable

  • The tests passed – yarn test packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.test.tsx packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvents.test.tsx packages/web/src/views/Calendar/components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventContainer.test.tsx
  • Linting passed – yarn prettier . --write
  • Documentation has been added
  • CHANGELOG.md has been updated

Copilot AI review requested due to automatic review settings March 3, 2026 21:56
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

Adds a recurring-event visual indicator in the Calendar UI by displaying a repeat icon inline before event titles when an event is recurring, aligning with issue #954.

Changes:

  • Import and render RepeatIcon before titles in grid events, all-day events, and someday sidebar events.
  • Remove the previous “can’t migrate recurring events” UI in SomedayEventRectangle and replace it with icon-based indication.
  • Introduce new recurring detection logic in SomedayEventRectangle (currently inconsistent with other components).

Reviewed changes

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

File Description
packages/web/src/views/Calendar/components/Sidebar/SomedayTab/SomedayEvents/SomedayEventContainer/SomedayEventRectangle.tsx Adds repeat icon and changes migrate/recurrence UI logic for someday sidebar event rows.
packages/web/src/views/Calendar/components/Grid/AllDayRow/AllDayEvent.tsx Adds repeat icon before title in all-day event rendering.
packages/web/src/views/Calendar/components/Event/Grid/GridEvent/GridEvent.tsx Adds repeat icon before title in time-grid event rendering.

Copy link
Contributor

@tyler-dane tyler-dane left a comment

Choose a reason for hiding this comment

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

Hey @tibisabau , thanks for the submission! Please address copilot's review and tag me once you think it's in good shape according to the contribution guidelines.

@tibisabau tibisabau requested a review from tyler-dane March 4, 2026 20:57
Comment on lines -62 to -67
) : (
<Flex>
<StyledRecurrenceText
onClick={(e) => {
e.stopPropagation();
alert("Can't migrate recurring events");
Copy link
Contributor

Choose a reason for hiding this comment

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

Plz readd this; it's important to stop the user from trying to migrate recurring events, which is a feature we don't support yet

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.

Add repeat icon to recurring events

3 participants