Skip to content

Make block spans and block position consistent#9931

Draft
pmario wants to merge 1 commit into
TiddlyWiki:masterfrom
pmario:feat/block-span-discipline
Draft

Make block spans and block position consistent#9931
pmario wants to merge 1 commit into
TiddlyWiki:masterfrom
pmario:feat/block-span-discipline

Conversation

@pmario

@pmario pmario commented Jul 12, 2026

Copy link
Copy Markdown
Member

Cover exactly the block syntax with block node spans and record the parse position; separators live in the gaps.

  • Trim one trailing line end run at the block dispatch, never into a child's span; fix the table row and container ends and the typedblock start directly in their rules
  • Stamp blockPosition beside the rule name; isBlock is untouched for rendering compatibility
  • Note for third-party code: start/end shift for six block rules (positions shipped in v5.4.0) and block nodes gain blockPosition; rendering is unchanged

Part of #9908.

Cover exactly the block syntax with block node spans and record the
parse position; separators live in the gaps. Fixes TiddlyWiki#9914.

* Trim one trailing line end run at the block dispatch, never into a
  child's span; fix the table row and container ends and the
  typedblock start directly in their rules
* Stamp blockPosition beside the rule name; isBlock is untouched for
  rendering compatibility
* Note for third-party code: start/end shift for six block rules
  (positions shipped in v5.4.0) and block nodes gain blockPosition;
  rendering is unchanged
@netlify

netlify Bot commented Jul 12, 2026

Copy link
Copy Markdown

Deploy Preview for tiddlywiki-previews ready!

Name Link
🔨 Latest commit 13d6987
🔍 Latest deploy log https://app.netlify.com/projects/tiddlywiki-previews/deploys/6a541c02f793a800083fd7c2
😎 Deploy Preview https://deploy-preview-9931--tiddlywiki-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown

Confirmed: pmario has already signed the Contributor License Agreement (see contributing.md)

@github-actions

github-actions Bot commented Jul 12, 2026

Copy link
Copy Markdown

📊 Build Size Comparison: empty.html

Branch Size
Base (master) 2492.6 KB
PR 2494.7 KB

Diff: ⬆️ Increase: +2.1 KB


✅ Change Note Status

All change notes are properly formatted and validated!

📝 $:/changenotes/5.5.0/#9914

Type: bugfix | Category: developer
Release: 5.5.0

Block node spans cover exactly their own syntax and record their parse position

🔗 #9914

👥 Contributors: pmario


📖 Change Note Guidelines

Change notes help track and communicate changes effectively. See the full documentation for details.

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.

Block node spans and block position are inconsistent in the parse tree

1 participant