mediawiki-skins-MinervaNeue/resources/skins.minerva.scripts/page-issues/page/PageIssueLearnMoreLink.js

/**
 * Creates a "read more" button with given text.
 *
 * @internal
 * @ignore
 * @param {string} msg
 * @return {jQuery}
 */
function newPageIssueLearnMoreLink( msg ) {
	return $( '<span>' )
		.addClass( 'ambox-learn-more' )
		.text( msg );
}

module.exports = newPageIssueLearnMoreLink;
Remove redundant closure for all modules with packageFiles Modules loaded with packageFiles are always executed in module scope (with a closure), even in debug mode. The behaviour of non-packageFiles debug mode is the only reason files have closures. https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Closure Change-Id: I14ac680eb67d892618bbc13be9298ec9b8e0e2e9 2024-08-30 10:49:35 +00:00			`/**`
			`* Creates a "read more" button with given text.`
			`*`
			`* @internal`
			`* @ignore`
			`* @param {string} msg`
			`* @return {jQuery}`
			`*/`
			`function newPageIssueLearnMoreLink( msg ) {`
			`return $( '<span>' )`
			`.addClass( 'ambox-learn-more' )`
			`.text( msg );`
			`}`
Hygiene: separate page issue view logic - Move page issue view components that do not modify the DOM during during construction to PageIssueLearnMoreLink.js and PageIssueLink.js. PascalCase is used optimistically for filenaming in the hopes that these functions can become something like a JSX component. A "new" function prefix is used in the meantime. - Move page issue view logic that munges the existing DOM to pageIssueFormatter.js. Substitute "create" prefixes for insert so that clients won't forget that calling the function is a modify operation. Alternative naming welcome but it shouldn't be confused with more idealistic components that do not depend on DOM state for construction. - Consolidate createPageIssueBanner() and createPageIssueBannerMultiple() into insertPageIssueBanner() as the code was quite similar and were it a true component, it would probably be a single component. All new files appear under page/ to keep their distinction from the overlay code clear. Some view logic remains in pageIssues.js but it shall be difficult to isolate. Bug: T212376 Change-Id: Iccce709c34fa8de5a28a5a00098add5775e3dc9a 2019-01-23 20:16:11 +00:00
Remove redundant closure for all modules with packageFiles Modules loaded with packageFiles are always executed in module scope (with a closure), even in debug mode. The behaviour of non-packageFiles debug mode is the only reason files have closures. https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Closure Change-Id: I14ac680eb67d892618bbc13be9298ec9b8e0e2e9 2024-08-30 10:49:35 +00:00			`module.exports = newPageIssueLearnMoreLink;`