Docs updates

Backwards compatibility

bin/build-site.sh

@@ -120,6 +120,7 @@ node $TW5_BUILD_TIDDLYWIKI \
 	|| exit 1
 
 # /empty.html					Empty
+# /empty.hta					For Internet Explorer
 # /empty-external-core.html		External core empty
 # /tiddlywikicore-<version>.js	Core plugin javascript
 node $TW5_BUILD_TIDDLYWIKI \

boot/boot.js

@@ -613,7 +613,7 @@ $tw.utils.evalGlobal = function(code,context,filename,sandbox,allowGlobals) {
 	// Compile the code into a function
 	var fn;
 	if($tw.browser) {
-		fn = Function("return " + code + "\n\n//# sourceURL=" + filename)(); // See https://github.com/TiddlyWiki/TiddlyWiki5/issues/6839
+		fn = window["eval"](code + "\n\n//# sourceURL=" + filename); // eslint-disable-line no-eval -- See https://github.com/TiddlyWiki/TiddlyWiki5/issues/6839
 	} else {
 		if(sandbox){
 			fn = vm.runInContext(code,sandbox,filename)

core-server/server/routes/get-tiddler-html.js

@@ -33,8 +33,8 @@ exports.handler = function(request,response,state) {
 		}
 		var text = state.wiki.renderTiddler(renderType,renderTemplate,{parseAsInline: true, variables: {currentTiddler: title}});
 
-		var headers = {"Content-Type": renderType};
-		state.sendResponse(200,headers,text,"utf8");
+		// Naughty not to set a content-type, but it's the easiest way to ensure the browser will see HTML pages as HTML, and accept plain text tiddlers as CSS or JS
+		state.sendResponse(200,{},text,"utf8");
 	} else {
 		response.writeHead(404);
 		response.end();

core/language/en-GB/ControlPanel.multids

@@ -6,7 +6,6 @@ Appearance/Caption: Appearance
 Appearance/Hint: Ways to customise the appearance of your TiddlyWiki.
 Basics/AnimDuration/Prompt: Animation duration
 Basics/AutoFocus/Prompt: Default focus field for new tiddlers
-Basics/AutoFocusEdit/Prompt: Default focus field for existing tiddlers
 Basics/Caption: Basics
 Basics/DefaultTiddlers/BottomHint: Use [[double square brackets]] for titles with spaces. Or you can choose to {{retain story ordering||$:/snippets/retain-story-ordering-button}}
 Basics/DefaultTiddlers/Prompt: Default tiddlers

core/language/en-GB/Draft.multids

@@ -1,4 +0,0 @@
-title: $:/language/Draft/
-
-Attribution: Draft of '<<draft-title>>' by {{$:/status/UserName}}
-Title: Draft of '<<draft-title>>'

core/language/en-GB/TiddlerInfo.multids

@@ -9,11 +9,6 @@ Advanced/ShadowInfo/NotShadow/Hint: The tiddler <$link to=<<infoTiddler>>><$text
 Advanced/ShadowInfo/Shadow/Hint: The tiddler <$link to=<<infoTiddler>>><$text text=<<infoTiddler>>/></$link> is a shadow tiddler
 Advanced/ShadowInfo/Shadow/Source: It is defined in the plugin <$link to=<<pluginTiddler>>><$text text=<<pluginTiddler>>/></$link>
 Advanced/ShadowInfo/OverriddenShadow/Hint: It is overridden by an ordinary tiddler
-Advanced/CascadeInfo/Heading: Cascade Details
-Advanced/CascadeInfo/Hint: These are the view template segments (tagged <<tag "$:/tags/ViewTemplate">>) using a cascade filter and their resulting template for the current tiddler.
-Advanced/CascadeInfo/Detail/View: View
-Advanced/CascadeInfo/Detail/ActiveCascadeFilter: Active cascade filter
-Advanced/CascadeInfo/Detail/Template: Template
 Fields/Caption: Fields
 List/Caption: List
 List/Empty: This tiddler does not have a list

core/modules/background-actions.js

@@ -1,116 +0,0 @@
-/*\
-title: $:/core/modules/background-actions.js
-type: application/javascript
-module-type: global
-
-Class to dispatch actions when filters change
-
-\*/
-
-"use strict";
-
-class BackgroundActionDispatcher {
-	constructor(filterTracker, wiki) {
-		this.filterTracker = filterTracker;
-		this.wiki = wiki;
-		this.nextTrackedFilterId = 1;
-		this.trackedFilters = new Map(); // Use Map for better key management
-		// Track the filter for the background actions
-		this.filterTracker.track({
-			filterString: "[all[tiddlers+shadows]tag[$:/tags/BackgroundAction]!is[draft]]",
-			fnEnter: title => this.trackFilter(title),
-			fnLeave: (title, enterValue) => this.untrackFilter(enterValue),
-			fnChange: (title, enterValue) => {
-				this.untrackFilter(enterValue);
-				return this.trackFilter(title);
-			},
-			fnProcess: changes => this.process(changes)
-		});
-	}
-
-	trackFilter(title) {
-		const tiddler = this.wiki.getTiddler(title);
-		const id = this.nextTrackedFilterId++;
-		const tracker = new BackgroundActionTracker({
-			wiki: this.wiki,
-			title,
-			trackFilter: tiddler.fields["track-filter"],
-			actions: tiddler.fields.text
-		});
-		this.trackedFilters.set(id, tracker);
-		return id;
-	}
-
-	untrackFilter(enterValue) {
-		const tracker = this.trackedFilters.get(enterValue);
-		if(tracker) {
-			tracker.destroy();
-		}
-		this.trackedFilters.delete(enterValue);
-	}
-
-	process(changes) {
-		for(const tracker of this.trackedFilters.values()) {
-			tracker.process(changes);
-		}
-	}
-}
-
-/*
-Represents an individual tracked filter. Options include:
-wiki: wiki to use
-title: title of the tiddler being tracked
-trackFilter: filter string to track changes
-actions: actions to be executed when the filter changes
-*/
-class BackgroundActionTracker {
-	constructor({wiki, title, trackFilter, actions}) {
-		this.wiki = wiki;
-		this.title = title;
-		this.trackFilter = trackFilter;
-		this.actions = actions;
-		this.filterTracker = new $tw.FilterTracker(this.wiki);
-		this.hasChanged = false;
-		this.trackerID = this.filterTracker.track({
-			filterString: this.trackFilter,
-			fnEnter: () => { this.hasChanged = true; },
-			fnLeave: () => { this.hasChanged = true; },
-			fnProcess: changes => {
-				if(this.hasChanged) {
-					this.hasChanged = false;
-					console.log("Processing background action", this.title);
-					const tiddler = this.wiki.getTiddler(this.title);
-					let doActions = true;
-					if(tiddler && tiddler.fields.platforms) {
-						doActions = false;
-						const platforms = $tw.utils.parseStringArray(tiddler.fields.platforms);
-						if(($tw.browser && platforms.includes("browser")) || ($tw.node && platforms.includes("node"))) {
-							doActions = true;
-						}
-					}
-					if(doActions) {
-						this.wiki.invokeActionString(
-							this.actions,
-							null,
-							{
-								currentTiddler: this.title
-							},{
-								parentWidget: $tw.rootWidget
-							}
-						);
-					}
-				}
-			}
-		});
-	}
-
-	process(changes) {
-		this.filterTracker.handleChangeEvent(changes);
-	}
-
-	destroy() {
-		this.filterTracker.untrack(this.trackerID);
-	}
-}
-
-exports.BackgroundActionDispatcher = BackgroundActionDispatcher;

core/modules/editor/factory.js

@@ -48,8 +48,8 @@ function editTextWidgetFactory(toolbarEngine,nonToolbarEngine) {
 			this.toolbarNode = this.document.createElement("div");
 			this.toolbarNode.className = "tc-editor-toolbar";
 			parent.insertBefore(this.toolbarNode,nextSibling);
-			this.domNodes.push(this.toolbarNode);
 			this.renderChildren(this.toolbarNode,null);
+			this.domNodes.push(this.toolbarNode);
 		}
 		// Create our element
 		var editInfo = this.getEditInfo(),

core/modules/filter-tracker.js

@@ -1,106 +0,0 @@
-/*\
-title: $:/core/modules/filter-tracker.js
-type: application/javascript
-module-type: global
-
-Class to track the results of a filter string
-
-\*/
-
-"use strict";
-
-class FilterTracker {
-	constructor(wiki) {
-		this.wiki = wiki;
-		this.trackers = new Map();
-		this.nextTrackerId = 1;
-	}
-
-	handleChangeEvent(changes) {
-		this.processTrackers();
-		this.processChanges(changes);
-	}
-
-	/*
-	Add a tracker to the filter tracker. Returns null if any of the parameters are invalid, or a tracker id if the tracker was added successfully. Options include:
-	filterString: the filter string to track
-	fnEnter: function to call when a title enters the filter results. Called even if the tiddler does not actually exist. Called as (title), and should return a truthy value that is stored in the tracker as the "enterValue"
-	fnLeave: function to call when a title leaves the filter results. Called as (title,enterValue)
-	fnChange: function to call when a tiddler changes in the filter results. Only called for filter results that identify a tiddler or shadow tiddler. Called as (title,enterValue), and may optionally return a replacement enterValue
-	fnProcess: function to call each time the tracker is processed, after any enter, leave or change functions are called. Called as (changes)
-	*/
-	track(options = {}) {
-		const {
-			filterString,
-			fnEnter,
-			fnLeave,
-			fnChange,
-			fnProcess
-		} = options;
-		const id = this.nextTrackerId++;
-		const tracker = {
-			id,
-			filterString,
-			fnEnter,
-			fnLeave,
-			fnChange,
-			fnProcess,
-			previousResults: [],
-			resultValues: {}
-		};
-		this.trackers.set(id, tracker);
-		// Process the tracker
-		this.processTracker(id);
-		return id;
-	}
-
-	untrack(id) {
-		this.trackers.delete(id);
-	}
-
-	processTrackers() {
-		for(const id of this.trackers.keys()) {
-			this.processTracker(id);
-		}
-	}
-
-	processTracker(id) {
-		const tracker = this.trackers.get(id);
-		if(!tracker) return;
-		const results = [];
-		// Evaluate the filter and remove duplicate results
-		$tw.utils.each(this.wiki.filterTiddlers(tracker.filterString), title => {
-			$tw.utils.pushTop(results, title);
-		});
-		// Process the newly entered results
-		results.forEach(title => {
-			if(!tracker.previousResults.includes(title) && !tracker.resultValues[title] && tracker.fnEnter) {
-				tracker.resultValues[title] = tracker.fnEnter(title) || true;
-			}
-		});
-		// Process the results that have just left
-		tracker.previousResults.forEach(title => {
-			if(!results.includes(title) && tracker.resultValues[title] && tracker.fnLeave) {
-				tracker.fnLeave(title, tracker.resultValues[title]);
-				delete tracker.resultValues[title];
-			}
-		});
-		// Update the previous results
-		tracker.previousResults = results;
-	}
-
-	processChanges(changes) {
-		for(const tracker of this.trackers.values()) {
-			Object.keys(changes).forEach(title => {
-				if(title && tracker.previousResults.includes(title) && tracker.fnChange) {
-					tracker.resultValues[title] = tracker.fnChange(title, tracker.resultValues[title]) || tracker.resultValues[title];
-				}
-			});
-			if(tracker.fnProcess) {
-				tracker.fnProcess(changes);
-			}
-		}
-	}
-}
-
-exports.FilterTracker = FilterTracker;

core/modules/filters.js

@@ -88,8 +88,8 @@ function parseFilterOperation(operators,filterString,p) {
 						rexMatch = rex.exec(filterString.substring(p));
 					if(rexMatch) {
 						operator.regexp = new RegExp(rexMatch[1], rexMatch[2]);
-	// DEPRECATION WARNING
-	console.log("WARNING: Filter",operator.operator,"has a deprecated regexp operand",operator.regexp);
+						// DEPRECATION WARNING
+						console.log("WARNING: Filter",operator.operator,"has a deprecated regexp operand",operator.regexp);
 						nextBracketPos = p + rex.lastIndex - 1;
 					}
 					else {
@@ -232,7 +232,12 @@ exports.getFilterRunPrefixes = function() {
 
 exports.filterTiddlers = function(filterString,widget,source) {
 	var fn = this.compileFilter(filterString);
-	return fn.call(this,source,widget);
+	try {
+		const fnResult = fn.call(this,source,widget);
+		return fnResult;
+	} catch(e) {
+		return [`${$tw.language.getString("Error/Filter")}: ${e}`];
+	}
 };
 
 /*
@@ -314,19 +319,19 @@ exports.compileFilter = function(filterString) {
 
 				// Invoke the appropriate filteroperator module
 				results = operatorFunction(accumulator,{
-							operator: operator.operator,
-							operand: operands.length > 0 ? operands[0] : undefined,
-							operands: operands,
-							multiValueOperands: multiValueOperands,
-							isMultiValueOperand: isMultiValueOperand,
-							prefix: operator.prefix,
-							suffix: operator.suffix,
-							suffixes: operator.suffixes,
-							regexp: operator.regexp
-						},{
-							wiki: self,
-							widget: widget
-						});
+					operator: operator.operator,
+					operand: operands.length > 0 ? operands[0] : undefined,
+					operands: operands,
+					multiValueOperands: multiValueOperands,
+					isMultiValueOperand: isMultiValueOperand,
+					prefix: operator.prefix,
+					suffix: operator.suffix,
+					suffixes: operator.suffixes,
+					regexp: operator.regexp
+				},{
+					wiki: self,
+					widget: widget
+				});
 				if($tw.utils.isArray(results)) {
 					accumulator = self.makeTiddlerIterator(results);
 				} else {

core/modules/info/mediaquerytracker.js

@@ -1,67 +0,0 @@
-/*\
-title: $:/core/modules/info/mediaquerytracker.js
-type: application/javascript
-module-type: info
-
-Initialise $:/info/ tiddlers derived from media queries via 
-
-\*/
-
-"use strict";
-
-exports.getInfoTiddlerFields = function(updateInfoTiddlersCallback) {
-	if($tw.browser) {
-		// Functions to start and stop tracking a particular media query tracker tiddler
-		function track(title) {
-			var result = {},
-				tiddler = $tw.wiki.getTiddler(title);
-			if(tiddler) {
-				var mediaQuery = tiddler.fields["media-query"],
-					infoTiddler = tiddler.fields["info-tiddler"],
-					infoTiddlerAlt = tiddler.fields["info-tiddler-alt"];
-				if(mediaQuery && infoTiddler) {
-					// Evaluate and track the media query
-					result.mqList = window.matchMedia(mediaQuery);
-					function getResultTiddlers() {
-						var value = result.mqList.matches ? "yes" : "no",
-							tiddlers = [];
-						tiddlers.push({title: infoTiddler, text: value});
-						if(infoTiddlerAlt) {
-							tiddlers.push({title: infoTiddlerAlt, text: value});
-						}
-						return tiddlers;
-					};
-					updateInfoTiddlersCallback(getResultTiddlers());
-					result.handler = function(event) {
-						updateInfoTiddlersCallback(getResultTiddlers());
-					};
-					result.mqList.addEventListener("change",result.handler);
-				}
-			}
-			return result;
-		}
-		function untrack(enterValue) {
-			if(enterValue.mqList && enterValue.handler) {
-				enterValue.mqList.removeEventListener("change",enterValue.handler);
-			}
-		}
-		// Track media query tracker tiddlers
-		function fnEnter(title) {
-			return track(title);
-		}
-		function fnLeave(title,enterValue) {
-			untrack(enterValue);
-		}
-		function fnChange(title,enterValue) {
-			untrack(enterValue);
-			return track(title);
-		}
-		$tw.filterTracker.track({
-			filterString: "[all[tiddlers+shadows]tag[$:/tags/MediaQueryTracker]!is[draft]]",
-			fnEnter: fnEnter,
-			fnLeave: fnLeave,
-			fnChange: fnChange
-		});
-	}
-	return [];
-};

core/modules/info/platform.js

@@ -33,6 +33,13 @@ exports.getInfoTiddlerFields = function(updateInfoTiddlersCallback) {
 		// Screen size
 		infoTiddlerFields.push({title: "$:/info/browser/screen/width", text: window.screen.width.toString()});
 		infoTiddlerFields.push({title: "$:/info/browser/screen/height", text: window.screen.height.toString()});
+ 		// Dark mode through event listener on MediaQueryList
+ 		var mqList = window.matchMedia("(prefers-color-scheme: dark)"),
+ 			getDarkModeTiddler = function() {return {title: "$:/info/darkmode", text: mqList.matches ? "yes" : "no"};};
+ 		infoTiddlerFields.push(getDarkModeTiddler());
+ 		mqList.addListener(function(event) {
+ 			updateInfoTiddlersCallback([getDarkModeTiddler()]);
+ 		});
 		// Language
 		infoTiddlerFields.push({title: "$:/info/browser/language", text: navigator.language || ""});
 	}

core/modules/parsers/parseutils.js

@@ -143,14 +143,7 @@ exports.parseParameterDefinition = function(paramString,options) {
 		var paramInfo = {name: paramMatch[1]},
 			defaultValue = paramMatch[2] || paramMatch[3] || paramMatch[4] || paramMatch[5];
 		if(defaultValue !== undefined) {
-			// Check for an MVV reference ((varname))
-			var mvvDefaultMatch = /^\(\(([^)|]+)\)\)$/.exec(defaultValue);
-			if(mvvDefaultMatch) {
-				paramInfo.defaultType = "multivalue-variable";
-				paramInfo.defaultVariable = mvvDefaultMatch[1];
-			} else {
-				paramInfo["default"] = defaultValue;
-			}
+			paramInfo["default"] = defaultValue;
 		}
 		params.push(paramInfo);
 		// Look for the next parameter
@@ -254,46 +247,6 @@ exports.parseMacroInvocationAsTransclusion = function(source,pos) {
 	return node;
 };
 
-/*
-Look for an MVV (multi-valued variable) reference as a transclusion, i.e. ((varname)) or ((varname params))
-Returns null if not found, or a parse tree node of type "transclude" with isMVV: true
-*/
-exports.parseMVVReferenceAsTransclusion = function(source,pos) {
-	var node = {
-		type: "transclude",
-		isMVV: true,
-		start: pos,
-		attributes: {},
-		orderedAttributes: []
-	};
-	// Define our regexps
-	var reVarName = /([^\s>"'=:)]+)/g;
-	// Skip whitespace
-	pos = $tw.utils.skipWhiteSpace(source,pos);
-	// Look for a double opening parenthesis
-	var token = $tw.utils.parseTokenString(source,pos,"((");
-	if(!token) {
-		return null;
-	}
-	pos = token.end;
-	// Get the variable name
-	token = $tw.utils.parseTokenRegExp(source,pos,reVarName);
-	if(!token) {
-		return null;
-	}
-	$tw.utils.addAttributeToParseTreeNode(node,"$variable",token.match[1]);
-	pos = token.end;
-	// Skip whitespace
-	pos = $tw.utils.skipWhiteSpace(source,pos);
-	// Look for a double closing parenthesis
-	token = $tw.utils.parseTokenString(source,pos,"))");
-	if(!token) {
-		return null;
-	}
-	node.end = token.end;
-	return node;
-};
-
 /*
 Parse macro parameters as attributes. Returns the position after the last attribute
 */
@@ -368,20 +321,19 @@ exports.parseMacroParameterAsAttribute = function(source,pos) {
 				node.type = "indirect";
 				node.textReference = indirectValue.match[1];
 			} else {
-				// Look for a macro invocation value
-				var macroInvocation = $tw.utils.parseMacroInvocationAsTransclusion(source,pos);
-				if(macroInvocation && isNewStyleSeparator) {
-					pos = macroInvocation.end;
-					node.type = "macro";
-					node.value = macroInvocation;
+				// Look for a unquoted value
+				var unquotedValue = $tw.utils.parseTokenRegExp(source,pos,reUnquotedAttribute);
+				if(unquotedValue) {
+					pos = unquotedValue.end;
+					node.type = "string";
+					node.value = unquotedValue.match[1];
 				} else {
-					// Look for an MVV reference value
-					var mvvReference = $tw.utils.parseMVVReferenceAsTransclusion(source,pos);
-					if(mvvReference && isNewStyleSeparator) {
-						pos = mvvReference.end;
+					// Look for a macro invocation value
+					var macroInvocation = $tw.utils.parseMacroInvocationAsTransclusion(source,pos);
+					if(macroInvocation && isNewStyleSeparator) {
+						pos = macroInvocation.end;
 						node.type = "macro";
-						node.value = mvvReference;
-						node.isMVV = true;
+						node.value = macroInvocation;
 					} else {
 						var substitutedValue = $tw.utils.parseTokenRegExp(source,pos,reSubstitutedValue);
 						if(substitutedValue && isNewStyleSeparator) {
@@ -389,14 +341,6 @@ exports.parseMacroParameterAsAttribute = function(source,pos) {
 							node.type = "substituted";
 							node.rawValue = substitutedValue.match[1] || substitutedValue.match[2];
 						} else {
-							// Look for a unquoted value
-							var unquotedValue = $tw.utils.parseTokenRegExp(source,pos,reUnquotedAttribute);
-							if(unquotedValue) {
-								pos = unquotedValue.end;
-								node.type = "string";
-								node.value = unquotedValue.match[1];
-							} else {
-							}
 						}
 					}
 				}
@@ -527,20 +471,19 @@ exports.parseAttribute = function(source,pos) {
 					node.type = "indirect";
 					node.textReference = indirectValue.match[1];
 				} else {
-					// Look for a macro invocation value
-					var macroInvocation = $tw.utils.parseMacroInvocationAsTransclusion(source,pos);
-					if(macroInvocation) {
-						pos = macroInvocation.end;
-						node.type = "macro";
-						node.value = macroInvocation;
+					// Look for a unquoted value
+					var unquotedValue = $tw.utils.parseTokenRegExp(source,pos,reUnquotedAttribute);
+					if(unquotedValue) {
+						pos = unquotedValue.end;
+						node.type = "string";
+						node.value = unquotedValue.match[1];
 					} else {
-						// Look for an MVV reference value
-						var mvvReference = $tw.utils.parseMVVReferenceAsTransclusion(source,pos);
-						if(mvvReference) {
-							pos = mvvReference.end;
+						// Look for a macro invocation value
+						var macroInvocation = $tw.utils.parseMacroInvocationAsTransclusion(source,pos);
+						if(macroInvocation) {
+							pos = macroInvocation.end;
 							node.type = "macro";
-							node.value = mvvReference;
-							node.isMVV = true;
+							node.value = macroInvocation;
 						} else {
 							var substitutedValue = $tw.utils.parseTokenRegExp(source,pos,reSubstitutedValue);
 							if(substitutedValue) {
@@ -548,16 +491,8 @@ exports.parseAttribute = function(source,pos) {
 								node.type = "substituted";
 								node.rawValue = substitutedValue.match[1] || substitutedValue.match[2];
 							} else {
-								// Look for a unquoted value
-								var unquotedValue = $tw.utils.parseTokenRegExp(source,pos,reUnquotedAttribute);
-								if(unquotedValue) {
-									pos = unquotedValue.end;
-									node.type = "string";
-									node.value = unquotedValue.match[1];
-								} else {
-									node.type = "string";
-									node.value = "true";
-								}
+								node.type = "string";
+								node.value = "true";
 							}
 						}
 					}

core/modules/parsers/wikiparser/rules/fnprocdef.js

@@ -32,7 +32,7 @@ Instantiate parse rule
 exports.init = function(parser) {
 	this.parser = parser;
 	// Regexp to match
-	this.matchRegExp = /\\(function|procedure|widget)\s+([^(\s]+)\((\s*([^)]*(?:\)\)[^)]*)*))?\)(\s*\r?\n)?/mg;
+	this.matchRegExp = /\\(function|procedure|widget)\s+([^(\s]+)\((\s*([^)]*))?\)(\s*\r?\n)?/mg;
 };
 
 /*

core/modules/parsers/wikiparser/rules/mvvdisplayinline.js

@@ -1,95 +0,0 @@
-/*\
-title: $:/core/modules/parsers/wikiparser/rules/mvvdisplayinline.js
-type: application/javascript
-module-type: wikirule
-
-Wiki rule for inline display of multi-valued variables and filter results.
-
-Variable display: ((varname)) or ((varname||separator))
-Filter display: (((filter))) or (((filter||separator)))
-
-The default separator is ", " (comma space).
-
-\*/
-
-"use strict";
-
-exports.name = "mvvdisplayinline";
-exports.types = {inline: true};
-
-exports.init = function(parser) {
-	this.parser = parser;
-};
-
-exports.findNextMatch = function(startPos) {
-	var source = this.parser.source;
-	var nextStart = startPos;
-	while((nextStart = source.indexOf("((",nextStart)) >= 0) {
-		if(source.charAt(nextStart + 2) === "(") {
-			// Filter mode: (((filter))) or (((filter||sep)))
-			var match = /^\(\(\(([\s\S]+?)\)\)\)/.exec(source.substring(nextStart));
-			if(match) {
-				// Check for separator: split on last || before )))
-				var inner = match[1];
-				var sepIndex = inner.lastIndexOf("||");
-				if(sepIndex >= 0) {
-					this.nextMatch = {
-						type: "filter",
-						filter: inner.substring(0,sepIndex),
-						separator: inner.substring(sepIndex + 2),
-						start: nextStart,
-						end: nextStart + match[0].length
-					};
-				} else {
-					this.nextMatch = {
-						type: "filter",
-						filter: inner,
-						separator: ", ",
-						start: nextStart,
-						end: nextStart + match[0].length
-					};
-				}
-				return nextStart;
-			}
-		} else {
-			// Variable mode: ((varname)) or ((varname||sep))
-			var match = /^\(\(([^()|]+?)(?:\|\|([^)]*))?\)\)/.exec(source.substring(nextStart));
-			if(match) {
-				this.nextMatch = {
-					type: "variable",
-					varName: match[1],
-					separator: match[2] !== undefined ? match[2] : ", ",
-					start: nextStart,
-					end: nextStart + match[0].length
-				};
-				return nextStart;
-			}
-		}
-		nextStart += 2;
-	}
-	return undefined;
-};
-
-/*
-Parse the most recent match
-*/
-exports.parse = function() {
-	var match = this.nextMatch;
-	this.nextMatch = null;
-	this.parser.pos = match.end;
-	var filter, sep = match.separator;
-	if(match.type === "variable") {
-		filter = "[(" + match.varName + ")join[" + sep + "]]";
-	} else {
-		filter = match.filter + " +[join[" + sep + "]]";
-	}
-	return [{
-		type: "text",
-		attributes: {
-			text: {name: "text", type: "filtered", filter: filter}
-		},
-		orderedAttributes: [
-			{name: "text", type: "filtered", filter: filter}
-		]
-	}];
-};

core/modules/parsers/wikiparser/rules/transcludeblock.js

@@ -23,6 +23,27 @@ exports.init = function(parser) {
 	this.matchRegExp = /\{\{([^\{\}\|]*)(?:\|\|([^\|\{\}]+))?(?:\|([^\{\}]+))?\}\}(?:\r?\n|$)/mg;
 };
 
+/*
+Reject the match if we don't have a template or text reference
+*/
+exports.findNextMatch = function(startPos) {
+	this.matchRegExp.lastIndex = startPos;
+	this.match = this.matchRegExp.exec(this.parser.source);
+	if(this.match) {
+		var template = $tw.utils.trim(this.match[2]),
+			textRef = $tw.utils.trim(this.match[1]);
+		// Bail if we don't have a template or text reference
+		if(!template && !textRef) {
+			return undefined;
+		} else {
+			return this.match.index;
+		}
+	} else {
+		return undefined;
+	}
+	return this.match ? this.match.index : undefined;
+};
+
 exports.parse = function() {
 	// Move past the match
 	this.parser.pos = this.matchRegExp.lastIndex;

core/modules/parsers/wikiparser/rules/transcludeinline.js

@@ -23,6 +23,27 @@ exports.init = function(parser) {
 	this.matchRegExp = /\{\{([^\{\}\|]*)(?:\|\|([^\|\{\}]+))?(?:\|([^\{\}]+))?\}\}/mg;
 };
 
+/*
+Reject the match if we don't have a template or text reference
+*/
+exports.findNextMatch = function(startPos) {
+	this.matchRegExp.lastIndex = startPos;
+	this.match = this.matchRegExp.exec(this.parser.source);
+	if(this.match) {
+		var template = $tw.utils.trim(this.match[2]),
+			textRef = $tw.utils.trim(this.match[1]);
+		// Bail if we don't have a template or text reference
+		if(!template && !textRef) {
+			return undefined;
+		} else {
+			return this.match.index;
+		}
+	} else {
+		return undefined;
+	}
+	return this.match ? this.match.index : undefined;
+};
+
 exports.parse = function() {
 	// Move past the match
 	this.parser.pos = this.matchRegExp.lastIndex;

core/modules/savers/postmessage.js

@@ -6,7 +6,10 @@ module-type: saver
 Handles saving changes via window.postMessage() to the window.parent
 
 \*/
+(function(){
 
+/*jslint node: true, browser: true */
+/*global $tw: false */
 "use strict";
 
 /*
@@ -60,3 +63,4 @@ exports.create = function(wiki) {
 	return new PostMessageSaver(wiki);
 };
 
+})();

core/modules/startup/load-modules.js

@@ -13,11 +13,6 @@ Load core modules
 exports.name = "load-modules";
 exports.synchronous = true;
 
-// Set to `true` to enable performance instrumentation
-var PERFORMANCE_INSTRUMENTATION_CONFIG_TITLE = "$:/config/Performance/Instrumentation";
-
-var widget = require("$:/core/modules/widgets/widget.js");
-
 exports.startup = function() {
 	// Load modules
 	$tw.modules.applyMethods("utils",$tw.utils);
@@ -36,27 +31,6 @@ exports.startup = function() {
 	$tw.modules.applyMethods("tiddlerdeserializer",$tw.Wiki.tiddlerDeserializerModules);
 	$tw.macros = $tw.modules.getModulesByTypeAsHashmap("macro");
 	$tw.wiki.initParsers();
-	// --------------------------
-	// The rest of the startup process here is not strictly to do with loading modules, but are needed before other startup
-	// modules are executed. It is easier to put them here than to introduce a new startup module
-	// --------------------------
-	// Create a root widget for attaching event handlers. By using it as the parentWidget for another widget tree, one can reuse the event handlers
-	$tw.rootWidget = new widget.widget({
-		type: "widget",
-		children: []
-	},{
-		wiki: $tw.wiki,
-		document: $tw.browser ? document : $tw.fakeDocument
-	});
-	// Set up the performance framework
-	$tw.perf = new $tw.Performance($tw.wiki.getTiddlerText(PERFORMANCE_INSTRUMENTATION_CONFIG_TITLE,"no") === "yes");
-	// Kick off the filter tracker
-	$tw.filterTracker = new $tw.FilterTracker($tw.wiki);
-	$tw.wiki.addEventListener("change",function(changes) {
-		$tw.filterTracker.handleChangeEvent(changes);
-	});
-	// Kick off the background action dispatcher
-	$tw.backgroundActionDispatcher = new $tw.BackgroundActionDispatcher($tw.filterTracker,$tw.wiki);
 	if($tw.node) {
 		$tw.Commander.initCommands();
 	}

core/modules/startup/startup.js

@@ -14,6 +14,11 @@ exports.name = "startup";
 exports.after = ["load-modules"];
 exports.synchronous = true;
 
+// Set to `true` to enable performance instrumentation
+var PERFORMANCE_INSTRUMENTATION_CONFIG_TITLE = "$:/config/Performance/Instrumentation";
+
+var widget = require("$:/core/modules/widgets/widget.js");
+
 exports.startup = function() {
 	// Minimal browser detection
 	if($tw.browser) {
@@ -49,6 +54,16 @@ exports.startup = function() {
 	}
 	// Initialise version
 	$tw.version = $tw.utils.extractVersionInfo();
+	// Set up the performance framework
+	$tw.perf = new $tw.Performance($tw.wiki.getTiddlerText(PERFORMANCE_INSTRUMENTATION_CONFIG_TITLE,"no") === "yes");
+	// Create a root widget for attaching event handlers. By using it as the parentWidget for another widget tree, one can reuse the event handlers
+	$tw.rootWidget = new widget.widget({
+		type: "widget",
+		children: []
+	},{
+		wiki: $tw.wiki,
+		document: $tw.browser ? document : $tw.fakeDocument
+	});
 	// Kick off the language manager and switcher
 	$tw.language = new $tw.Language();
 	$tw.languageSwitcher = new $tw.PluginSwitcher({

core/modules/utils/errors.js

@@ -6,7 +6,6 @@ module-type: utils
 Custom errors for TiddlyWiki.
 
 \*/
-
 function TranscludeRecursionError() {
 	Error.apply(this,arguments);
 	this.signatures = Object.create(null);

core/modules/utils/messaging.js

@@ -8,7 +8,10 @@ Messaging utilities for use with window.postMessage() etc.
 This module intentionally has no dependencies so that it can be included in non-TiddlyWiki projects
 
 \*/
+(function(){
 
+/*jslint node: true, browser: true */
+/*global $tw: false */
 "use strict";
 
 var RESPONSE_TIMEOUT = 2 * 1000;
@@ -119,3 +122,5 @@ BrowserMessagingPublisher.prototype.close = function() {
 };
 
 exports.BrowserMessagingPublisher = BrowserMessagingPublisher;
+
+})();

core/modules/utils/utils.js

@@ -242,53 +242,6 @@ exports.slowInSlowOut = function(t) {
 	return (1 - ((Math.cos(t * Math.PI) + 1) / 2));
 };
 
-exports.copyObjectPropertiesSafe = function(object) {
-	const seen = new Set(),
-		isDOMElement = value => value instanceof Node || value instanceof Window;
-
-	function safeCopy(obj) {
-		// skip circular references
-		if(seen.has(obj)) {
-			return undefined;
-		}
-		// primitives and null are safe
-		if(typeof obj !== "object" || obj === null) {
-			return obj;
-		}
-		// skip DOM elements
-		if(isDOMElement(obj)) {
-			return undefined;
-		}
-		// copy arrays, preserving positions
-		if(Array.isArray(obj)) {
-			return obj.map(item => {
-				const value = safeCopy(item);
-				return value === undefined ? null : value;
-			});
-		}
-
-		seen.add(obj);
-		const copy = {};
-		let key,
-			value;
-		for(key in obj) {
-			try {
-				value = safeCopy(obj[key]);
-				if(value !== undefined) {
-					copy[key] = value;
-				}
-			} catch(e) {
-				// silently skip unserializable properties
-			}
-		}
-		return copy;
-	}
-
-	const result = safeCopy(object);
-	seen.clear();
-	return result;
-};
-
 exports.formatTitleString = function(template,options) {
 	var base = options.base || "",
 		separator = options.separator || "",

core/modules/widgets/browse.js

@@ -80,8 +80,8 @@ BrowseWidget.prototype.render = function(parent,nextSibling) {
 	});
 	// Insert element
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 /*

core/modules/widgets/button.js

@@ -135,8 +135,8 @@ ButtonWidget.prototype.render = function(parent,nextSibling) {
 	}
 	// Insert element
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 /*

core/modules/widgets/checkbox.js

@@ -64,8 +64,8 @@ CheckboxWidget.prototype.render = function(parent,nextSibling) {
 	]);
 	// Insert the label into the DOM and render any children
 	parent.insertBefore(this.labelDomNode,nextSibling);
-	this.domNodes.push(this.labelDomNode);
 	this.renderChildren(this.spanDomNode,null);
+	this.domNodes.push(this.labelDomNode);
 };
 
 CheckboxWidget.prototype.getValue = function() {

core/modules/widgets/diff-text.js

@@ -59,8 +59,6 @@ DiffTextWidget.prototype.render = function(parent,nextSibling) {
 	var domContainer = this.document.createElement("div"), 
 		domDiff = this.createDiffDom(diffs);
 	parent.insertBefore(domContainer,nextSibling);
-	// Save our container
-	this.domNodes.push(domContainer);
 	// Set variables
 	this.setVariable("diff-count",diffs.reduce(function(acc,diff) {
 		if(diff[0] !== dmp.DIFF_EQUAL) {
@@ -72,6 +70,8 @@ DiffTextWidget.prototype.render = function(parent,nextSibling) {
 	this.renderChildren(domContainer,null);
 	// Render the diff
 	domContainer.appendChild(domDiff);
+	// Save our container
+	this.domNodes.push(domContainer);
 };
 
 /*

core/modules/widgets/draggable.js

@@ -56,7 +56,6 @@ DraggableWidget.prototype.render = function(parent,nextSibling) {
 	});
 	// Insert the node into the DOM and render any children
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
 	// Add event handlers
 	if(this.dragEnable) {
@@ -71,6 +70,7 @@ DraggableWidget.prototype.render = function(parent,nextSibling) {
 			selector: self.dragHandleSelector
 		});
 	}
+	this.domNodes.push(domNode);
 };
 
 /*

core/modules/widgets/droppable.js

@@ -57,8 +57,8 @@ DroppableWidget.prototype.render = function(parent,nextSibling) {
 	}
 	// Insert element
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 	// Stack of outstanding enter/leave events
 	this.currentlyEntered = [];
 };

core/modules/widgets/element.js

@@ -77,8 +77,8 @@ ElementWidget.prototype.render = function(parent,nextSibling) {
 	// Allow hooks to manipulate the DOM node. Eg: Add debug info
 	$tw.hooks.invokeHook("th-dom-rendering-element", domNode, this);
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 /*

core/modules/widgets/eventcatcher.js

@@ -39,12 +39,75 @@ EventWidget.prototype.render = function(parent,nextSibling) {
 	this.domNode = domNode;
 	// Assign classes
 	this.assignDomNodeClasses();
-	// Add our event handlers
-	this.toggleListeners();
+	// Add our event handler
+	$tw.utils.each(this.types,function(type) {
+		domNode.addEventListener(type,function(event) {
+			var selector = self.getAttribute("selector"),
+				matchSelector = self.getAttribute("matchSelector"),
+				actions = self.getAttribute("$"+type),
+				stopPropagation = self.getAttribute("stopPropagation","onaction"),
+				selectedNode = event.target,
+				selectedNodeRect,
+				catcherNodeRect,
+				variables = {};
+			// Firefox can fire dragover and dragenter events on text nodes instead of their parents
+			if(selectedNode.nodeType === 3) {
+				selectedNode = selectedNode.parentNode;
+			}
+			// Check that the selected node matches any matchSelector
+			if(matchSelector && !$tw.utils.domMatchesSelector(selectedNode,matchSelector)) {
+				return false;
+			}
+			if(selector) {
+				// Search ancestors for a node that matches the selector
+				while(!$tw.utils.domMatchesSelector(selectedNode,selector) && selectedNode !== domNode) {
+					selectedNode = selectedNode.parentNode;
+				}
+				// Exit if we didn't find one
+				if(selectedNode === domNode) {
+					return false;
+				}
+				// Only set up variables if we have actions to invoke
+				if(actions) {
+					variables = $tw.utils.collectDOMVariables(selectedNode,self.domNode,event);
+				}
+			}
+			// Execute our actions with the variables
+			if(actions) {
+				// Add a variable for the modifier key
+				variables.modifier = $tw.keyboardManager.getEventModifierKeyDescriptor(event);
+				// Add a variable for the mouse button
+				if("button" in event) {
+					if(event.button === 0) {
+						variables["event-mousebutton"] = "left";
+					} else if(event.button === 1) {
+						variables["event-mousebutton"] = "middle";
+					} else if(event.button === 2) {
+						variables["event-mousebutton"] = "right";
+					}
+				}
+				variables["event-type"] = event.type.toString();
+				if(typeof event.detail === "object" && !!event.detail) {
+					$tw.utils.each(event.detail,function(detailValue,detail) {
+						variables["event-detail-" + detail] = detailValue.toString();
+					});
+				} else if(!!event.detail) {
+					variables["event-detail"] = event.detail.toString();
+				}
+				self.invokeActionString(actions,self,event,variables);
+			}
+			if((actions && stopPropagation === "onaction") || stopPropagation === "always") {
+				event.preventDefault();
+				event.stopPropagation();
+				return true;
+			}
+			return false;
+		},false);
+	});
 	// Insert element
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 /*
@@ -59,232 +122,11 @@ EventWidget.prototype.execute = function() {
 			self.types.push(key.slice(1));
 		}
 	});
-	this.pointerCaptureMode = this.getAttribute("pointerCapture","no");
 	this.elementTag = this.getAttribute("tag");
 	// Make child widgets
 	this.makeChildWidgets();
 };
 
-/*
-Cache and pre-create all event listeners, called when first needed
-*/
-EventWidget.prototype.cacheEventListeners = function() {
-	if(this._eventListeners) {
-		return;
-	}
-	this._eventListeners = Object.create(null);
-	this._captureActiveListeners = Object.create(null);
-	this._dynamicOnlyEvents = ["pointerup","pointercancel","pointermove"];
-
-	const clearPointerCapture = event => {
-		if(Number.isInteger(this._capturePointerId)) {
-			this.stopPointerCapture(this._capturePointerId);
-		}
-	};
-
-	const attachDynamicOnlyListeners = () => {
-		this._dynamicOnlyEvents.forEach(dt => {
-			const listener = this._eventListeners[dt];
-			if(listener) {
-				this._captureActiveListeners[dt] = listener;
-				this.domNode.addEventListener(dt, listener, false);
-			}
-		});
-	};
-
-	// Dynamic pointer capture listeners
-	if(this.pointerCaptureMode === "dynamic") {
-		["pointerup","pointercancel"].forEach(type => {
-			this._eventListeners[type] = event => {
-				const selectedNode = this.checkEvent(event, type);
-				if(selectedNode) {
-					clearPointerCapture(event);
-				}
-				// Remove dynamic-only listeners
-				this.cleanupDynamicListeners();
-				return this.handleEvent(event, type, selectedNode);
-			};
-		});
-		if(!this.types.includes("pointerdown")) {
-			this.types.push("pointerdown");
-		}
-	}
-
-	// Create any listeners not already defined above
-	this.types.forEach(type => {
-		if(!this._eventListeners[type]) {
-			this._eventListeners[type] = event => {
-				const selectedNode = this.checkEvent(event, type);
-				if(!selectedNode) {
-					return false;
-				}
-				// Handle pointer capture for pointerdown
-				if(type === "pointerdown") {
-					if(this.pointerCaptureMode !== "no") {
-						this.startPointerCapture(event.pointerId, event.target);
-					}
-
-					if(this.pointerCaptureMode === "dynamic") {
-						attachDynamicOnlyListeners();
-					}
-				} else if(type === "pointerup" || type === "pointercancel") {
-					clearPointerCapture(event);
-				}
-				return this.handleEvent(event, type, selectedNode);
-			};
-		}
-	});
-};
-
-/*
-Check if an event qualifies and return the matching selected node
-*/
-EventWidget.prototype.checkEvent = function(event, type) {
-	const domNode = this.domNode;
-	let node = event.target;
-
-	// Use capture target if valid
-	if(this._captureTarget && event.pointerId !== undefined) {
-		if(document.contains(this._captureTarget)) {
-			node = this._captureTarget;
-		} else {
-			// Clear stale reference
-			this.stopPointerCapture(this._capturePointerId);
-			node = event.target;
-		}
-	}
-
-	if(node && node.nodeType === 3) {
-		node = node.parentNode;
-	}
-	if(!node || node.nodeType !== 1) {
-		return null;
-	}
-
-	const selector = this.getAttribute("selector"),
-		matchSelector = this.getAttribute("matchSelector");
-
-	if(matchSelector && !node.matches(matchSelector)) {
-		return null;
-	}
-	if(selector) {
-		const match = node.closest(selector);
-		if(!match || match === domNode || !domNode.contains(match)) {
-			return null;
-		}
-		return match;
-	}
-	return node;
-};
-
-/*
-Handle the event and execute actions
-*/
-EventWidget.prototype.handleEvent = function(event, type, selectedNode) {
-	if(!selectedNode) {
-		return false;
-	}
-	let actions = this.getAttribute("$"+type),
-		stopPropagation = this.getAttribute("stopPropagation","onaction");
-
-	if(actions) {
-		let variables = $tw.utils.extend(
-			{},
-			$tw.utils.collectDOMVariables(selectedNode, this.domNode, event),
-			{
-				"eventJSON": JSON.stringify($tw.utils.copyObjectPropertiesSafe(event)),
-				"modifier": $tw.keyboardManager.getEventModifierKeyDescriptor(event),
-				"event-type": event.type.toString()
-			}
-		);
-
-		if("button" in event) {
-			const mouseButtonMap = {0:"left",1:"middle",2:"right"};
-			variables["event-mousebutton"] = mouseButtonMap[event.button];
-		}
-		this.invokeActionString(actions, this, event, variables);
-	}
-
-	if((actions && stopPropagation === "onaction") || stopPropagation === "always") {
-		event.preventDefault();
-		event.stopPropagation();
-		return true;
-	}
-	return false;
-};
-
-EventWidget.prototype.startPointerCapture = function(pointerId, captureTarget) {
-	// Start capture only if none active; pointerId can be 0 
-	if(!Number.isInteger(this._capturePointerId) && this.domNode && this.domNode.setPointerCapture) {
-		this.domNode.setPointerCapture(pointerId);
-		this._capturePointerId = pointerId;
-		this._captureTarget = captureTarget;
-	}
-};
-
-EventWidget.prototype.stopPointerCapture = function(pointerId) {
-	if(this.domNode && this.domNode.hasPointerCapture && this.domNode.hasPointerCapture(pointerId)) {
-		this.domNode.releasePointerCapture(pointerId);
-	}
-	this._capturePointerId = undefined;
-	this._captureTarget = undefined;
-};
-
-/*
-Attach all relevant listeners
-*/
-EventWidget.prototype.attachListeners = function() {
-	this.cacheEventListeners();
-	const domNode = this.domNode;
-	Object.keys(this._eventListeners).forEach(type => {
-		if(this.pointerCaptureMode === "dynamic" && this._dynamicOnlyEvents.includes(type)) {
-			return; //skip dynamic-only events
-		}
-		domNode.addEventListener(type, this._eventListeners[type], false);
-	});
-};
-
-/*
-Remove dynamic active listeners
-*/
-EventWidget.prototype.cleanupDynamicListeners = function() {
-	const domNode = this.domNode;
-	Object.keys(this._captureActiveListeners || {}).forEach(type => {
-		domNode.removeEventListener(type, this._captureActiveListeners[type], false);
-	});
-	this._captureActiveListeners = Object.create(null);
-};
-
-/*
-Remove all listeners
-*/
-EventWidget.prototype.removeAllListeners = function() {
-	if(Number.isInteger(this._capturePointerId)) {
-		this.stopPointerCapture(this._capturePointerId);
-	}
-	const domNode = this.domNode;
-	Object.keys(this._eventListeners || {}).forEach(type => {
-		domNode.removeEventListener(type, this._eventListeners[type], false);
-	});
-	this.cleanupDynamicListeners();
-	this._captureTarget = null;
-};
-
-/*
-Enable or disable listeners
-*/
-EventWidget.prototype.toggleListeners = function() {
-	let disabled = this.getAttribute("disabled","no") === "yes";
-	if(disabled) {
-		this.removeAllListeners();
-	} else {
-		this.attachListeners();
-	}
-};
-
-/*
-Assign DOM node classes
-*/
 EventWidget.prototype.assignDomNodeClasses = function() {
 	var classes = this.getAttribute("class","").split(" ");
 	classes.push("tc-eventcatcher");
@@ -292,23 +134,18 @@ EventWidget.prototype.assignDomNodeClasses = function() {
 };
 
 /*
-Refresh widget
+Selectively refreshes the widget if needed. Returns true if the widget or any of its children needed re-rendering
 */
 EventWidget.prototype.refresh = function(changedTiddlers) {
-	const changedAttributes = this.computeAttributes(),
-		changedKeys = Object.keys(changedAttributes),
-		canUpdateAttributes = changedKeys.every(key => key === "class" || key === "disabled");
-	if(canUpdateAttributes) {
-		if(changedAttributes["class"]) {
-			this.assignDomNodeClasses();
-		}
-		if(changedAttributes["disabled"]) {
-			this.toggleListeners();
-		}
-		return this.refreshChildren(changedTiddlers);
+	var changedAttributes = this.computeAttributes(),
+		changedAttributesCount = $tw.utils.count(changedAttributes);
+	if(changedAttributesCount === 1 && changedAttributes["class"]) {
+		this.assignDomNodeClasses();
+	} else if(changedAttributesCount > 0) {
+		this.refreshSelf();
+		return true;
 	}
-	this.refreshSelf();
-	return true;
+	return this.refreshChildren(changedTiddlers);
 };
 
 exports.eventcatcher = EventWidget;

core/modules/widgets/image.js

@@ -45,7 +45,7 @@ ImageWidget.prototype.render = function(parent,nextSibling) {
 	this.execute();
 	// Create element
 	// Determine what type of image it is
-	var tag = "img", src = "", self = this,
+	var tag = "img", src = "",
 		tiddler = this.wiki.getTiddler(this.imageSource);
 	if(!tiddler) {
 		// The source isn't the title of a tiddler, so we'll assume it's a URL
@@ -115,21 +115,11 @@ ImageWidget.prototype.render = function(parent,nextSibling) {
 	if(this.lazyLoading && tag === "img") {
 		domNode.setAttribute("loading",this.lazyLoading);
 	}
-	this.assignAttributes(domNode,{
-		sourcePrefix: "data-",
-		destPrefix: "data-"
-	});
 	// Add classes when the image loads or fails
 	$tw.utils.addClass(domNode,"tc-image-loading");
-	domNode.addEventListener("load",function(event) {
+	domNode.addEventListener("load",function() {
 		$tw.utils.removeClass(domNode,"tc-image-loading");
 		$tw.utils.addClass(domNode,"tc-image-loaded");
-		if(self.loadedActions) {
-			var variables = $tw.utils.collectDOMVariables(domNode,null,event);
-			variables["img-natural-width"] = domNode.naturalWidth.toString();
-			variables["img-natural-height"] = domNode.naturalHeight.toString();
-			self.invokeActionString(self.loadedActions,self,event,variables);		
-		}
 	},false);
 	domNode.addEventListener("error",function() {
 		$tw.utils.removeClass(domNode,"tc-image-loading");
@@ -153,31 +143,17 @@ ImageWidget.prototype.execute = function() {
 	this.imageTooltip = this.getAttribute("tooltip");
 	this.imageAlt = this.getAttribute("alt");
 	this.lazyLoading = this.getAttribute("loading");
-	this.loadedActions = this.getAttribute("loadActions");
 };
 
 /*
 Selectively refreshes the widget if needed. Returns true if the widget or any of its children needed re-rendering
 */
 ImageWidget.prototype.refresh = function(changedTiddlers) {
-	var changedAttributes = this.computeAttributes(),
-		hasChangedAttributes = $tw.utils.count(changedAttributes) > 0;
-	if(changedAttributes.source || changedAttributes["class"] || changedAttributes.usemap || changedAttributes.tooltip || changedTiddlers[this.imageSource] ||changedAttributes.loadActions) {
+	var changedAttributes = this.computeAttributes();
+	if(changedAttributes.source || changedAttributes.width || changedAttributes.height || changedAttributes["class"] || changedAttributes.usemap || changedAttributes.tooltip || changedTiddlers[this.imageSource]) {
 		this.refreshSelf();
 		return true;
-	} else if(hasChangedAttributes) {
-		this.assignAttributes(this.domNodes[0],{
-			sourcePrefix: "data-",
-			destPrefix: "data-"
-		});
-		if(changedAttributes.width) {
-			this.domNodes[0].setAttribute("width",this.getAttribute("width"));
-		}
-		if(changedAttributes.height) {
-			this.domNodes[0].setAttribute("height",this.getAttribute("height"));
-		}
-	}
-	else {
+	} else {
 		return false;
 	}
 };

core/modules/widgets/keyboard.js

@@ -45,8 +45,8 @@ KeyboardWidget.prototype.render = function(parent,nextSibling) {
 	]);
 	// Insert element
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 KeyboardWidget.prototype.handleChangeEvent = function(event) {

core/modules/widgets/link.js

@@ -50,8 +50,8 @@ LinkWidget.prototype.render = function(parent,nextSibling) {
 			destPrefix: "aria-"
 		});
 		parent.insertBefore(domNode,nextSibling);
-		this.domNodes.push(domNode);
 		this.renderChildren(domNode,null);
+		this.domNodes.push(domNode);
 	}
 };
 
@@ -86,7 +86,7 @@ LinkWidget.prototype.renderLink = function(parent,nextSibling) {
 			classes.push(this.linkClasses);
 		}
 	} else if(this.overrideClasses !== "") {
-		classes.push(this.overrideClasses);
+		classes.push(this.overrideClasses)
 	}
 	if(classes.length > 0) {
 		domNode.setAttribute("class",classes.join(" "));
@@ -97,7 +97,7 @@ LinkWidget.prototype.renderLink = function(parent,nextSibling) {
 	if(wikilinkTransformFilter) {
 		// Use the filter to construct the href
 		wikiLinkText = this.wiki.filterTiddlers(wikilinkTransformFilter,this,function(iterator) {
-			iterator(self.wiki.getTiddler(self.to),self.to);
+			iterator(self.wiki.getTiddler(self.to),self.to)
 		})[0];
 	} else {
 		// Expand the tv-wikilink-template variable to construct the href
@@ -121,12 +121,12 @@ LinkWidget.prototype.renderLink = function(parent,nextSibling) {
 	var tooltipWikiText = this.tooltip || this.getVariable("tv-wikilink-tooltip");
 	if(tooltipWikiText) {
 		var tooltipText = this.wiki.renderText("text/plain","text/vnd.tiddlywiki",tooltipWikiText,{
-			parseAsInline: true,
-			variables: {
-				currentTiddler: this.to
-			},
-			parentWidget: this
-		});
+				parseAsInline: true,
+				variables: {
+					currentTiddler: this.to
+				},
+				parentWidget: this
+			});
 		domNode.setAttribute("title",tooltipText);
 	}
 	if(this.role) {
@@ -135,7 +135,7 @@ LinkWidget.prototype.renderLink = function(parent,nextSibling) {
 	this.assignAttributes(domNode,{
 		sourcePrefix: "aria-",
 		destPrefix: "aria-"
-	});
+	})
 	// Add a click event handler
 	$tw.utils.addEventListeners(domNode,[
 		{name: "click", handlerObject: this, handlerMethod: "handleClickEvent"},
@@ -145,8 +145,6 @@ LinkWidget.prototype.renderLink = function(parent,nextSibling) {
 		$tw.utils.makeDraggable({
 			domNode: domNode,
 			dragTiddlerFn: function() {return self.to;},
-			startActions: self.startActions,
-			endActions: self.endActions,
 			widget: this
 		});
 	} else if(this.draggable === "no") {
@@ -159,8 +157,8 @@ LinkWidget.prototype.renderLink = function(parent,nextSibling) {
 	});
 	// Insert the link into the DOM and render any children
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 LinkWidget.prototype.handleClickEvent = function(event) {
@@ -205,8 +203,6 @@ LinkWidget.prototype.execute = function() {
 	this.overrideClasses = this.getAttribute("overrideClass");
 	this.tabIndex = this.getAttribute("tabindex");
 	this.draggable = this.getAttribute("draggable","yes");
-	this.startActions = this.getAttribute("startactions");
-	this.endActions = this.getAttribute("endactions");
 	this.linkTag = this.getAttribute("tag","a");
 	// Determine the link characteristics
 	this.isMissing = !this.wiki.tiddlerExists(this.to);

core/modules/widgets/parameters.js

@@ -61,9 +61,7 @@ ParametersWidget.prototype.execute = function() {
 			if(name.substr(0,2) === "$$") {
 				name = name.substr(1);
 			}
-			var defaultValue = (self.multiValuedAttributes && self.multiValuedAttributes[attr.name])
-					|| self.getAttribute(attr.name,"");
-			var value = pointer.getTransclusionParameter(name,index,defaultValue);
+			var value = pointer.getTransclusionParameter(name,index,self.getAttribute(attr.name,""));
 			self.setVariable(name,value);
 		});
 		// Assign any metaparameters
@@ -82,8 +80,7 @@ ParametersWidget.prototype.execute = function() {
 			if(name.substr(0,2) === "$$") {
 				name = name.substr(1);
 			}
-			var value = (self.multiValuedAttributes && self.multiValuedAttributes[attr.name])
-					|| self.getAttribute(attr.name,"");
+			var value = self.getAttribute(attr.name,"");
 			self.setVariable(name,value);
 		});
 	}

core/modules/widgets/password.js

@@ -42,8 +42,8 @@ PasswordWidget.prototype.render = function(parent,nextSibling) {
 	]);
 	// Insert the label into the DOM and render any children
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 };
 
 PasswordWidget.prototype.handleChangeEvent = function(event) {

core/modules/widgets/radio.js

@@ -59,8 +59,8 @@ RadioWidget.prototype.render = function(parent,nextSibling) {
 	]);
 	// Insert the label into the DOM and render any children
 	parent.insertBefore(this.labelDomNode,nextSibling);
-	this.domNodes.push(this.labelDomNode);
 	this.renderChildren(this.spanDomNode,null);
+	this.domNodes.push(this.labelDomNode);
 };
 
 RadioWidget.prototype.getValue = function() {

core/modules/widgets/reveal.js

@@ -40,7 +40,6 @@ RevealWidget.prototype.render = function(parent,nextSibling) {
 		domNode.setAttribute("style",this.style);
 	}
 	parent.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
 	if(!domNode.isTiddlyWikiFakeDom && this.type === "popup" && this.isOpen) {
 		this.positionPopup(domNode);
@@ -49,6 +48,7 @@ RevealWidget.prototype.render = function(parent,nextSibling) {
 	if(!this.isOpen) {
 		domNode.setAttribute("hidden","true");
 	}
+	this.domNodes.push(domNode);
 };
 
 RevealWidget.prototype.positionPopup = function(domNode) {

core/modules/widgets/scrollable.js

@@ -168,8 +168,8 @@ ScrollableWidget.prototype.render = function(parent,nextSibling) {
 	this.outerDomNode.className = this["class"] || "";
 	// Insert element
 	parent.insertBefore(this.outerDomNode,nextSibling);
-	this.domNodes.push(this.outerDomNode);
 	this.renderChildren(this.innerDomNode,null);
+	this.domNodes.push(this.outerDomNode);
 	// If the scroll position is bound to a tiddler
 	if(this.scrollableBind) {
 		// After a delay for rendering, scroll to the bound position

core/modules/widgets/select.js

@@ -63,8 +63,8 @@ SelectWidget.prototype.render = function(parent,nextSibling) {
 		domNode.setAttribute("title",this.selectTooltip);
 	}
 	this.parentDomNode.insertBefore(domNode,nextSibling);
-	this.domNodes.push(domNode);
 	this.renderChildren(domNode,null);
+	this.domNodes.push(domNode);
 	this.setSelectValue();
 	if(this.selectFocus == "yes") {
 		this.getSelectDomNode().focus();

core/modules/widgets/transclude.js

@@ -32,26 +32,16 @@ TranscludeWidget.prototype.render = function(parent,nextSibling) {
 	} catch(error) {
 		if(error instanceof $tw.utils.TranscludeRecursionError) {
 			// We were infinite looping.
-			// We need to try and abort as much of the loop as we
-			// can, so we will keep "throwing" upward until we find
-			// a transclusion that has a different signature.
-			// Hopefully that will land us just outside where the
-			// loop began. That's where we want to issue an error.
-			// Rendering widgets beneath this point may result in a
-			// freezing browser if they explode exponentially.
+			// We need to try and abort as much of the loop as we can, so we will keep "throwing" upward until we find a transclusion that has a different signature.
+			// Hopefully that will land us just outside where the loop began. That's where we want to issue an error.
+			// Rendering widgets beneath this point may result in a freezing browser if they explode exponentially.
 			var transcludeSignature = this.getVariable("transclusion");
 			if(this.getAncestorCount() > $tw.utils.TranscludeRecursionError.MAX_WIDGET_TREE_DEPTH - 50) {
-				// For the first fifty transcludes we climb up,
-				// we simply collect signatures.
-				// We're assuming those first 50 will likely
-				// include all transcludes involved in the loop.
+				// For the first fifty transcludes we climb up, we simply collect signatures.
+				// We're assuming that those first 50 will likely include all transcludes involved in the loop.
 				error.signatures[transcludeSignature] = true;
 			} else if(!error.signatures[transcludeSignature]) {
-				// Now that we're past the first 50, look for
-				// the first signature that wasn't in that loop.
-				// That's where we print the error and resume
-				// rendering.
-				this.removeChildDomNodes();
+				// Now that we're past the first 50, let's look for the first signature that wasn't in the loop. That'll be where we print the error and resume rendering.
 				this.children = [this.makeChildWidget({type: "error", attributes: {
 					"$message": {type: "string", value: $tw.language.getString("Error/RecursiveTransclusion")}
 				}})];
@@ -158,10 +148,8 @@ Collect string parameters
 TranscludeWidget.prototype.collectStringParameters = function() {
 	var self = this;
 	this.stringParametersByName = Object.create(null);
-	this.multiValuedParametersByName = Object.create(null);
 	if(!this.legacyMode) {
 		$tw.utils.each(this.attributes,function(value,name) {
-			var attrName = name; // Save original attribute name for MVV lookup
 			if(name.charAt(0) === "$") {
 				if(name.charAt(1) === "$") {
 					// Attributes starting $$ represent parameters starting with a single $
@@ -172,9 +160,6 @@ TranscludeWidget.prototype.collectStringParameters = function() {
 				}
 			}
 			self.stringParametersByName[name] = value;
-			if(self.multiValuedAttributes && self.multiValuedAttributes[attrName]) {
-				self.multiValuedParametersByName[name] = self.multiValuedAttributes[attrName];
-			}
 		});
 	}
 };
@@ -318,16 +303,7 @@ TranscludeWidget.prototype.parseTransclusionTarget = function(parseAsInline) {
 							if(name.charAt(0) === "$") {
 								name = "$" + name;
 							}
-							if(param.defaultType === "multivalue-variable") {
-								// Construct MVV attribute for the default
-								var mvvNode = {type: "transclude", isMVV: true, attributes: {}, orderedAttributes: []};
-								$tw.utils.addAttributeToParseTreeNode(mvvNode,"$variable",param.defaultVariable);
-								$tw.utils.addAttributeToParseTreeNode(parser.tree[0],{
-									name: name, type: "macro", isMVV: true, value: mvvNode
-								});
-							} else {
-								$tw.utils.addAttributeToParseTreeNode(parser.tree[0],name,param["default"]);
-							}
+							$tw.utils.addAttributeToParseTreeNode(parser.tree[0],name,param["default"])
 						});
 					} else if(srcVariable && !srcVariable.isFunctionDefinition) {
 						// For macros and ordinary variables, wrap the parse tree in a vars widget assigning the parameters to variables named "__paramname__"
@@ -378,11 +354,7 @@ TranscludeWidget.prototype.getOrderedTransclusionParameters = function() {
 	// Collect the parameters
 	for(var name in this.stringParametersByName) {
 		var value = this.stringParametersByName[name];
-		var param = {name: name, value: value};
-		if(this.multiValuedParametersByName[name]) {
-			param.multiValue = this.multiValuedParametersByName[name];
-		}
-		result.push(param);
+		result.push({name: name, value: value});
 	}
 	// Sort numerical parameter names first
 	result.sort(function(a,b) {
@@ -412,16 +384,10 @@ Fetch the value of a parameter
 */
 TranscludeWidget.prototype.getTransclusionParameter = function(name,index,defaultValue) {
 	if(name in this.stringParametersByName) {
-		if(this.multiValuedParametersByName[name]) {
-			return this.multiValuedParametersByName[name];
-		}
 		return this.stringParametersByName[name];
 	} else {
 		var name = "" + index;
 		if(name in this.stringParametersByName) {
-			if(this.multiValuedParametersByName[name]) {
-				return this.multiValuedParametersByName[name];
-			}
 			return this.stringParametersByName[name];
 		}
 	}

core/modules/widgets/widget.js

@@ -151,7 +151,7 @@ Widget.prototype.getVariableInfo = function(name,options) {
 		} else if(variable.isFunctionDefinition) {
 			// Function evaluations
 			params = self.resolveVariableParameters(variable.params,actualParams);
-			var variables = $tw.utils.extend({},options.variables);
+			var variables = options.variables || Object.create(null);
 			// Apply default parameter values
 			$tw.utils.each(variable.params,function(param,index) {
 				if(param["default"]) {
@@ -160,7 +160,7 @@ Widget.prototype.getVariableInfo = function(name,options) {
 			});
 			// Parameters are an array of {name:, value:, multivalue:} pairs (name and multivalue are optional)
 			$tw.utils.each(params,function(param) {
-				if(param.multiValue && param.multiValue.length) {
+				if(param.multiValue) {
 					variables[param.name] = param.multiValue;
 				} else {
 					variables[param.name] = param.value || "";
@@ -233,10 +233,8 @@ Widget.prototype.resolveVariableParameters = function(formalParams,actualParams)
 			paramMultiValue = typeof param === "string" ? [param] : (param.multiValue || [paramValue]);
 		}
 		// If we've still not got a value, use the default, if any
-		if(!paramValue) {
-			paramValue = paramInfo["default"] || "";
-			paramMultiValue = [paramValue];
-		}
+		paramValue = paramValue || paramInfo["default"] || "";
+		paramMultiValue = paramMultiValue || [paramValue];
 		// Store the parameter name and value
 		results.push({name: paramInfo.name, value: paramValue, multiValue: paramMultiValue});
 	}
@@ -343,7 +341,7 @@ Widget.prototype.makeFakeWidgetWithVariables = function(variables) {
 				}
 			} else {
 				opts = opts || {};
-				opts.variables = $tw.utils.extend({},variables,opts.variables);
+				opts.variables = variables;
 				return self.getVariable(name,opts);
 			};
 		},
@@ -381,31 +379,19 @@ filterFn: only include attributes where filterFn(name) returns true
 Widget.prototype.computeAttributes = function(options) {
 	options = options || {};
 	var changedAttributes = {},
-		self = this,
-		newMultiValuedAttributes = Object.create(null);
+		self = this;
 	$tw.utils.each(this.parseTreeNode.attributes,function(attribute,name) {
 		if(options.filterFn) {
 			if(!options.filterFn(name)) {
 				return;
 			}
 		}
-		var value = self.computeAttribute(attribute),
-			multiValue = null;
-		if($tw.utils.isArray(value)) {
-			multiValue = value;
-			newMultiValuedAttributes[name] = multiValue;
-			value = value[0] || "";
-		}
-		var changed = (self.attributes[name] !== value);
-		if(!changed && multiValue && self.multiValuedAttributes) {
-			changed = !$tw.utils.isArrayEqual(self.multiValuedAttributes[name] || [], multiValue);
-		}
-		if(changed) {
+		var value = self.computeAttribute(attribute);
+		if(self.attributes[name] !== value) {
 			self.attributes[name] = value;
 			changedAttributes[name] = true;
 		}
 	});
-	this.multiValuedAttributes = newMultiValuedAttributes;
 	return changedAttributes;
 };
 
@@ -443,7 +429,7 @@ Widget.prototype.computeAttribute = function(attribute,options) {
 		});
 		// Invoke the macro
 		var variableInfo = this.getVariableInfo(macroName,{params: params});
-		if(options.asList || attribute.isMVV) {
+		if(options.asList) {
 			value = variableInfo.resultList;
 		} else {
 			value = variableInfo.text;
@@ -799,9 +785,9 @@ Widget.prototype.findNextSiblingDomNode = function(startIndex) {
 	// Refer to this widget by its index within its parents children
 	var parent = this.parentWidget,
 		index = startIndex !== undefined ? startIndex : parent.children.indexOf(this);
-	if(index === -1) {
-		throw "node not found in parents children";
-	}
+if(index === -1) {
+	throw "node not found in parents children";
+}
 	// Look for a DOM node in the later siblings
 	while(++index < parent.children.length) {
 		var domNode = parent.children[index].findFirstDomNode();
@@ -839,60 +825,21 @@ Widget.prototype.findFirstDomNode = function() {
 };
 
 /*
-Entry into destroy procedure
-options include:
-	removeDOMNodes: boolean (default true)
-*/
-Widget.prototype.destroyChildren = function(options) {
-	$tw.utils.each(this.children,function(childWidget) {
-		childWidget.destroy(options);
-	});
-};
-
-/*
-Legacy entry into destroy procedure
+Remove any DOM nodes created by this widget or its children
 */
 Widget.prototype.removeChildDomNodes = function() {
-	this.destroy({removeDOMNodes: true});
-};
-
-/*
-Default destroy
-options include:
-- removeDOMNodes: boolean (default true)
-*/
-Widget.prototype.destroy = function(options) {
-	const { removeDOMNodes = true } = options || {};
-	let removeChildDOMNodes = removeDOMNodes;
-	if(removeDOMNodes && this.domNodes.length > 0) {
-		// If this widget will remove its own DOM nodes, children should not remove theirs
-		removeChildDOMNodes = false;
-	}
-	// Destroy children first
-	this.destroyChildren({removeDOMNodes: removeChildDOMNodes});
-	this.children = [];
-	
-	// Call custom cleanup method if implemented
-	if(typeof this.onDestroy === "function") {
-		this.onDestroy();
-	}
-	
-	// Remove our DOM nodes if needed
-	if(removeDOMNodes) {
-		this.removeLocalDomNodes();	
-	}
-};
-
-/*
-Remove any DOM nodes created by this widget 
-*/
-Widget.prototype.removeLocalDomNodes = function() {
-	for(const domNode of this.domNodes) {
-		if(domNode.parentNode) {
+	// If this widget has directly created DOM nodes, delete them and exit. This assumes that any child widgets are contained within the created DOM nodes, which would normally be the case
+	if(this.domNodes.length > 0) {
+		$tw.utils.each(this.domNodes,function(domNode) {
 			domNode.parentNode.removeChild(domNode);
-		}
+		});
+		this.domNodes = [];
+	} else {
+		// Otherwise, ask the child widgets to delete their DOM nodes
+		$tw.utils.each(this.children,function(childWidget) {
+			childWidget.removeChildDomNodes();
+		});
 	}
-	this.domNodes = [];
 };
 
 /*

core/modules/wiki.js

@@ -1122,16 +1122,15 @@ Parse a block of text of a specified MIME type
 Options include:
 	substitutions: an optional array of substitutions
 */
-exports.getSubstitutedText = function(text,thisWidget,options) {
+exports.getSubstitutedText = function(text,widget,options) {
 	options = options || {};
 	text = text || "";
 	var self = this,
-		widgetClass = widget.widget,
 		substitutions = options.substitutions || [],
 		output;
 	// Evaluate embedded filters and substitute with first result
 	output = text.replace(/\$\{([\S\s]+?)\}\$/g, function(match,filter) {
-		return self.filterTiddlers(filter,thisWidget)[0] || "";
+		return self.filterTiddlers(filter,widget)[0] || "";
 	});
 	// Process any substitutions provided in options
 	$tw.utils.each(substitutions,function(substitute) {
@@ -1139,7 +1138,7 @@ exports.getSubstitutedText = function(text,thisWidget,options) {
 	});
 	// Substitute any variable references with their values
 	return output.replace(/\$\((.+?)\)\$/g, function(match,varname) {
-		return widgetClass.evaluateVariable(thisWidget,varname, {defaultValue: ""})[0];
+		return widget.getVariable(varname,{defaultValue: ""});
 	});
 };
 
@@ -1664,14 +1663,12 @@ exports.addToStory = function(title,fromTitle,storyTitle,options) {
 Generate a title for the draft of a given tiddler
 */
 exports.generateDraftTitle = function(title) {
-	let c = 0,
-		draftTitle;
-	const username = this.getTiddlerText("$:/status/UserName");
+	var c = 0,
+		draftTitle,
+		username = this.getTiddlerText("$:/status/UserName"),
+		attribution = username ? " by " + username : "";
 	do {
-		draftTitle = username ? $tw.language.getString("Draft/Attribution", {variables: {"draft-title": title}}) : $tw.language.getString("Draft/Title", {variables: {"draft-title": title}});
-		if(c) {
-			draftTitle = draftTitle.concat(" ", (c + 1).toString());
-		}
+		draftTitle = "Draft " + (c ? (c + 1) + " " : "") + "of '" + title + "'" + attribution;
 		c++;
 	} while(this.tiddlerExists(draftTitle));
 	return draftTitle;

core/stylesheets/custom-properties.tid

@@ -1,30 +0,0 @@
-title: $:/core/stylesheets/custom-properties
-
-\rules only transcludeinline macrocallinline html transcludeblock
-
-/* Tiddlywiki's CSS properties */
-
-:root {
-	<$list filter="[[$:/palettes/Vanilla]indexes[]]">
-	--tpc-<<currentTiddler>>: <$transclude $variable="colour" $mode="inline" name=<<currentTiddler>>/>;
-	</$list>
-
-	/* CSS settings */
-	--tp-code-wrapping: {{$:/themes/tiddlywiki/vanilla/options/codewrapping}};
-	--tp-font-family: {{$:/themes/tiddlywiki/vanilla/settings/fontfamily}};
-	--tp-code-font-family: {{$:/themes/tiddlywiki/vanilla/settings/codefontfamily}};
-	--tp-editor-font-family: {{$:/themes/tiddlywiki/vanilla/settings/editorfontfamily}};
-	--tp-font-size: {{$:/themes/tiddlywiki/vanilla/metrics/fontsize}};
-	--tp-line-height: {{$:/themes/tiddlywiki/vanilla/metrics/lineheight}};
-	--tp-body-font-size: {{$:/themes/tiddlywiki/vanilla/metrics/bodyfontsize}};
-	--tp-body-line-height: {{$:/themes/tiddlywiki/vanilla/metrics/bodylineheight}};
-	--tp-story-left: {{$:/themes/tiddlywiki/vanilla/metrics/storyleft}};
-	--tp-story-top: {{$:/themes/tiddlywiki/vanilla/metrics/storytop}};
-	--tp-story-right: {{$:/themes/tiddlywiki/vanilla/metrics/storyright}};
-	--tp-story-width: {{$:/themes/tiddlywiki/vanilla/metrics/storyrwidth}};
-	--tp-tiddler-width: {{$:/themes/tiddlywiki/vanilla/metrics/tiddlerwidth}};
-	--tp-sidebar-breakpoint: {{$:/themes/tiddlywiki/vanilla/metrics/sidebarbreakpoint}};
-	--tp-sidebar-width: {{$:/themes/tiddlywiki/vanilla/metrics/sidebarwidth}};
-
-	--tp-animation-duration: {{{ [{$:/config/AnimationDuration}addsuffix[ms]] }}};
-}

core/ui/ControlPanel/Basics.tid

@@ -18,7 +18,6 @@ caption: {{$:/language/ControlPanel/Basics/Caption}}
 |<$link to="$:/config/NewTiddler/Tags"><<lingo NewTiddler/Tags/Prompt>></$link> |<$vars currentTiddler="$:/config/NewTiddler/Tags" tagField="text">{{||$:/core/ui/EditTemplate/tags}}<$list filter="[<currentTiddler>tags[]] +[limit[1]]" variable="ignore"><$button tooltip={{$:/language/ControlPanel/Basics/RemoveTags/Hint}}><<lingo RemoveTags>><$action-listops $tiddler=<<currentTiddler>> $field="text" $subfilter={{{ [<currentTiddler>get[tags]] }}}/><$action-setfield $tiddler=<<currentTiddler>> tags=""/></$button></$list></$vars> |
 |<$link to="$:/config/NewJournal/Tags"><<lingo NewJournal/Tags/Prompt>></$link> |<$vars currentTiddler="$:/config/NewJournal/Tags" tagField="text">{{||$:/core/ui/EditTemplate/tags}}<$list filter="[<currentTiddler>tags[]] +[limit[1]]" variable="ignore"><$button tooltip={{$:/language/ControlPanel/Basics/RemoveTags/Hint}}><<lingo RemoveTags>><$action-listops $tiddler=<<currentTiddler>> $field="text" $subfilter={{{ [<currentTiddler>get[tags]] }}}/><$action-setfield $tiddler=<<currentTiddler>> tags=""/></$button></$list></$vars> |
 |<$link to="$:/config/AutoFocus"><<lingo AutoFocus/Prompt>></$link> |{{$:/snippets/minifocusswitcher}} |
-|<$link to="$:/config/AutoFocusEdit"><<lingo AutoFocusEdit/Prompt>></$link> |{{$:/snippets/minifocuseditswitcher}} |
 |<<lingo Language/Prompt>> |{{$:/snippets/minilanguageswitcher}} |
 |<<lingo Tiddlers/Prompt>> |<<show-filter-count "[!is[system]sort[title]]">> |
 |<<lingo Tags/Prompt>> |<<show-filter-count "[tags[]sort[title]]">> |

core/ui/EditTemplate/body-editor.tid

@@ -8,7 +8,7 @@ title: $:/core/ui/EditTemplate/body/editor
   class="tc-edit-texteditor tc-edit-texteditor-body"
   placeholder={{$:/language/EditTemplate/Body/Placeholder}}
   tabindex={{$:/config/EditTabIndex}}
-  focus={{{ [{!!draft.of}is[tiddler]then{$:/config/AutoFocusEdit}match[text]then[true]] ~[{$:/config/AutoFocus}match[text]then[true]] ~[[false]] }}}
+  focus={{{ [{$:/config/AutoFocus}match[text]then[true]] ~[[false]] }}}
   cancelPopups="yes"
   fileDrop={{{ [{$:/config/DragAndDrop/Enable}match[no]] :else[subfilter{$:/config/Editor/EnableImportFilter}then[yes]else[no]] }}}
 

core/ui/EditTemplate/body-toolbar-button.tid

@@ -103,9 +103,9 @@ title: $:/core/ui/EditTemplate/body/toolbar/button
 <$set
 
   name="buttonClasses"
-  value={{{ [subfilter{!!button-classes}] :and[join[ ]] }}}
+  value={{!!button-classes}}
 
 ><<toolbar-button>></$set>
 \end
 
-<<toolbar-button-outer>>
+<<toolbar-button-outer>>

core/ui/EditTemplate/fields.tid

@@ -1,181 +1,157 @@
 title: $:/core/ui/EditTemplate/fields
 tags: $:/tags/EditTemplate
 
-\whitespace trim
-
 \procedure lingo-base() $:/language/EditTemplate/
+\function tf.config-title() [[$:/config/EditTemplateFields/Visibility/]addsuffix[$(currentField)$]substitute[]get[text]]
+
+\function tf.config-filter() [[hide]] :except[title<tf.config-title>]
+
+<!-- Beware this is duplicated from EditTemplate.tid. For details see bug #7054 -->
+\procedure get-field-value-tiddler-filter() [subfilter<get-field-editor-filter>sha256[16]addprefix[/]addprefix<newFieldValueTiddlerPrefix>]
+\procedure get-field-editor-filter() [<newFieldNameTiddler>get[text]else[]] :cascade[all[shadows+tiddlers]tag[$:/tags/FieldEditorFilter]!is[draft]get[text]] :and[!is[blank]else{$:/core/ui/EditTemplate/fieldEditor/default}]
+
+\procedure prefix.bracket() [
+\procedure suffix.bracket() ]
+
+\function tf.current-tiddler-new-field-selector() [[data-tiddler-title=]addprefix[$(prefix.bracket)$]substitute[]addsuffix<currentTiddlerCSSescaped>addsuffix[$(suffix.bracket)$]substitute[]] .tc-edit-field-add-name-wrapper input :and[join[ ]]
+
+\procedure new-field-actions()
+\whitespace trim
+<$action-sendmessage $message="tm-add-field" $name={{{ [<newFieldNameTiddler>get[text]] }}} $value={{{ [<newFieldNameTiddler>get[text]] :map[subfilter<get-field-value-tiddler-filter>get[text]] }}}/>
+<$set name="safeNewFieldValueTiddlerPrefix" value=<<newFieldValueTiddlerPrefix>> emptyValue=<<qualify "$:/temp/NewFieldValue">> >
+	<$action-deletetiddler $filter="[<newFieldNameTiddler>] [prefix[$:/temp/NewFieldValue]prefix<safeNewFieldValueTiddlerPrefix>] [<storeTitle>] [<searchListState>]"/>
+</$set>
+<$action-sendmessage $message="tm-focus-selector" $param=<<tf.current-tiddler-new-field-selector>>/>
+\end
 
 \procedure delete-state-tiddlers() <$action-deletetiddler $filter="[<newFieldNameTiddler>] [<storeTitle>] [<searchListState>]"/>
 
-\procedure focus-new-field-input() <$action-sendmessage $message="tm-focus-selector" $param=`[data-tiddler-title="$(storyTiddler)$"] .tc-edit-field-add-name-wrapper input` />
-
-\procedure new-field-actions()
-	<$action-setfield $tiddler=<<storyTiddler>> $field={{{ [<newFieldNameTiddler>get[text]] }}} $value={{{ [<newFieldValueTiddler>get[text]] }}} />
-	<$action-deletetiddler $filter="[prefix[$:/temp/NewFieldValue]prefix<newFieldValueTiddlerPrefix>]"/>
-	<<delete-state-tiddlers>>
-	<<focus-new-field-input>>
-\end
-
-\procedure delete-field-actions()
-	<$action-deletefield $field=<<currentField>>/>
-	<<focus-new-field-input>>
+\procedure cancel-search-actions-inner()
+\whitespace trim
+<$list
+	filter="[<storeTitle>has[text]] [<newFieldNameTiddler>has[text]]"
+	variable="ignore"
+	emptyMessage="<<cancel-delete-tiddler-actions 'cancel'>>">
+<<delete-state-tiddlers>>
+</$list>
 \end
 
 \procedure cancel-search-actions()
-	<$let userInput={{{ [<storeTitle>get[text]] }}}>
-		<%if [<newFieldNameTiddler>get[text]!match<userInput>] %>
-			<$action-setfield $tiddler=<<newFieldNameTiddler>> text=<<userInput>>/>
-			<$action-setfield $tiddler=<<refreshTitle>> text="yes"/>
-		<%else%>
-			<%if [<storeTitle>has[text]] [<newFieldNameTiddler>has[text]] %>
-				<<delete-state-tiddlers>>	
-			<%else%>
-				<<cancel-delete-tiddler-actions 'cancel'>>
-			<%endif%>
-		<%endif%>
-	</$let>
+\whitespace trim
+<$set name="userInput" value={{{ [<storeTitle>get[text]] }}}>
+<$list
+	filter="[<newFieldNameTiddler>get[text]!match<userInput>]"
+	emptyMessage="<<cancel-search-actions-inner>>">
+<$action-setfield $tiddler=<<newFieldNameTiddler>> text=<<userInput>>/><$action-setfield $tiddler=<<refreshTitle>> text="yes"/>
+</$list>
+</$set>
 \end
 
 \procedure new-field()
-	<%if [<newFieldNameTiddler>get[text]!is[blank]] %>
-		<$button actions="<<new-field-actions>>" tooltip={{$:/language/EditTemplate/Fields/Add/Button/Hint}}>
-			<<lingo Fields/Add/Button>>
-		</$button>	
-	<%else%>
-		<$button>	
-			<<lingo Fields/Add/Button>>
-		</$button>	
-	<%endif%>
+\whitespace trim
+<$vars name={{{ [<newFieldNameTiddler>get[text]] }}}>
+<$reveal type="nomatch" text="" default=<<name>>>
+<$button tooltip={{$:/language/EditTemplate/Fields/Add/Button/Hint}}>
+<$action-sendmessage $message="tm-add-field"
+$name=<<name>>
+$value={{{ [subfilter<get-field-value-tiddler-filter>get[text]] }}}/>
+<$set name="safeNewFieldValueTiddlerPrefix" value=<<newFieldValueTiddlerPrefix>> emptyValue=<<qualify "$:/temp/NewFieldValue">> >
+	<$action-deletetiddler $filter="[<newFieldNameTiddler>] [prefix[$:/temp/NewFieldValue]prefix<safeNewFieldValueTiddlerPrefix>] [<storeTitle>] [<searchListState>]"/>
+</$set>
+<<lingo Fields/Add/Button>>
+</$button>
+</$reveal>
+<$reveal type="match" text="" default=<<name>>>
+<$button>
+<<lingo Fields/Add/Button>>
+</$button>
+</$reveal>
+</$vars>
 \end
+\whitespace trim
 
-\function tf.config-filter() [lookup:show[$:/config/EditTemplateFields/Visibility/]!match[hide]]
+<$set name="newFieldValueTiddlerPrefix" value=<<newFieldValueTiddlerPrefix>> emptyValue=<<qualify "$:/temp/NewFieldValue">> >
+<div class="tc-edit-fields">
+<table class={{{ [all[current]fields[]] :filter[lookup[$:/config/EditTemplateFields/Visibility/]!match[hide]] :and[count[]!match[0]] :and[then[tc-edit-fields]] :else[[tc-edit-fields tc-edit-fields-small]] }}}>
+<tbody>
+<$list filter="[all[current]fields[]] :and[sort[title]]" variable="currentField" storyview="pop">
+<$list filter=<<tf.config-filter>> variable="temp">
+<tr class="tc-edit-field">
+<td class="tc-edit-field-name">
+<$text text=<<currentField>>/>:</td>
+<td class="tc-edit-field-value">
+<$keyboard key="((delete-field))" actions="""<$action-deletefield $field=<<currentField>>/><$set name="currentTiddlerCSSescaped" value={{{ [<currentTiddler>escapecss[]] }}}><$action-sendmessage $message="tm-focus-selector" $param=<<tf.current-tiddler-new-field-selector>>/></$set>""">
+<$transclude tiddler={{{ [<currentField>] :cascade[all[shadows+tiddlers]tag[$:/tags/FieldEditorFilter]!is[draft]get[text]] :and[!is[blank]else{$:/core/ui/EditTemplate/fieldEditor/default}] }}} />
+</$keyboard>
+</td>
+<td class="tc-edit-field-remove">
+<$button class="tc-btn-invisible" tooltip={{$:/language/EditTemplate/Field/Remove/Hint}} aria-label={{$:/language/EditTemplate/Field/Remove/Caption}}>
+<$action-deletefield $field=<<currentField>>/>
+{{$:/core/images/delete-button}}
+</$button>
+</td>
+</tr>
+</$list>
+</$list>
+</tbody>
+</table>
+</div>
 
-\function tf.field-cascade()
-	[<currentField>]
-	:cascade[all[shadows+tiddlers]tag[$:/tags/FieldEditorFilter]!is[draft]get[text]]
-	:and[!is[blank]else{$:/core/ui/EditTemplate/fieldEditor/default}]
-\end
-
-\function tf.get-field-editor()
-	[<newFieldNameTiddler>get[text]else[]]
-	:cascade[all[shadows+tiddlers]tag[$:/tags/FieldEditorFilter]!is[draft]get[text]]
-	:and[!is[blank]else{$:/core/ui/EditTemplate/fieldEditor/default}]
-\end
-
-\function tf.primary-list-exceptions() created creator draft.of draft.title modified modifier tags text title type
-
-\function tf.list-selection-class(listSuffix) [<searchListState>get[text]removesuffix<listSuffix>match<currentField>then[tc-list-item-selected]]
-
-
-<$let newFieldValueTiddlerPrefix={{{ [<newFieldValueTiddlerPrefix>!is[blank]else<qualify "$:/temp/NewFieldValue">] }}} >
-	<div class="tc-edit-fields">
-		<!-- table of user fields of the current tiddler -->
-		<table class=`tc-edit-fields ${ [all[current]fields[]] :filter[tf.config-filter[]] :and[count[]match[0]then[tc-edit-fields-small]] }$`>
-			<tbody>
-				<$list filter="[all[current]fields[]] :and[sort[title]]" variable="currentField" storyview="pop">
-					<%if [<currentField>tf.config-filter[]] %>
-						<tr class="tc-edit-field">
-							<td class="tc-edit-field-name">
-								<$text text=<<currentField>>/>:
-							</td>
-							<td class="tc-edit-field-value">
-								<$keyboard key="((delete-field))" actions="<<delete-field-actions>>">
-									<$transclude tiddler=<<tf.field-cascade>> />
-								</$keyboard>
-							</td>
-							<td class="tc-edit-field-remove">
-								<$button actions="<<delete-field-actions>>"
-									aria-label={{$:/language/EditTemplate/Field/Remove/Caption}}
-									class="tc-btn-invisible"
-									tooltip={{$:/language/EditTemplate/Field/Remove/Hint}}
-								>
-									{{$:/core/images/delete-button}}
-								</$button>
-							</td>
-						</tr>
-					<%endif%>
-				</$list>
-			</tbody>
-		</table>
-	</div>
-	
-	<!-- input control for new field name with selection dropdown -->
-	<div class="tc-edit-field-add">
-		<em class="tc-edit tc-small-gap-right">
-			<<lingo Fields/Add/Prompt>>
-		</em>
-		<$let refreshTitle=<<qualify "$:/temp/fieldname/refresh">>
-			storeTitle=<<newFieldNameInputTiddler>>
-			searchListState=<<newFieldNameSelectionTiddler>>
-		>
-			<div class="tc-edit-field-add-name-wrapper">
-				<$transclude $variable="keyboard-driven-input"
-					cancelPopups="yes"
-					class=`tc-edit-texteditor tc-popup-handle ${ [<newFieldNameTiddler>get[text]] :intersection[<storyTiddler>fields[]] :then[[tc-edit-field-exists]] }$`
-					configTiddlerFilter="[[$:/config/EditMode/fieldname-filter]]"
-					default=""
-					focus={{{ [{!!draft.of}is[tiddler]then{$:/config/AutoFocusEdit}match[fields]then[true]] :else[{$:/config/AutoFocus}match[fields]then[true]] :else[[false]] }}}
-					focusPopup=<<qualify "$:/state/popup/field-dropdown">>
-					inputAcceptVariantActions=<<save-tiddler-actions>>
-					inputCancelActions=<<cancel-search-actions>>
-					placeholder={{$:/language/EditTemplate/Fields/Add/Name/Placeholder}}
-					refreshTitle=<<refreshTitle>>
-					selectionStateTitle=<<searchListState>>
-					storeTitle=<<storeTitle>>
-					tag="input"
-					tabindex={{$:/config/EditTabIndex}}
-					tiddler=<<newFieldNameTiddler>>
-				/>
-				<$button aria-label={{$:/language/EditTemplate/Field/Dropdown/Caption}}
-					class="tc-btn-invisible tc-btn-dropdown tc-small-gap"
-					popup=<<qualify "$:/state/popup/field-dropdown">>
-					tooltip={{$:/language/EditTemplate/Field/Dropdown/Hint}}
-				>
-					{{$:/core/images/down-arrow}}
-				</$button>
-				<$reveal state=<<qualify "$:/state/popup/field-dropdown">> type="nomatch" text="" default="" tag="div" class="tc-block-dropdown tc-edit-type-dropdown">
-					<$let tv-show-missing-links="yes">
-						<$linkcatcher to=<<newFieldNameTiddler>>>
-							<div class="tc-dropdown-item">
-								<<lingo Fields/Add/Dropdown/User>>
-							</div>
-							<$let newFieldName={{{ [<storeTitle>get[text]] }}}
-								primaryListFields={{{ [!is[shadow]!is[system]fields[]format:titlelist[]join[ ]] }}}
-							>
-								<$list filter="[enlist<primaryListFields>search:title<newFieldName>sort[]] :except[tf.primary-list-exceptions[]]" variable="currentField">
-									<$link to=<<currentField>> class=<<tf.list-selection-class "-primaryList">> >
-										<$text text=<<currentField>>/>
-									</$link>
-								</$list>
-								<div class="tc-dropdown-item">
-									<<lingo Fields/Add/Dropdown/System>>
-								</div>
-								<$list filter="[fields[]search:title<newFieldName>!enlist<primaryListFields>sort[]]" variable="currentField">
-									<$link to=<<currentField>> class=<<tf.list-selection-class "-secondaryList">>>
-										<$text text=<<currentField>>/>
-									</$link>
-								</$list>
-							</$let>
-						</$linkcatcher>
-					</$let>
-				</$reveal>
-			</div>
-
-			<!-- input control for new field content -->
-			<$let currentFieldName={{{ [<newFieldNameTiddler>get[text]] }}}
-				fieldEditor=<<tf.get-field-editor>>
-				newFieldValueTiddler={{{ [<newFieldValueTiddlerPrefix>] [[/]] [<fieldEditor>sha256[16]] :and[join[]] }}}
-				currentTiddler=<<newFieldValueTiddler>>
-			>
-				<span class="tc-edit-field-add-value tc-small-gap-right">
-					<$keyboard key="((add-field))" actions="<<new-field-actions>>">
-						<$transclude $tiddler=<<fieldEditor>> />
-					</$keyboard>
-				</span>
-				<span class="tc-edit-field-add-button">
-					<$transclude $variable="new-field"/>
-				</span>
-			</$let>
-		</$let>
-	</div>
-</$let>
+<$fieldmangler>
+<div class="tc-edit-field-add">
+<em class="tc-edit tc-small-gap-right">
+<<lingo Fields/Add/Prompt>>
+</em>
+<$vars refreshTitle=<<qualify "$:/temp/fieldname/refresh">> storeTitle=<<newFieldNameInputTiddler>> searchListState=<<newFieldNameSelectionTiddler>>>
+<div class="tc-edit-field-add-name-wrapper">
+<$transclude $variable="keyboard-driven-input" tiddler=<<newFieldNameTiddler>> storeTitle=<<storeTitle>> refreshTitle=<<refreshTitle>>
+		selectionStateTitle=<<searchListState>> tag="input" default="" placeholder={{$:/language/EditTemplate/Fields/Add/Name/Placeholder}}
+		focusPopup=<<qualify "$:/state/popup/field-dropdown">> class="tc-edit-texteditor tc-popup-handle" tabindex={{$:/config/EditTabIndex}}
+		focus={{{ [{$:/config/AutoFocus}match[fields]then[true]] :else[[false]] }}} cancelPopups="yes"
+		configTiddlerFilter="[[$:/config/EditMode/fieldname-filter]]" inputCancelActions=<<cancel-search-actions>> />
+<$button popup=<<qualify "$:/state/popup/field-dropdown">> class="tc-btn-invisible tc-btn-dropdown tc-small-gap" tooltip={{$:/language/EditTemplate/Field/Dropdown/Hint}} aria-label={{$:/language/EditTemplate/Field/Dropdown/Caption}}>{{$:/core/images/down-arrow}}</$button>
+<$reveal state=<<qualify "$:/state/popup/field-dropdown">> type="nomatch" text="" default="">
+<div class="tc-block-dropdown tc-edit-type-dropdown">
+<$set name="tv-show-missing-links" value="yes">
+<$linkcatcher to=<<newFieldNameTiddler>>>
+<div class="tc-dropdown-item">
+<<lingo Fields/Add/Dropdown/User>>
+</div>
+<$set name="newFieldName" value={{{ [<storeTitle>get[text]] }}}>
+<$list filter="[!is[shadow]!is[system]fields[]search:title<newFieldName>sort[]] :except[[created]] :except[[creator]] :except[[draft.of]] :except[[draft.title]] :except[[modified]] :except[[modifier]] :except[[tags]] :except[[text]] :except[[title]] :except[[type]]"  variable="currentField">
+<$list filter="[<currentField>addsuffix[-primaryList]] :except[<searchListState>get[text]]" emptyMessage="""<$link to=<<currentField>> class="tc-list-item-selected"><$text text=<<currentField>>/></$link>""">
+<$link to=<<currentField>>>
+<$text text=<<currentField>>/>
+</$link>
+</$list>
+</$list>
+<div class="tc-dropdown-item">
+<<lingo Fields/Add/Dropdown/System>>
+</div>
+<$list filter="[fields[]search:title<newFieldName>sort[]] :except[!is[shadow]!is[system]fields[]]" variable="currentField">
+<$list filter="[<currentField>addsuffix[-secondaryList]] :except[<searchListState>get[text]]" emptyMessage="""<$link to=<<currentField>> class="tc-list-item-selected"><$text text=<<currentField>>/></$link>""">
+<$link to=<<currentField>>>
+<$text text=<<currentField>>/>
+</$link>
+</$list>
+</$list>
+</$set>
+</$linkcatcher>
+</$set>
+</div>
+</$reveal>
+</div>
+<$let currentTiddlerCSSescaped={{{ [<currentTiddler>escapecss[]] }}} currentTiddler={{{ [subfilter<get-field-value-tiddler-filter>] }}} currentField="text" currentFieldName={{{ [<newFieldNameTiddler>get[text]] }}}>
+<span class="tc-edit-field-add-value tc-small-gap-right">
+<$keyboard key="((add-field))" actions=<<new-field-actions>>>
+<$transclude tiddler={{{ [subfilter<get-field-editor-filter>] }}} />
+</$keyboard>
+</span>
+<span class="tc-edit-field-add-button">
+<$transclude $variable="new-field"/>
+</span>
+</$let>
+</$vars>
+</div>
+</$fieldmangler>
+</$set>

core/ui/EditTemplate/tags.tid

@@ -17,7 +17,7 @@ tags: $:/tags/EditTemplate
 	<$let backgroundColor=<<colour>> >
 		<span class="tc-tag-label tc-tag-list-item tc-small-gap-right"
 			data-tag-title=<<currentTiddler>>
-			style=`color:$(foregroundColor)$; background-color:$(backgroundColor)$; --tp-remove-tag-button-color:$(foregroundColor)$`
+			style=`color:$(foregroundColor)$; background-color:$(backgroundColor)$;`
 		>
 			<$transclude tiddler=<<icon>>/>
 			<$view field="title" format="text"/>

core/ui/EditTemplate/title.tid

@@ -2,11 +2,7 @@ title: $:/core/ui/EditTemplate/title
 tags: $:/tags/EditTemplate
 
 \whitespace trim
-<$edit-text field="draft.title" class="tc-titlebar tc-edit-texteditor"
-	focus={{{ [{!!draft.of}is[tiddler]then{$:/config/AutoFocusEdit}match[title]then[true]] ~[{$:/config/AutoFocus}match[title]then[true]] ~[[false]] }}}
-	tabindex={{$:/config/EditTabIndex}}
-	cancelPopups="yes"
-/>
+<$edit-text field="draft.title" class="tc-titlebar tc-edit-texteditor" focus={{{ [{$:/config/AutoFocus}match[title]then[true]] ~[[false]] }}} tabindex={{$:/config/EditTabIndex}} cancelPopups="yes"/>
 
 <$vars pattern="""[\|\[\]{}]""" bad-chars="""`| [ ] { }`""">
 

core/ui/EditTemplate/type.tid

@@ -4,28 +4,13 @@ first-search-filter: [all[shadows+tiddlers]prefix[$:/language/Docs/Types/]sort[d
 
 \procedure lingo-base() $:/language/EditTemplate/
 \procedure input-cancel-actions() <$list filter="[<storeTitle>get[text]] [<currentTiddler>get[type]] :and[limit[1]]" emptyMessage="""<<cancel-delete-tiddler-actions "cancel">>"""><$action-sendmessage $message="tm-remove-field" $param="type"/><$action-deletetiddler $filter="[<typeInputTiddler>] [<refreshTitle>] [<typeSelectionTiddler>]"/></$list>
-
 \whitespace trim
 <$set name="refreshTitle" value=<<qualify "$:/temp/type-search/refresh">>>
 <div class="tc-edit-type-selector-wrapper">
 <em class="tc-edit tc-small-gap-right"><<lingo Type/Prompt>></em>
 <div class="tc-type-selector-dropdown-wrapper">
-<div class="tc-type-selector">
-<$fieldmangler>
-<$transclude $variable="keyboard-driven-input" tiddler=<<currentTiddler>> storeTitle=<<typeInputTiddler>> refreshTitle=<<refreshTitle>>
-	selectionStateTitle=<<typeSelectionTiddler>> field="type" tag="input" default="" placeholder={{$:/language/EditTemplate/Type/Placeholder}}
-	focusPopup=<<qualify "$:/state/popup/type-dropdown">> class="tc-edit-typeeditor tc-edit-texteditor tc-popup-handle tc-keep-focus"
-	tabindex={{$:/config/EditTabIndex}}
-	focus={{{ [{!!draft.of}is[tiddler]then{$:/config/AutoFocusEdit}match[type]then[true]] :else[{$:/config/AutoFocus}match[type]then[true]] :else[[false]] }}}
-	cancelPopups="yes" configTiddlerFilter="[[$:/core/ui/EditTemplate/type]]"
-	inputCancelActions=<<input-cancel-actions>>
-/>
-	<$button popup=<<qualify "$:/state/popup/type-dropdown">> class="tc-btn-invisible tc-btn-dropdown tc-small-gap" tooltip={{$:/language/EditTemplate/Type/Dropdown/Hint}} aria-label={{$:/language/EditTemplate/Type/Dropdown/Caption}}>
-		{{$:/core/images/down-arrow}}
-	</$button>
-	<$button message="tm-remove-field" param="type" class="tc-btn-invisible tc-btn-icon" tooltip={{$:/language/EditTemplate/Type/Delete/Hint}} aria-label={{$:/language/EditTemplate/Type/Delete/Caption}}>
-		{{$:/core/images/delete-button}}<$action-deletetiddler $filter="[<typeInputTiddler>] [<storeTitle>] [<refreshTitle>] [<selectionStateTitle>]"/>
-	</$button>
+<div class="tc-type-selector"><$fieldmangler>
+<$transclude $variable="keyboard-driven-input" tiddler=<<currentTiddler>> storeTitle=<<typeInputTiddler>> refreshTitle=<<refreshTitle>> selectionStateTitle=<<typeSelectionTiddler>> field="type" tag="input" default="" placeholder={{$:/language/EditTemplate/Type/Placeholder}} focusPopup=<<qualify "$:/state/popup/type-dropdown">> class="tc-edit-typeeditor tc-edit-texteditor tc-popup-handle tc-keep-focus" tabindex={{$:/config/EditTabIndex}} focus={{{ [{$:/config/AutoFocus}match[type]then[true]] :else[[false]] }}} cancelPopups="yes" configTiddlerFilter="[[$:/core/ui/EditTemplate/type]]" inputCancelActions=<<input-cancel-actions>>/><$button popup=<<qualify "$:/state/popup/type-dropdown">> class="tc-btn-invisible tc-btn-dropdown tc-small-gap" tooltip={{$:/language/EditTemplate/Type/Dropdown/Hint}} aria-label={{$:/language/EditTemplate/Type/Dropdown/Caption}}>{{$:/core/images/down-arrow}}</$button><$button message="tm-remove-field" param="type" class="tc-btn-invisible tc-btn-icon" tooltip={{$:/language/EditTemplate/Type/Delete/Hint}} aria-label={{$:/language/EditTemplate/Type/Delete/Caption}}>{{$:/core/images/delete-button}}<$action-deletetiddler $filter="[<typeInputTiddler>] [<storeTitle>] [<refreshTitle>] [<selectionStateTitle>]"/></$button>
 </$fieldmangler></div>
 
 <div class="tc-block-dropdown-wrapper">

core/ui/Filters/StoryList.tid

@@ -1,5 +1,5 @@
 title: $:/core/Filters/StoryList
 tags: $:/tags/Filter
-filter: [<tv-story-list>is[variable]then<tv-story-list>else[$:/StoryList]] =>storylist [list<storylist>] -$:/AdvancedSearch
+filter: [list[$:/StoryList]] -$:/AdvancedSearch
 description: {{$:/language/Filters/StoryList}}
 

core/ui/PageStylesheet.tid

@@ -8,8 +8,6 @@ code-body: yes
 
 <$set name="languageTitle" value={{!!name}}>
 
-<$transclude $tiddler="$:/core/stylesheets/custom-properties" $mode="block"/>
-
 <$list filter="[all[shadows+tiddlers]tag[$:/tags/Stylesheet]!has[draft.of]]">
 <$transclude mode="block"/>
 </$list>

core/ui/TiddlerInfo/Advanced/CascadeInfo.tid

@@ -1,43 +0,0 @@
-title: $:/core/ui/TiddlerInfo/Advanced/CascadeInfo
-tags: $:/tags/TiddlerInfo/Advanced
-
-\define lingo-base() $:/language/TiddlerInfo/Advanced/CascadeInfo/
-
-<$let infoTiddler=<<currentTiddler>>>
-
-''<<lingo Heading>>''
-
-<<lingo Hint>>
-
-<table class="tc-max-width">
-	<thead>
-		<$list filter="[[View]] [[ActiveCascadeFilter]] [[Template]]" variable="th">
-			<th><$transclude $variable="lingo" title=`Detail/$(th)$`/></th>
-		</$list>
-	</thead>
-	<$list filter="[[$:/tags/ViewTemplate]tagging[]]" variable="ViewTemplate">
-		<tr>
-			<$let
-				view={{{ [<ViewTemplate>]+[split[/]last[]] }}}
-				tagFilter=`$:/tags/ViewTemplate${ [<view>titlecase[]] }$Filter`
-				activeCascadeFilterTiddler={{{ [all[shadows+tiddlers]tag<tagFilter>!is[draft]]:filter[<storyTiddler>subfilter{!!text}]+[first[]] }}}
-				activeCascadeFilter={{{ [<activeCascadeFilterTiddler>get[text]] }}}
-				activeTemplateTiddler={{{ [<currentTiddler>]:cascade[all[shadows+tiddlers]tag<tagFilter>!is[draft]get[text]] }}}
-			>
-			<%if [<activeCascadeFilterTiddler>!is[blank]]%>
-				<td>
-					<$link to=<<ViewTemplate>> ><<view>></$link>
-				</td>
-				<td>
-					<$link to=<<activeCascadeFilterTiddler>> />
-				</td>
-				<td style="text-align:center;">
-					<$link class="tc-btn-invisible" to=<<activeTemplateTiddler>>>
-						<$button class="tc-btn-invisible">{{$:/core/images/file}}</$button>
-					</$link>
-				</td>
-			<%endif%>
-			</$let>
-		</tr>
-	</$list>
-</table>

core/wiki/config/MediaQueryTrackers/DarkLightPreferred.tid

@@ -1,5 +0,0 @@
-title: $:/core/wiki/config/MediaQueryTrackers/DarkLightPreferred
-tags: $:/tags/MediaQueryTracker
-media-query: (prefers-color-scheme: dark)
-info-tiddler: $:/info/browser/darkmode
-info-tiddler-alt: $:/info/darkmode

core/wiki/macros/list.tid

@@ -22,7 +22,7 @@ tags: $:/tags/Macro
 <$action-listops $tiddler=<<targetTiddler>> $field=<<targetField>> $subfilter="+[insertbefore<actionTiddler>,<currentTiddler>]"/>
 \end
 
-\define list-links-draggable(tiddler,field:"list",emptyMessage,type:"ul",subtype:"li",class:"",itemTemplate,displayField:"caption",startactions,endactions)
+\define list-links-draggable(tiddler,field:"list",emptyMessage,type:"ul",subtype:"li",class:"",itemTemplate, displayField:"caption")
 \whitespace trim
 <$set name="_tiddler" value="""$tiddler$""" emptyValue=<<currentTiddler>> >
 	<$let field-reference={{{ [<_tiddler>] "!!" [[$field$]] +[join[]] }}}
@@ -39,11 +39,8 @@ tags: $:/tags/Macro
 					>
 						<div class="tc-droppable-placeholder"/>
 						<div>
-							<$transclude tiddler=<<__itemTemplate__>>>
-								<$link to={{!!title}}
-									startactions=<<__startactions__>>
-									endactions=<<__endactions__>>
-								>
+							<$transclude tiddler="""$itemTemplate$""">
+								<$link to={{!!title}}>
 									<$let tv-wikilinks="no">
 										<$transclude field=<<__displayField__>>>
 											<$view field="title"/>
@@ -95,7 +92,7 @@ tags: $:/tags/Macro
 </$set>
 \end
 
-\define list-tagged-draggable(tag,subFilter,emptyMessage,itemTemplate,elementTag:"div",storyview:"",displayField:"title",startactions,endactions)
+\define list-tagged-draggable(tag,subFilter,emptyMessage,itemTemplate,elementTag:"div",storyview:"",displayField:"title")
 \whitespace trim
 <span class="tc-tagged-draggable-list">
 	<$set name="tag" value=<<__tag__>>>
@@ -111,11 +108,8 @@ tags: $:/tags/Macro
 				>
 					<$genesis $type=<<__elementTag__>> class="tc-droppable-placeholder"/>
 					<$genesis $type=<<__elementTag__>>>
-						<$transclude tiddler=<<__itemTemplate__>>>
-							<$link to={{!!title}}
-								startactions=<<__startactions__>>
-								endactions=<<__endactions__>>
-							>
+						<$transclude tiddler="""$itemTemplate$""">
+							<$link to={{!!title}}>
 								<$let tv-wikilinks="no">
 									<$transclude field=<<__displayField__>>>
 										<$view field="title"/>

core/wiki/macros/tag-picker.tid

@@ -118,7 +118,7 @@ The second ESC tries to close the "draft tiddler"
 				focusPopup=<<tf.tagpicker-dropdown-id>>
 				class="tc-edit-texteditor tc-popup-handle"
 				tabindex=<<tabIndex>>
-				focus={{{ [{!!draft.of}is[tiddler]then{$:/config/AutoFocusEdit}match[tags]then[true]] :else[{$:/config/AutoFocus}match[tags]then[true]] :else[[false]] }}}
+				focus={{{ [{$:/config/AutoFocus}match[tags]then[true]] :else[[false]] }}}
 				filterMinLength={{$:/config/Tags/MinLength}}
 				cancelPopups=<<cancelPopups>>
 				configTiddlerFilter="[[$:/core/macros/tag-picker]]"

core/wiki/macros/toc.tid

@@ -15,18 +15,7 @@ tags: $:/tags/Macro
 </span>
 \end
 
-\define toc-level-indicator()
-\whitespace trim
-<%if [<__level__>compare:number:gt[0]]%>
-  <%if [<currentTiddler>tagging[]] %>
-    <span class="tc-tiny-gap-left">{{$:/core/images/new-button}}</span>
-  <%else%>
-    <span class="tc-tiny-gap-left">{{$:/core/images/blank}}</span>
-  <%endif%>
-<% endif %>
-\end
-
-\define toc-body(tag,sort:"",itemClassFilter,exclude,path,level)
+\define toc-body(tag,sort:"",itemClassFilter,exclude,path)
 \whitespace trim
 <ol class="tc-toc">
   <$list filter="""[all[shadows+tiddlers]tag<__tag__>!has[draft.of]$sort$] -[<__tag__>] -[subfilter<__exclude__>]""">
@@ -34,26 +23,10 @@ tags: $:/tags/Macro
       <$set name="excluded" filter="[subfilter<__exclude__>] [<__tag__>]">
         <$set name="toc-item-class" filter=<<__itemClassFilter__>> emptyValue="toc-item-selected" value="toc-item">
           <li class=<<toc-item-class>>>
-            <$list filter="[all[current]toc-link[no]]" >
-              <$list-empty>
-                <!-- link to target-field or currentTiddler -->
-                <$link to={{{ [<currentTiddler>get[target]else<currentTiddler>] }}}>
-                  <<toc-level-indicator>>
-                  <<toc-caption>>
-                </$link>
-              </$list-empty>
-              <!-- toc-link = no -->
-              <<toc-level-indicator>>
+            <$list filter="[all[current]toc-link[no]]" emptyMessage="<$link to={{{ [<currentTiddler>get[target]else<currentTiddler>] }}}><<toc-caption>></$link>">
               <<toc-caption>>
             </$list>
-            <$let _level={{{ [<__level__>subtract[1]] }}}>
-              <%if [<_level>compare:number:gt[0]]%>
-                <$macrocall $name="toc-body" tag=<<item>> sort=<<__sort__>> itemClassFilter=<<__itemClassFilter__>> exclude=<<excluded>> path=<<path>> level=<<_level>>/>
-              <%elseif [<_level>match[-1]]%>
-                <!-- show full toc, no level defined -->
-                <$macrocall $name="toc-body" tag=<<item>> sort=<<__sort__>> itemClassFilter=<<__itemClassFilter__>> exclude=<<excluded>> path=<<path>>/>
-              <%endif%>
-            </$let>
+            <$macrocall $name="toc-body" tag=<<item>> sort=<<__sort__>> itemClassFilter=<<__itemClassFilter__>> exclude=<<excluded>> path=<<path>>/>
           </li>
         </$set>
       </$set>
@@ -62,10 +35,10 @@ tags: $:/tags/Macro
 </ol>
 \end
 
-\define toc(tag,sort:"",itemClassFilter:"",exclude,level)
+\define toc(tag,sort:"",itemClassFilter:"", exclude)
 \whitespace trim
 <$let __tag__={{{ [<__tag__>is[blank]then<currentTiddler>else<__tag__>] }}} >
-  <$macrocall $name="toc-body"  tag=<<__tag__>> sort=<<__sort__>> itemClassFilter=<<__itemClassFilter__>> exclude=<<__exclude__>> level=<<__level__>>/>
+  <$macrocall $name="toc-body"  tag=<<__tag__>> sort=<<__sort__>> itemClassFilter=<<__itemClassFilter__>> exclude=<<__exclude__>>/>
 </$let>
 \end
 

core/wiki/minifocuseditswitcher.tid

@@ -1,8 +0,0 @@
-title: $:/snippets/minifocuseditswitcher
-
-\whitespace trim
-<$select tiddler="$:/config/AutoFocusEdit" default={{$:/config/AutoFocus}}>
-<$list filter="title tags text">
-<option><<currentTiddler>></option>
-</$list>
-</$select>

core/wiki/tags/TiddlerInfoAdvanced.tid

@@ -1,2 +1,2 @@
 title: $:/tags/TiddlerInfo/Advanced
-list: [[$:/core/ui/TiddlerInfo/Advanced/ShadowInfo]] [[$:/core/ui/TiddlerInfo/Advanced/PluginInfo]] [[$:/core/ui/TiddlerInfo/Advanced/CascadeInfo]]
+list: [[$:/core/ui/TiddlerInfo/Advanced/ShadowInfo]] [[$:/core/ui/TiddlerInfo/Advanced/PluginInfo]] 

editions/de-AT/tiddlywiki.info

@@ -20,7 +20,8 @@
 		"index": [
 			"--rendertiddler","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--rendertiddler","$:/editions/de-AT-DE/download-empty","empty.html","text/plain"],
+			"--rendertiddler","$:/editions/de-AT-DE/download-empty","empty.html","text/plain",
+			"--rendertiddler","$:/editions/de-AT-DE/download-empty","empty.hta","text/plain"],
 		"favicon": [
 			"--savetiddler","$:/favicon.ico","favicon.ico"],
 		"static": [

editions/de-DE/tiddlywiki.info

@@ -24,7 +24,8 @@
 		"index": [
 			"--rendertiddler","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--rendertiddler","$:/editions/de-AT-DE/download-empty","empty.html","text/plain"],
+			"--rendertiddler","$:/editions/de-AT-DE/download-empty","empty.html","text/plain",
+			"--rendertiddler","$:/editions/de-AT-DE/download-empty","empty.hta","text/plain"],
 		"favicon": [
 			"--savetiddler","$:/favicon.ico","favicon.ico"],
 		"static": [

editions/empty/tiddlywiki.info

@@ -10,7 +10,8 @@
 		"index": [
 			"--render","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--render","$:/core/save/all","empty.html","text/plain"],
+			"--render","$:/core/save/all","empty.html","text/plain",
+			"--render","$:/core/save/all","empty.hta","text/plain"],
 		"emptyexternalcore": [
 			"--render","$:/core/save/offline-external-js","empty-external-core.html","text/plain",
 			"--render","$:/core/templates/tiddlywiki5.js","[[tiddlywikicore-]addsuffix<version>addsuffix[.js]]","text/plain"],

editions/es-ES/tiddlywiki.info

@@ -27,7 +27,8 @@
 			"--setfield","[tag[external-image]] [tag[external-text]]","text","","text/plain",
 			"--rendertiddler","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--rendertiddler","$:/editions/es-ES/download-empty","empty.html","text/plain"],
+			"--rendertiddler","$:/editions/es-ES/download-empty","empty.html","text/plain",
+			"--rendertiddler","$:/editions/es-ES/download-empty","empty.hta","text/plain"],
 		"favicon": [
 			"--savetiddler","$:/favicon.ico","favicon.ico",
 			"--savetiddler","$:/green_favicon.ico","static/favicon.ico"],

editions/fr-FR/tiddlywiki.info

@@ -27,7 +27,8 @@
 			"--setfield","[tag[external-image]] [tag[external-text]]","text","","text/plain",
 			"--rendertiddler","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--rendertiddler","$:/editions/fr-FR/download-empty","empty.html","text/plain"],
+			"--rendertiddler","$:/editions/fr-FR/download-empty","empty.html","text/plain",
+			"--rendertiddler","$:/editions/fr-FR/download-empty","empty.hta","text/plain"],
 		"favicon": [
 			"--savetiddler","$:/favicon.ico","favicon.ico",
 			"--savetiddler","$:/green_favicon.ico","static/favicon.ico"],

editions/ja-JP/tiddlywiki.info

@@ -27,7 +27,8 @@
 			"--setfield","[tag[external-image]] [tag[external-text]]","text","","text/plain",
 			"--rendertiddler","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--rendertiddler","$:/editions/ja-JP/download-empty","empty.html","text/plain"],
+			"--rendertiddler","$:/editions/ja-JP/download-empty","empty.html","text/plain",
+			"--rendertiddler","$:/editions/ja-JP/download-empty","empty.hta","text/plain"],
 		"favicon": [
 			"--savetiddler","$:/favicon.ico","favicon.ico",
 			"--savetiddler","$:/green_favicon.ico","static/favicon.ico"],

editions/ko-KR/tiddlywiki.info

@@ -27,7 +27,8 @@
 			"--setfield","[tag[external-image]] [tag[external-text]]","text","","text/plain",
 			"--rendertiddler","$:/core/save/all","index.html","text/plain"],
 		"empty": [
-			"--rendertiddler","$:/editions/ko-KR/download-empty","empty.html","text/plain"],
+			"--rendertiddler","$:/editions/ko-KR/download-empty","empty.html","text/plain",
+			"--rendertiddler","$:/editions/ko-KR/download-empty","empty.hta","text/plain"],
 		"favicon": [
 			"--savetiddler","$:/favicon.ico","favicon.ico",
 			"--savetiddler","$:/green_favicon.ico","static/favicon.ico"],

editions/test/tiddlers/tests/data/functions/FunctionDefaultValues.tid

@@ -1,16 +0,0 @@
-title: Functions/FunctionDefaultValues
-description: Use defaults for missing parameters in functions in filters
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-\function .test(prefix:Default) [[ Content]addprefix<prefix>]
-
-<$text text={{{ [.test[Special]] }}}/>,<$text text={{{ [.test[]] }}}/>
-
-+
-title: ExpectedResult
-
-<p>Special Content,Default Content</p>

editions/test/tiddlers/tests/data/functions/FunctionSubstituteResolution.tid

@@ -1,38 +0,0 @@
-title: Functions/FunctionResolutionInSubstitute
-description: Functions should resolve correctly in the substitute operator
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-
-\function getIndex() [<index>add[1]] 
-
-\procedure template-with-var() $(getIndex)$
-
-\procedure template-with-filteredexpression() ${ [<getIndex>] }$
-
-\function test-with-substitute-variable()
-[[abc]split[]] :map[<template-with-var>substitute[]] :and[join[ / ]]
-\end
-
-\function test-with-substitute-filteredexpression()
-[[abc]split[]] :map[<template-with-filteredexpression>substitute[]] :and[join[ / ]]
-\end
-
-\function test-with-function()
-[[abc]split[]] :map[function[getIndex]substitute[]] :and[join[ / ]]
-\end
-
-
-
-<<test-with-substitute-variable>>|
-<<test-with-substitute-filteredexpression>>|
-<<test-with-function>>
-
-
-+
-title: ExpectedResult
-
-<p>1 / 2 / 3|1 / 2 / 3|1 / 2 / 3</p>

editions/test/tiddlers/tests/data/multi-valued-variables/AttributeFirstValue.tid

@@ -1,16 +0,0 @@
-title: MultiValuedVariables/AttributeFirstValue
-description: ((var)) on non-MVV-aware widget attribute returns first value only
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-<$let items={{{ [all[tiddlers]sort[]] }}}>
-<$text text=((items))/>
-</$let>
-+
-title: ExpectedResult
-
-<p>$:/core</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/DefaultParameterMVV.tid

@@ -1,19 +0,0 @@
-title: MultiValuedVariables/DefaultParameterMVV
-description: Procedure default parameter value using ((var)) syntax to provide MVV default
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-\procedure showItems(itemList:((defaults)))
-<$text text={{{ [(itemList)join[-]] }}}/>
-\end
-<$let defaults={{{ [all[tiddlers]sort[]] }}}>
-<<showItems>>
-</$let>
-+
-title: ExpectedResult
-
-<p>$:/core-ExpectedResult-Output</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/InlineDisplay.tid

@@ -1,16 +0,0 @@
-title: MultiValuedVariables/InlineDisplay
-description: ((var)) in inline wikitext displays MVV with default comma-space separator
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-<$let items={{{ [all[tiddlers]sort[]] }}}>
-((items))
-</$let>
-+
-title: ExpectedResult
-
-<p>$:/core, ExpectedResult, Output</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/InlineDisplaySeparator.tid

@@ -1,16 +0,0 @@
-title: MultiValuedVariables/InlineDisplaySeparator
-description: ((var||separator)) in inline wikitext displays MVV with custom separator
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-<$let items={{{ [all[tiddlers]sort[]] }}}>
-((items||:))
-</$let>
-+
-title: ExpectedResult
-
-<p>$:/core:ExpectedResult:Output</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/InlineFilterDisplay.tid

@@ -1,14 +0,0 @@
-title: MultiValuedVariables/InlineFilterDisplay
-description: (((filter))) in inline wikitext displays filter results with default comma-space separator
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-((([all[tiddlers]sort[]])))
-+
-title: ExpectedResult
-
-<p>$:/core, ExpectedResult, Output</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/InlineFilterDisplaySeparator.tid

@@ -1,14 +0,0 @@
-title: MultiValuedVariables/InlineFilterDisplaySeparator
-description: (((filter||separator))) in inline wikitext displays filter results with custom separator
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-((([all[tiddlers]sort[]]||:)))
-+
-title: ExpectedResult
-
-<p>$:/core:ExpectedResult:Output</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/TranscludeParameter.tid

@@ -1,19 +0,0 @@
-title: MultiValuedVariables/TranscludeParameter
-description: Multi-valued variable passed as procedure parameter via ((var)) syntax
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-\procedure showItems(itemList)
-<$text text={{{ [(itemList)join[-]] }}}/>
-\end
-<$let items={{{ [all[tiddlers]sort[]] }}}>
-<$transclude $variable="showItems" itemList=((items))/>
-</$let>
-+
-title: ExpectedResult
-
-<p>$:/core-ExpectedResult-Output</p>
-+

editions/test/tiddlers/tests/data/multi-valued-variables/TranscludeParameterFunction.tid

@@ -1,17 +0,0 @@
-title: MultiValuedVariables/TranscludeParameterFunction
-description: Multi-valued variable passed as function parameter via ((var)) in $transclude
-type: text/vnd.tiddlywiki-multiple
-tags: [[$:/tags/wiki-test-spec]]
-
-title: Output
-
-\whitespace trim
-\function showItems(itemList) [(itemList)sort[]join[-]]
-<$let items={{{ [all[tiddlers]] }}}>
-<$transclude $variable="showItems" itemList=((items))/>
-</$let>
-+
-title: ExpectedResult
-
-<p>$:/core-ExpectedResult-Output</p>
-+

editions/test/tiddlers/tests/test-widget.js

@@ -177,32 +177,6 @@ describe("Widget module", function() {
 		expect(wrapper.innerHTML).toBe("<span class=\"tc-error\">Recursive transclusion error in transclude widget</span> <span class=\"tc-error\">Recursive transclusion error in transclude widget</span>");
 	});
 
-	// Do NOT use a for-of or for-in here. Each jasmine test must be
-	// defined in its own function context, or the `tag` variable will
-	// end up being the same value for all iterations of the test. 
-	$tw.utils.each(["div","$button","$checkbox","$diff-text","$draggable","$droppable","dropzone","$eventcatcher","$keyboard","$link","$list filter=x variable=x","$radio","$reveal type=nomatch","$scrollable","$select","$view field=x"],function(tag) {
-		it(`${tag} cleans itself up if children rendering fails`, function() {
-			var wiki = new $tw.Wiki();
-			wiki.addTiddler({title: "TiddlerOne", text: `<$tiddler tiddler='TiddlerOne'><${tag}><$transclude />`});
-			var parseTreeNode = {type: "widget", children: [
-				{type: "transclude", attributes: {
-					"tiddler": {type: "string", value: "TiddlerOne"}
-				}}
-			]};
-			// Construct the widget node
-			var widgetNode = createWidgetNode(parseTreeNode,wiki);
-			// Render the widget node to the DOM
-			var wrapper = renderWidgetNode(widgetNode);
-			// We don't actually care exactly what the HTML contains,
-			// only that it's reasonably sized. If it's super large,
-			// that means the widget containing the bad transclusion
-			// didn't figure out how to clean itself up, and it cloned a bunch.
-			var html = wrapper.innerHTML;
-			expect(html).toContain("Recursive transclusion error in transclude widget");
-			expect(html.length).toBeLessThan(256, "CONTENTS: " + html);
-		});
-	});
-
 	it("should handle many-tiddler recursion with branching nodes", function() {
 		var wiki = new $tw.Wiki();
 		// Add a tiddler

editions/translators/tiddlers/Instructions for Translators.tid

@@ -48,7 +48,6 @@ Prioritise the groups to translate first. A translation can be useful without be
 * Buttons
 * Control Panel
 * Date
-* Draft
 * Edit Template
 * Getting Started
 * Import

editions/translators/tiddlers/system/$__status_UserName.tid

@@ -1,3 +0,0 @@
-title: $:/status/UserName
-
-Jermolene

editions/translators/tiddlywiki.info

@@ -51,7 +51,6 @@
 			"--render","$:/plugins/tiddlywiki/translators/templates/ControlPanel.multids","language/ControlPanel.multids","text/plain",
 			"--render","$:/plugins/tiddlywiki/translators/templates/CoreReadMe.tid","language/CoreReadMe.tid","text/plain",
 			"--render","$:/plugins/tiddlywiki/translators/templates/Dates.multids","language/Dates.multids","text/plain",
-			"--render","$:/plugins/tiddlywiki/translators/templates/Draft.multids","language/Draft.multids","text/plain",
 			"--render","$:/plugins/tiddlywiki/translators/templates/EditTemplate.multids","language/EditTemplate.multids","text/plain",
 			"--render","$:/plugins/tiddlywiki/translators/templates/Exporters.multids","language/Exporters.multids","text/plain",
 			"--render","$:/plugins/tiddlywiki/translators/templates/Fields.multids","language/Fields.multids","text/plain",

editions/tw5.com/tiddlers/Core CSS Variables.tid

@@ -1,11 +0,0 @@
-created: 20251108075028089
-modified: 20260130070027124
-tags: Reference
-title: Core CSS Variables
-type: text/vnd.tiddlywiki
-
-<<.from-version 5.4.0>> Tiddlywiki CSS variable definitions starts with `--tp-*` and `--tpc-*`. They are mainly used to [[Write stylesheets in vanilla CSS|Writing stylesheets in vanilla CSS]]. These prefixes ''are reserved'' for Tiddlywiki, so it should not be used for user defined CSS variables. It is also not recommended to override these core CSS variables.
-
-Core CSS variables are defined in [[$:/core/stylesheets/custom-properties]].
-
-<<list-links "[tag[Core CSS Variables]]" class:"multi-columns">>

editions/tw5.com/tiddlers/Writing stylesheets in vanilla CSS.tid

@@ -1,69 +0,0 @@
-created: 20251108075645447
-modified: 20260201043953311
-title: Writing stylesheets in vanilla CSS
-type: text/vnd.tiddlywiki
-
-<<.from-version 5.4.0>> Before v5.4.0, theme developers have to mix wikitext syntax with CSS syntax when writing stylesheets to intergrate Tiddlywiki color palettes and theme settings. With the introduction of [[Core CSS Variables]] in v5.4.0, theme developers can intergrate most Tiddlywiki palettes with vanilla CSS.
-
-! Getting Tiddlywiki palette colors
-Tiddlywiki's custom properties for colors are prefixed `--tpc-`. Before v5.4.0, theme developers have to use the following wikitext to get a color value of a palette:
-
-```
-.tag {
-	background: <<colour tag-background>>;
-}
-```
-
-Since v5.4.0, theme developers can use the following CSS to get the palette color:
-
-```css
-.tag {
-	background: var(--tp-color-tag-background);
-}
-```
-
-! Getting and processing Tiddlywiki CSS settings
-See [[Core CSS Variables]] for the available CSS variables. Before v5.4.0, theme developers have to use macros with filters to get and process theme settings:
-
-```
-.tc-sidebar-header {
-	padding: 14px;
-	min-height: 32px;
-	margin-top: {{$:/themes/tiddlywiki/vanilla/metrics/storytop}};
-	transition:  min-height {{$:/config/AnimationDuration}}ms ease-in-out, padding-top {{$:/config/AnimationDuration}}ms ease-in-out, padding-bottom {{$:/config/AnimationDuration}}ms ease-in-out;
-}
-```
-
-Since v5.4.0, the theme settings are also available as the CSS variables, so theme developers can use the following code:
-
-```css
-.tc-sidebar-header {
-	padding: 14px;
-	min-height: 32px;
-	margin-top: var(--tp-story-top);
-	transition: min-height var(--tp-animation-duration) ease-in-out, padding-top var(--tp-animation-duration) ease-in-out, padding-bottom var(--tp-animation-duration) ease-in-out;
-}
-```
-
-! Limits
-CSS variables can only be used in rules, while wikitext can be used everywhere. See this example:
-
-Old way of using wikitext in media query definitions:
-
-```
-\define sidebarbreakpoint()
-<$text text={{$:/themes/tiddlywiki/vanilla/metrics/sidebarbreakpoint}}/>
-\end
-
-@media (min-width: <<sidebarbreakpoint>>) {
-	/* CSS rules */
-}
-```
-
-While using CSS variables in media quert definitions doesn't work at all:
-
-```css
-@media (min-width: var(--tp-sidebar-breakpoint)) {
-	/* Doesn't work */
-}
-```

editions/tw5.com/tiddlers/_tw_shared/doc-utilities/doc-styles.tid

@@ -193,11 +193,11 @@ tr.doc-table-subheading {
 }
 
 .doc-block-icon .tc-image-tip {
-	color: <<colour primary>>;
+	fill: <<colour primary>>;
 }
 
 .doc-block-icon .tc-image-warning {
-	color: <<colour alert-highlight>>;
+	fill: <<colour alert-highlight>>;
 }
 
 a.doc-from-version {
@@ -215,6 +215,7 @@ a.doc-from-version.doc-from-version-new {
 }
 
 a.doc-from-version svg {
+    fill: currentColor;
     vertical-align: sub;
 }
 
@@ -223,6 +224,7 @@ a.doc-deprecated-version.tc-tiddlylink {
     border-radius: 1em;
 	background: red;
 	color: <<colour background>>;
+	fill: <<colour background>>;
     padding: 0 0.4em;
     font-size: 0.7em;
     text-transform: uppercase;

editions/tw5.com/tiddlers/cards/card-styles.tid

@@ -36,6 +36,7 @@ type: text/vnd.tiddlywiki
 	border-radius: 6px;
 	padding: 0.5em;
 	color: <<colour background>>;
+	fill: <<colour background>>;
 }
 
 .tc-cards.tc-action-card .tc-card-button svg {

editions/tw5.com/tiddlers/css-variables/--tp-animation-duration CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-animation-duration
-created: 20260130075755351
-modified: 20260130075855780
-tags: [[Core CSS Variables]]
-title: --tp-animation-duration CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-animation-duration` CSS variable represents the "Animation duration" setting in the control panel. The `ms` suffix is added.

editions/tw5.com/tiddlers/css-variables/--tp-body-font-size Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-body-font-size
-created: 20260122111308483
-modified: 20260130045633904
-tags: [[Core CSS Variables]]
-title: --tp-body-font-size CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-body-font-size` CSS variable represents the "Font size for tiddler body" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-body-line-height Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-body-line-height
-created: 20260122111249446
-modified: 20260130045720240
-tags: [[Core CSS Variables]]
-title: --tp-body-line-height CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-body-line-height` CSS variable represents the "Line height for tiddler body" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-code-font-family Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-code-font-family
-created: 20260122110850673
-modified: 20260130045333134
-tags: [[Core CSS Variables]]
-title: --tp-code-font-family CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-code-font-family` CSS variable represents the "Code font family" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-code-wrapping Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-code-wrapping
-created: 20260122110618231
-modified: 20260130045345387
-tags: [[Core CSS Variables]]
-title: --tp-code-wrapping CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-code-wrapping` CSS variable represents the "Wrap long lines in code blocks" setting in Theme Tweaks. Its value is `pre` when the setting is set to "No", and `pre-wrap` when set to "Yes".

editions/tw5.com/tiddlers/css-variables/--tp-editor-font-family Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-editor-font-family
-created: 20260122110858791
-modified: 20260130045422071
-tags: [[Core CSS Variables]]
-title: --tp-editor-font-family CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-editor-font-family` CSS variable represents the "Editor font family" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-font-family Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-font-family
-created: 20260122110741282
-modified: 20260130045301357
-tags: [[Core CSS Variables]]
-title: --tp-font-family CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-font-family` CSS variable represents the "Font family" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-font-size Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-font-size
-created: 20260122110956954
-modified: 20260130045455466
-tags: [[Core CSS Variables]]
-title: --tp-font-size CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-font-size` CSS variable represents the "Font size" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-line-height Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-line-height
-created: 20260122111115266
-modified: 20260130045757204
-tags: [[Core CSS Variables]]
-title: --tp-line-height CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-line-height` CSS variable represents the "Line height" setting in Theme Tweaks.

editions/tw5.com/tiddlers/css-variables/--tp-sidebar-breakpoint CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-sidebar-breakpoint
-created: 20260130065756457
-modified: 20260130065841355
-tags: [[Core CSS Variables]]
-title: --tp-sidebar-breakpoint CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-sidebar-breakpoint` CSS variable represents the minimum page width at which the story river and sidebar will appear side by side.

editions/tw5.com/tiddlers/css-variables/--tp-sidebar-width CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-sidebar-width
-created: 20260130065850825
-modified: 20260130065925595
-tags: [[Core CSS Variables]]
-title: --tp-sidebar-width CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-sidebar-width` CSS variable represents the width of the sidebar in fluid-fixed layout.

editions/tw5.com/tiddlers/css-variables/--tp-story-left CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-story-left
-created: 20260130065252380
-modified: 20260130065342364
-tags: [[Core CSS Variables]]
-title: --tp-story-left CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-story-left` CSS variable represents how far the left margin of the story river (tiddler area) is from the left of the page.

editions/tw5.com/tiddlers/css-variables/--tp-story-right CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-story-right
-created: 20260130065417448
-modified: 20260130065457358
-tags: [[Core CSS Variables]]
-title: --tp-story-right CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-story-right` CSS variable represents how far the left margin of the sidebar is from the left of the page.

editions/tw5.com/tiddlers/css-variables/--tp-story-top CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-story-top
-created: 20260130065351753
-modified: 20260130065451132
-tags: [[Core CSS Variables]]
-title: --tp-story-top CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-story-top` CSS variable represents how far the top margin of the story river is from the top of the page.

editions/tw5.com/tiddlers/css-variables/--tp-story-width CSS Variable.tid

@@ -1,8 +0,0 @@
-caption: --tp-story-width
-created: 20260130065510314
-modified: 20260130065552027
-tags: [[Core CSS Variables]]
-title: --tp-story-width CSS Variable
-type: text/vnd.tiddlywiki
-
-The `--tp-story-width` CSS variable represents the overall width of the story river.