Skip to content

Fix sheet corner border positioning and styling#179

Merged
ecto merged 1 commit intomainfrom
claude/fix-medieval-wrapper-corners-efNEB
May 10, 2026
Merged

Fix sheet corner border positioning and styling#179
ecto merged 1 commit intomainfrom
claude/fix-medieval-wrapper-corners-efNEB

Conversation

@ecto
Copy link
Copy Markdown
Owner

@ecto ecto commented May 10, 2026

Summary

Adjusted the positioning and styling of the sheet corner borders to improve visual alignment and consistency.

Key Changes

  • Updated vertical positioning of top corners (.sheet::before and .sheet::after) from top: -10px to top: -9px
  • Updated horizontal positioning of all corners from ±1px to ±10px for consistent offset
  • Removed content property from bottom corner selectors (.corner-bl and .corner-br) as it was invalid for non-pseudo-elements
  • Updated vertical positioning of bottom corners from bottom: -10px to bottom: -9px

Implementation Details

The changes normalize the corner border positioning to use consistent -9px vertical offset and -10px horizontal offset across all four corners. The removal of the content property from the bottom corner classes suggests these elements rely on their HTML content rather than CSS-generated content.

https://claude.ai/code/session_01A7yh22bp9YqwdmTdbMfXee

Corner divs had `padding: 0 4px` shifting the box-drawing character
~9px inward, but were only offset by `-1px` from the sheet edge, so
the └/┘/┌/┐ glyphs appeared floating well inside the box rather than
sitting on the corners. Increase the offset to `-10px` horizontally
and `-9px` vertically so each glyph's bar lines up with the sheet
border. Also drop the dead `content:` declarations on .corner-bl /
.corner-br (only valid on pseudo-elements; the actual glyphs come
from inline spans).
@vercel
Copy link
Copy Markdown

vercel Bot commented May 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mecheval Ready Ready Preview, Comment May 10, 2026 0:23am
vcad Ready Ready Preview, Comment May 10, 2026 0:23am
vcad-docs Ready Ready Preview, Comment May 10, 2026 0:23am
vcad-mcp Ready Ready Preview, Comment May 10, 2026 0:23am

Request Review

@ecto ecto merged commit 70bcd3c into main May 10, 2026
11 checks passed
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.

2 participants