Ticket #676 (closed enhancement: wontfix)

Opened 4 years ago

Last modified 3 years ago

"#story:StoryName" paramifier should scan for embedded links

Reported by: EricShulman Owned by: JeremyRuston
Priority: minor Milestone: 2.5.1
Component: core Version:
Severity: low Keywords:
Cc:

Description

rather than using parseParams() to process tiddler content as a space-separated list, use internal tiddler.links information to scan for embedded links. This allows *any* tiddler to be used as a story, regardless of other wiki-syntax contained in the tiddler. If the specified tiddler is a shadow, then fallback to using parseParams() to extract link list from shadow content.

config.paramifiers.story = {
	onstart: function(v) {
		var t=store.getTiddler(v); if (t) t.changed();
		var list=t?t.links:store.getTiddlerText(v,"").parseParams("open",null,false);
		story.displayTiddlers(null,list);
	}
};

This change has been successfully implemented on TiddlyTools. see  http://www.TiddlyTools.com/#CoreTweaks

Change History

  Changed 4 years ago by MartinBudden

  • milestone set to 2.4.1

follow-up: ↓ 3   Changed 4 years ago by MartinBudden

  • milestone changed from 2.4.1 to 2.5

in reply to: ↑ 2   Changed 3 years ago by EricShulman

  • status changed from new to closed
  • resolution set to wontfix

While this change seems minor, in the last Developer Conference Call discussion Jeremy raised some concern that it might introduce a "behavioral mismatch" due to subtle differences in the semantics of how a 'story tiddler' would be processed.

In any case, this functionality is already part of TiddlyTools' StorySaverPlugin?, so it can always be installed as an option for those authors that want to use the behavior in their documents.

Given the above, I'm retracting the proposed change and closing this ticket. Of course, if others think that this should still be in the core, feel free to re-open the ticket.

Note: See TracTickets for help on using tickets.