Skip to content

Wrap hardlinebreaks regions in a void node#9924

Draft
pmario wants to merge 1 commit into
TiddlyWiki:masterfrom
pmario:fix/hardlinebreaks-void-wrapper
Draft

Wrap hardlinebreaks regions in a void node#9924
pmario wants to merge 1 commit into
TiddlyWiki:masterfrom
pmario:fix/hardlinebreaks-void-wrapper

Conversation

@pmario

@pmario pmario commented Jul 12, 2026

Copy link
Copy Markdown
Member

Preserve the rule names of the nodes inside a """ hard linebreaks region.

  • Return one void wrapper spanning the region, as styleblock does; the rule name lands on the wrapper and the inner nodes keep their own. Void nodes are a rendering pass-through, so output is unchanged
  • Stamp the synthesized br elements explicitly and retire the isRuleStart/isRuleEnd flags from feat: serialize AST node back to wikitext string #8258
  • Adapt the wikitext-serialize rule to the wrapper shape

Compatibility: region content is nested one level deeper and the flags (shipped in v5.4.0) are gone. Practical risk is low; the only known consumer is the wikitext-serialize plugin, adapted here.

Part of #9908.

Preserve the rule names of the nodes inside a """ hard linebreaks
region. Fixes TiddlyWiki#9906.

* Return one void wrapper spanning the region, as styleblock does; the
  rule name lands on the wrapper and the inner nodes keep their own.
  Void nodes are a rendering pass-through, so output is unchanged
* Stamp the synthesized br elements explicitly and retire the
  isRuleStart/isRuleEnd flags from TiddlyWiki#8258
* Adapt the wikitext-serialize rule to the wrapper shape

Compatibility: region content is nested one level deeper and the flags
(shipped in v5.4.0) are gone. Practical risk is low; the only known
consumer is the wikitext-serialize plugin, adapted here.
@netlify

netlify Bot commented Jul 12, 2026

Copy link
Copy Markdown

Deploy Preview for tiddlywiki-previews ready!

Name Link
🔨 Latest commit 5dcae50
🔍 Latest deploy log https://app.netlify.com/projects/tiddlywiki-previews/deploys/6a5400fdf0b97c0008b3a19b
😎 Deploy Preview https://deploy-preview-9924--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 2492.7 KB

Diff: ⬆️ Increase: +0.1 KB


✅ Change Note Status

All change notes are properly formatted and validated!

📝 $:/changenotes/5.5.0/#9906

Type: bugfix | Category: developer
Release: 5.5.0

Hard linebreaks regions keep the rule names of their inner nodes

🔗 #9906

👥 Contributors: pmario


⚠️ Impact: $:/changenotes/5.5.0/#9906/impacts/hardlinebreaks-parse-tree

Impact Type: compatibility-break
Related Change: $:/changenotes/5.5.0/#9906

The parse tree shape of """ hard linebreak regions has changed


📖 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.

Hard linebreaks rule overwrites the rule names of its inner nodes

1 participant