Skip to content

Replace some wikify widget with call dynamic attribute syntax#9816

Open
Leilei332 wants to merge 2 commits into
TiddlyWiki:masterfrom
Leilei332:avoid-wikify-widget
Open

Replace some wikify widget with call dynamic attribute syntax#9816
Leilei332 wants to merge 2 commits into
TiddlyWiki:masterfrom
Leilei332:avoid-wikify-widget

Conversation

@Leilei332

@Leilei332 Leilei332 commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

This PR replaces unnecessary wikify widget usage with call dynamic attribute syntax.

@netlify

netlify Bot commented Apr 17, 2026

Copy link
Copy Markdown

Deploy Preview for tiddlywiki-previews ready!

Name Link
🔨 Latest commit b987c5f
🔍 Latest deploy log https://app.netlify.com/projects/tiddlywiki-previews/deploys/69e2f58da46a5d0008228211
😎 Deploy Preview https://deploy-preview-9816--tiddlywiki-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@github-actions

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Apr 17, 2026

Copy link
Copy Markdown

📊 Build Size Comparison: empty.html

Branch Size
Base (master) 2488.9 KB
PR 2488.4 KB

Diff: ⬇️ Decrease: 0.5 KB


⚠️ Change Note Status

This PR appears to contain code changes but doesn't include a change note.

Please add a change note by creating a .tid file in editions/tw5.com/tiddlers/releasenotes/<version>/

📚 Documentation: Release Notes and Changes

💡 Note: If this is a documentation-only change, you can ignore this message.

@Jermolene

Copy link
Copy Markdown
Member

Thanks @Leilei332 it's great to see those sorts of simplifications.

Comment thread core/ui/EditTemplate/tags.tid Outdated
Comment thread core/ui/ViewToolbar/new-journal-here.tid Outdated
Comment thread core/ui/TagPickerTagTemplate.tid Outdated
Comment thread core/ui/Actions/new-journal.tid Outdated
@linonetwo

Copy link
Copy Markdown
Contributor

Oh, we can do this now? Previously doing so will only assign procedure's definition to the variable. In the new version it is its call result?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants