Skip to content

Collapse Lightweight section by default, split out hollow checkbox - #66

Merged
yawkat merged 3 commits into
mainfrom
editor-lightweight-collapsed
Sep 8, 2026
Merged

Collapse Lightweight section by default, split out hollow checkbox#66
yawkat merged 3 commits into
mainfrom
editor-lightweight-collapsed

Conversation

@yawkat

@yawkat yawkat commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • The "Lightweight" tab now starts collapsed, matching the "Numbering" tab
  • The hollow checkbox is now its own row inside the section instead of being merged into the section header

Test plan

  • editor.toml parses as valid TOML
  • Visually confirm in the GridFlock editor that the section is collapsed by default and the checkbox appears inside it

🤖 Generated with Claude Code

https://claude.ai/code/session_01HUY96G341Ti2WRTG9oVZas

Matches the Numbering tab: the section starts collapsed instead of
always expanded, and the hollow toggle is its own row instead of
being merged into the section header.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HUY96G341Ti2WRTG9oVZas
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

🚀 Preview deployed to https://ea9b1342.gridflock-yawk-at.pages.dev

yawkat and others added 2 commits September 8, 2026 10:54
The hollow/hollow_wall/remove_bottom_lip params already carry their
own descriptions from the .scad comments, so the section-level
description-html was duplicate text.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HUY96G341Ti2WRTG9oVZas
hollow_wall is only read by cell_core/segment_core, which are only
invoked when hollow is true, so it has no effect otherwise.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HUY96G341Ti2WRTG9oVZas
@yawkat
yawkat merged commit 962aad7 into main Sep 8, 2026
1 check 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