Skip to content

docs: tighten article.jdf example spacing - #17

Open
DenizSAHIN570 wants to merge 1 commit into
uurtech:masterfrom
DenizSAHIN570:fix/article-example-spacing
Open

DenizSAHIN570 wants to merge 1 commit into
uurtech:masterfrom
DenizSAHIN570:fix/article-example-spacing

Conversation

@DenizSAHIN570

Copy link
Copy Markdown
Contributor

Refs #11

The gaps reported in #11 are authored in docs/examples/article.jdf, not introduced by the renderers: elements are absolutely positioned and both renderers honor position.y. The y slots after paragraphs, lists and code blocks were sized for about three more lines than the content renders, leaving 92–110px of empty space between blocks.

This retunes the 8 affected y values (nothing else in the file changes):

Gap Before After
"Every fix-a-typo…" → "The premise" 92px 31px
"What if a document…" → JSON block 110px 20px
List → "Embedding on the web" 94px 29px
"Three lines and a tag" → code snippet 51px 20px
<link…> snippet → "What's next" 104px 29px

Heading → body gaps stay at 20px. The hero spacing at the top of page 1 is untouched.

Measured in the reader (edit mode) via a Playwright harness; jdf validate docs/examples/article.jdf passes. Data-only change, so the three-surface checklist doesn't apply.

🤖 Generated with Claude Code

The y slots after paragraphs/lists/code blocks were sized for ~3 more
lines than the content renders, leaving 92-110px of empty space (and a
detached-looking hover toolbar) between blocks. Retune the 8 affected y
values so block-to-block gaps are ~30px before headings and ~20px before
code blocks, matching the existing 20px heading-to-body rhythm.

Refs uurtech#11

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@DenizSAHIN570
DenizSAHIN570 force-pushed the fix/article-example-spacing branch from 947639a to 39a4415 Compare October 6, 2026 17:38

This branch has not been deployed

No deployments
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