Skip to content

fix(desktop): separate calendar day columns#5377

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/calendar-day-separators
May 29, 2026
Merged

fix(desktop): separate calendar day columns#5377
ComputelessComputer merged 1 commit into
mainfrom
fix/calendar-day-separators

Conversation

@ComputelessComputer
Copy link
Copy Markdown
Collaborator

@ComputelessComputer ComputelessComputer commented May 29, 2026

Add visible column dividers to calendar headers and day cells.


This is part 2 of 3 in a stack made with GitButler:


Note

Low Risk
CSS-only calendar layout tweaks with no logic, data, or security impact.

Overview
The desktop calendar grid now shows clearer vertical separators between day columns.

Weekday header cells get a right border on every column except the last, using border-r-neutral-200 so dividers align with the header row’s existing bottom border. Day cells use the same stronger right border while keeping the lighter bottom border (border-b-neutral-100), so column lines read consistently from the header through the grid.

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

Copy link
Copy Markdown

@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 OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9f2a87c. Configure here.

<div
className={cn([
"border-r border-b border-neutral-100",
"border-r border-b border-r-neutral-200 border-b-neutral-100",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Last column border inconsistency between header and cells

Low Severity

The header in calendar-view.tsx conditionally applies border-r only to columns before the last (i < visibleHeaders.length - 1), but DayCell unconditionally applies border-r border-r-neutral-200 to every cell including those in the last column. This creates a visible vertical line on the right edge of the last column's day cells with no matching border in the header above, producing a visual misalignment between headers and body.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9f2a87c. Configure here.

Base automatically changed from fix/calendar-today-button to main May 29, 2026 15:38
Add visible column dividers to calendar headers and day cells.
@ComputelessComputer ComputelessComputer force-pushed the fix/calendar-day-separators branch from 9f2a87c to 7a04f5d Compare May 29, 2026 15:38
@netlify
Copy link
Copy Markdown

netlify Bot commented May 29, 2026

Deploy Preview for old-char canceled.

Name Link
🔨 Latest commit 7a04f5d
🔍 Latest deploy log https://app.netlify.com/projects/old-char/deploys/6a19b3146fd8280008b98e49

@ComputelessComputer ComputelessComputer merged commit b62e39b into main May 29, 2026
10 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/calendar-day-separators branch May 29, 2026 16:56
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.

1 participant