Skip to content

fix: event time label cutoff#1611

Merged
tyler-dane merged 2 commits intomainfrom
feat/event-ux
Apr 5, 2026
Merged

fix: event time label cutoff#1611
tyler-dane merged 2 commits intomainfrom
feat/event-ux

Conversation

@tyler-dane
Copy link
Copy Markdown
Contributor

@tyler-dane tyler-dane commented Apr 5, 2026

Closes #918


Note

Medium Risk
UI refactor replaces styled-components event styling with inline/CSS-variable styles and new hover/drag/pending behaviors, which could cause visual or interaction regressions in the calendar. Logic change hides time labels for short events, affecting rendering expectations.

Overview
Fixes calendar event time-label cutoff by suppressing the time label when an event’s rendered height is below MIN_EVENT_HEIGHT_FOR_TIME_LABEL.

Refactors GridEvent and AllDayEvent to drop styled-components (components/Event/styled.ts removed) in favor of inline styles + utility classes, reimplementing priority-based colors, hover states, cursor behavior, past/draft visual treatment, and resize-handle hit areas using CSS variables and computed CSSProperties.

Reviewed by Cursor Bugbot for commit d5a6580. Bugbot is set up for automated code reviews on this repo. Configure here.

@tyler-dane tyler-dane merged commit 11d9e2b into main Apr 5, 2026
9 checks passed
@tyler-dane tyler-dane deleted the feat/event-ux branch April 5, 2026 12:33
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit d5a6580. Configure here.

e.preventDefault();
onKeyDown?.(event);
}}
>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

AllDayEvent lost text truncation and ellipsis styles

Medium Severity

The old StyledEvent component applied text truncation styles (display: -webkit-box, overflow: hidden, text-overflow: ellipsis, word-break: break-all, -webkit-box-orient: vertical, -webkit-line-clamp: 1) to the title span via its & span:first-child CSS rule. In GridEvent.tsx, these were correctly migrated into the titleStyle object. However, in AllDayEvent.tsx, no equivalent title styles were added. Long all-day event titles will now wrap and get hard-clipped by the parent's overflow-hidden instead of showing a clean single-line ellipsis.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit d5a6580. Configure here.

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.

Time label is cut-off in when event is 30-min long

1 participant