Skip to content

fix(css): give the install command its own top margin - #7

Merged
Sunrisepeak merged 1 commit into
mainfrom
fix/cmd-spacing
Aug 14, 2026
Merged

fix(css): give the install command its own top margin#7
Sunrisepeak merged 1 commit into
mainfrom
fix/cmd-spacing

Conversation

@Sunrisepeak

Copy link
Copy Markdown
Member

Summary

.cmd set only margin-bottom: 6px. Its top spacing came from .iface (margin: 16px 0 4px), the wrapper used on indexes whose plugin declares an interface line.

An index that declares none renders the install command directly after .lede (margin: 6px 0 0). With neither element contributing a margin between them, the description and the command touch:

A small, session-aware terminal UI for DeepSeek Harness
xlings install dsh:dsh-tui

That is a reader's report from dsh-index, whose packages are config layers rather than things you import, so no interface line is ever declared there.

Change

.cmd { margin: 14px 0 6px; }   /* was: margin-bottom: 6px */
.cmd + .cmd { margin-top: 6px; }
.iface .cmd { margin-top: 0; }

The command owns its top margin instead of borrowing an ancestor's.

Test plan

  • python3 -m pytest tests/ -q45 passed (CSS-only change; no test asserts on spacing).
  • Rebuilt dsh-index (71 packages) against this branch and loaded a package page: the lede and the install command are now separated, and pages that do declare an interface line are unchanged because .iface .cmd keeps its margin-top: 0.
  • Consecutive commands (interface + alternates) stay at the original 6px via .cmd + .cmd.

`.cmd` set only `margin-bottom`. Its top spacing came from `.iface`
(`margin: 16px 0 4px`), which wraps it on indexes whose plugin declares
an interface line.

An index that declares none -- dsh-index is one; its packages are config
layers, not things you import -- renders the install command directly
after `.lede`, whose own margin is `6px 0 0`. With no margin between
them the description and the command run together as one block, which
is what a reader of that page reported.

The command now owns its top margin. Consecutive commands stay tight at
6px, and inside `.iface` the wrapper still supplies the gap, so no
existing layout moves.
@Sunrisepeak
Sunrisepeak merged commit a791c67 into main Aug 14, 2026
5 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.

1 participant