Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 2, 2026

The reminder component was consuming excessive screen real estate on week and day views, both of which already display dense content. Additionally, the reminder onboarding steps added unnecessary friction for new users.

Changes

View modifications:

  • Removed reminder from week view header entirely
  • Made reminder conditional in day view header via showReminder prop (defaults to false)
  • Pass showReminder={true} in now view to display reminder

Onboarding:

  • Removed 4 reminder setup steps (intro, configuration, success screens)
  • Cleaned up step constants and exports

Implementation

// Day/Now shared header component
export const Header: FC<HeaderProps> = ({ showReminder = false }) => {
  const reminderRef = useRef<HTMLDivElement>(null);
  
  useReminderHotkey(
    showReminder ? () => reminderRef.current?.focus() : () => {},
    [reminderRef, showReminder],
  );

  return (
    <StyledHeaderRow>
      <StyledLeftGroup />
      {showReminder && <Reminder ref={reminderRef} />}
      <SelectView />
    </StyledHeaderRow>
  );
};

The hotkey handler is conditionally bound to avoid unnecessary event listener setup when the reminder is hidden.

Original prompt

This section details on the original issue you should resolve

<issue_title>only render reminder on the /now view</issue_title>
<issue_description>### Priority

None

Feature Description

The reminder takes up a lot of real-estate, making it too noisy on the week and /day views, which both have a lot of other content already.

Update so that the reminder only appears on the /now view

Also remove the reminder steps from onboarding.

Use Case

This'll make the app more approachable for new users who aren't used to this component.

Removing the reminder step from onboarding will speed up onboarding for users

Additional Context

No response</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Jan 2, 2026
Copilot AI changed the title [WIP] Update reminder to only render on /now view Restrict reminder component to /now view only Jan 2, 2026
Copilot AI requested a review from tyler-dane January 2, 2026 01:02
@tyler-dane
Copy link
Contributor

Closing in favor of #1402

@tyler-dane tyler-dane closed this Jan 2, 2026
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.

only render reminder on the /now view

2 participants