Skip to content

Drafts and projects stop reading as one panel with a rule through it - #125

Merged
unitypark merged 1 commit into
mainfrom
fix/two-collections-get-air-between-them
Aug 18, 2026
Merged

Drafts and projects stop reading as one panel with a rule through it#125
unitypark merged 1 commit into
mainfrom
fix/two-collections-get-air-between-them

Conversation

@unitypark

Copy link
Copy Markdown
Owner

/projects shows two collections when a setup was never finished — Setup in
progress
above Your projects. They are sibling .group panels, and
.group carried padding but no vertical margin, so the two sat flush: one
panel with a rule through it rather than two things.

.group + .group {
  margin-top: clamp(1.25rem, 2.5vw, 2rem);
}

Two choices worth a second's thought:

  • Adjacent-sibling, not a margin on every .group. The common case is a
    single collection — projects with no drafts — and a blanket top margin would
    have shifted that page's spacing against the shell for no reason.
  • The same clamp as .group's own padding, so the gap between panels tracks
    the space inside them as the viewport changes, rather than the two drifting
    apart at some width.

Verify

Confirmed against the running dev server rather than by reading the source —
/_next/static/css/app/projects/page.css serves:

.projects_group__yvkjW + .projects_group__yvkjW {
  margin-top: clamp(1.25rem, 2.5vw, 2rem);
}

One CSS rule; no typecheck or test surface touched.

The projects page shows two collections when a setup was never finished:
"Setup in progress" above "Your projects". They are sibling .group panels, and
.group carried padding but no vertical margin — so the two sat flush and read
as a single panel divided by a border rather than as two things.

Adjacent-sibling rather than a margin on every .group: the common case is one
collection, and a blanket top margin would have moved that page's spacing
against the shell for no reason. The value is .group's own padding clamp, so
the space between panels tracks the space inside them across viewports instead
of drifting apart at one width.
@unitypark unitypark self-assigned this Aug 18, 2026
@unitypark
unitypark merged commit 802869d into main Aug 18, 2026
2 checks passed
@unitypark
unitypark deleted the fix/two-collections-get-air-between-them branch August 18, 2026 08:58
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