Module:Portal bar - Revision history

Hike395: make "related" aspect ratio match other portal icons

2025-03-08T16:25:17Z

make "related" aspect ratio match other portal icons

← Previous revision		Revision as of 16:25, 8 March 2025
Line 66:		Line 66:
	container:addClass(sandboxVersion('portal-bar-content-related'))		container:addClass(sandboxVersion('portal-bar-content-related'))
	end		end
	local size = related and '~~27x25px~~' or '21x19px'		local size = related and '28x24px' or '21x19px'
	for _, portal in ipairs( portals ) do		for _, portal in ipairs( portals ) do
	container		container

Hike395: better implementation of nbsp

2024-05-22T22:00:26Z

better implementation of nbsp

← Previous revision		Revision as of 22:00, 22 May 2024
Line 71:		Line 71:
	:tag( 'li' )		:tag( 'li' )
	:addClass(sandboxVersion('portal-bar-item'))		:addClass(sandboxVersion('portal-bar-item'))
	:wikitext( string.format('[[File:%s\|%s]]~~ ~~[[Portal:%s\|%s]]',		:wikitext( string.format('<span class="nowrap">[[File:%s\|%s]] </span>[[Portal:%s\|%s]]',
	getImageName(portal,true), size, portal, portal))		getImageName(portal,true), size, portal, portal))
	end		end

Hike395: use semantically-meaningful HTML tags

2022-12-28T15:09:16Z

use semantically-meaningful HTML tags

← Previous revision		Revision as of 15:09, 28 December 2022
Line 61:		Line 61:
	header:wikitext(':')		header:wikitext(':')
	end		end
	local container = nav:tag('~~div~~')		local container = nav:tag('ul')
	container:addClass(sandboxVersion('portal-bar-content'))		container:addClass(sandboxVersion('portal-bar-content'))
	if related then		if related then
Line 69:		Line 69:
	for _, portal in ipairs( portals ) do		for _, portal in ipairs( portals ) do
	container		container
	:tag( '~~span~~' )		:tag( 'li' )
	:addClass(sandboxVersion('portal-bar-item'))		:addClass(sandboxVersion('portal-bar-item'))
	:wikitext( string.format('[[File:%s\|%s]] [[Portal:%s\|%s]]',		:wikitext( string.format('[[File:%s\|%s]] [[Portal:%s\|%s]]',

Legoktm: Replace Module:No globals with require( "strict" )

2022-10-21T06:53:44Z

Replace Module:No globals with require( "strict" )

← Previous revision		Revision as of 06:53, 21 October 2022
Line 1:		Line 1:
	-- This module implements {{portal bar}}.		-- This module implements {{portal bar}}.

	require('~~Module:No globals~~')		require('strict')

	-- determine whether we're being called from a sandbox		-- determine whether we're being called from a sandbox

Hike395: implement related portals with larger icons

2022-06-17T04:37:48Z

implement related portals with larger icons

@@ Line 49: / Line 49: @@
 	end
-	local header = nav:tag('span')
+	local related = yesno(args.related)
-		header:wikitext('s')
+		local header = nav:tag('span')
 	end
 	local container = nav:tag('div')
 	container:addClass(sandboxVersion('portal-bar-content'))
 	for _, portal in ipairs( portals ) do
 		container
-			:tag( 'div' )
+			:tag( 'span' )
 			:addClass(sandboxVersion('portal-bar-item'))
 	end

Hike395: automatically call Module:Portal/sandbox in test + new interface to getImageName

2022-06-05T03:22:45Z

automatically call Module:Portal/sandbox in test + new interface to getImageName

← Previous revision		Revision as of 03:22, 5 June 2022
Line 3:		Line 3:
	require('Module:No globals')		require('Module:No globals')

		⚫	-- determine whether we're being called from a sandbox
⚫	local portalModule = require('Module:Portal')
		⚫	local isSandbox = mw.getCurrentFrame():getTitle():find('sandbox', 1, true)
		⚫	local sandbox = isSandbox and '/sandbox' or ''

		⚫	local portalModule = require('Module:Portal'..sandbox)
	local getImageName = portalModule._image		local getImageName = portalModule._image
	local checkPortals = portalModule._checkPortals		local checkPortals = portalModule._checkPortals
Line 10:		Line 14:
	local getArgs = require('Module:Arguments').getArgs		local getArgs = require('Module:Arguments').getArgs
	local p = {}		local p = {}

⚫	-- determine whether we're being called from a sandbox
⚫	local isSandbox = mw.getCurrentFrame():getTitle():find('sandbox', 1, true)
⚫	local sandbox = isSandbox and '/sandbox' or ''

	local function sandboxVersion(s)		local function sandboxVersion(s)
Line 65:		Line 65:
	:addClass(sandboxVersion('portal-bar-logo'))		:addClass(sandboxVersion('portal-bar-logo'))
	:wikitext( string.format(		:wikitext( string.format(
	'[[File:%s\|21x19px]]', getImageName({ portal },true)		'[[File:%s\|21x19px]]', getImageName(portal,true)
	) )		) )
	:done()		:done()

Hike395: use border, alt, and link from Module:Portal/images

2022-06-04T05:21:42Z

use border, alt, and link from Module:Portal/images

← Previous revision		Revision as of 05:21, 4 June 2022
Line 4:		Line 4:

	local portalModule = require('Module:Portal')		local portalModule = require('Module:Portal')
	local getImageName = portalModule.~~image~~		local getImageName = portalModule._image
	local checkPortals = portalModule._checkPortals		local checkPortals = portalModule._checkPortals
	local processPortalArgs = portalModule._processPortalArgs		local processPortalArgs = portalModule._processPortalArgs
Line 65:		Line 65:
	:addClass(sandboxVersion('portal-bar-logo'))		:addClass(sandboxVersion('portal-bar-logo'))
	:wikitext( string.format(		:wikitext( string.format(
	'[[File:%s\|21x19px~~\|alt=~~]]', getImageName{ portal }		'[[File:%s\|21x19px]]', getImageName({ portal },true)
	) )		) )
	:done()		:done()

Hike395: auto-detect sandbox

2022-03-30T05:12:04Z

auto-detect sandbox

← Previous revision		Revision as of 05:12, 30 March 2022
Line 11:		Line 11:
	local p = {}		local p = {}

			-- determine whether we're being called from a sandbox
⚫	local function ~~sandbox~~(~~args,~~ s)
			local isSandbox = mw.getCurrentFrame():getTitle():find('sandbox', 1, true)
⚫	return ~~args.sandbox~~ and s.."-sand" or s
			local sandbox = isSandbox and '/sandbox' or ''

		⚫	local function sandboxVersion(s)
		⚫	return isSandbox and s.."-sand" or s
	end		end

Line 22:		Line 26:

	-- Normalize arguments		-- Normalize arguments
	for key, default in pairs({border=true,redlinks~~=false,sandbox~~=false,tracking=true}) do		for key, default in pairs({border=true,redlinks=false,tracking=true}) do
	if args[key] == nil then args[key] = default end		if args[key] == nil then args[key] = default end
	args[key] = yesno(args[key], default)		args[key] = yesno(args[key], default)
Line 28:		Line 32:

	local nav = mw.html.create( 'div' )		local nav = mw.html.create( 'div' )
	:addClass(~~sandbox~~(~~args,~~'portal-bar'))		:addClass(sandboxVersion('portal-bar'))
	:addClass( 'noprint metadata noviewer' )		:addClass( 'noprint metadata noviewer' )
	:attr( 'role', 'navigation' )		:attr( 'role', 'navigation' )
	:attr( 'aria-label' , 'Portals' )		:attr( 'aria-label' , 'Portals' )
	:addClass(~~sandbox~~(~~args,~~args.border and 'portal-bar-bordered' or 'portal-bar-unbordered'))		:addClass(sandboxVersion(args.border and 'portal-bar-bordered' or 'portal-bar-unbordered'))

	local trackingCat = ''		local trackingCat = ''
Line 46:		Line 50:

	local header = nav:tag('span')		local header = nav:tag('span')
	header:addClass(~~sandbox~~(~~args,~~'portal-bar-header'))		header:addClass(sandboxVersion('portal-bar-header'))
	header:wikitext('[[Wikipedia:Contents/Portals\|Portal]]')		header:wikitext('[[Wikipedia:Contents/Portals\|Portal]]')
	if #portals > 1 then		if #portals > 1 then
Line 53:		Line 57:
	header:wikitext(':')		header:wikitext(':')
	local container = nav:tag('div')		local container = nav:tag('div')
	container:addClass(~~sandbox~~(~~args,~~'portal-bar-content'))		container:addClass(sandboxVersion('portal-bar-content'))
	for _, portal in ipairs( portals ) do		for _, portal in ipairs( portals ) do
	container		container
	:tag( 'div' )		:tag( 'div' )
	:addClass(~~sandbox~~(~~args,~~'portal-bar-item'))		:addClass(sandboxVersion('portal-bar-item'))
	:tag( 'span' )		:tag( 'span' )
	:addClass(~~sandbox~~(~~args,~~'portal-bar-logo'))		:addClass(sandboxVersion('portal-bar-logo'))
	:wikitext( string.format(		:wikitext( string.format(
	'[[File:%s\|21x19px\|alt=]]', getImageName{ portal }		'[[File:%s\|21x19px\|alt=]]', getImageName{ portal }
Line 65:		Line 69:
	:done()		:done()
	:tag('span')		:tag('span')
	:addClass(~~sandbox~~(~~args,~~'portal-bar-link'))		:addClass(sandboxVersion('portal-bar-link'))
	:wikitext( string.format('[[Portal:%s\|%s]]', portal, portal))		:wikitext( string.format('[[Portal:%s\|%s]]', portal, portal))
	end		end

	local styleFile = 'Module:Portal bar/'		local styleFile = 'Module:Portal bar'..sandbox..'/styles.css'
	styleFile = styleFile..(args.sandbox and 'sandbox/' or '')
	styleFile = styleFile..'styles.css'
	return mw.getCurrentFrame():extensionTag{		return mw.getCurrentFrame():extensionTag{
	name = 'templatestyles', args = { src = styleFile }		name = 'templatestyles', args = { src = styleFile }

Hike395: factor out portal checking into Module:Portal

2022-01-03T06:56:58Z

factor out portal checking into Module:Portal

@@ Line 5: / Line 5: @@
 local portalModule = require('Module:Portal')
 local getImageName = portalModule.image
-local checkTracking = portalModule._checkTracking
+local checkPortals = portalModule._checkPortals
 local processPortalArgs = portalModule._processPortalArgs
 local yesno = require( 'Module:Yesno' )
@@ Line 17: / Line 17: @@
 -- Builds the portal bar used by {{portal bar}}.
 function p._main( portals, args )
 	-- check for sensible args
@@ Line 37: / Line 34: @@
 		:addClass(sandbox(args,args.border and 'portal-bar-bordered' or 'portal-bar-unbordered'))
 	local header = nav:tag('span')
 	header:addClass(sandbox(args,'portal-bar-header'))
@@ Line 87: / Line 74: @@
 	return mw.getCurrentFrame():extensionTag{
 		name = 'templatestyles', args = { src = styleFile }
-	} .. tostring( nav ) .. trackingCats
+	} .. tostring( nav )
 end

Hike395: factor duplicate code out into Module:Portal

2021-12-22T23:27:17Z

factor duplicate code out into Module:Portal

@@ Line 3: / Line 3: @@
 require('Module:No globals')
-local getImageName = require( 'Module:Portal' ).image
+local portalModule = require('Module:Portal')
 local yesno = require( 'Module:Yesno' )
 local getArgs = require('Module:Arguments').getArgs
@@ Line 10: / Line 13: @@
 local function sandbox(args, s)
 	return args.sandbox and s.."-sand" or s
 end
@@ Line 44: / Line 19: @@
 	-- Don't display a blank navbox if no portals were specified.
-	if #portals < 1 then return '' end
+	if (not portals) or (#portals < 1) then return '' end
 	-- Normalize arguments
-	args.redlinks = yesno(args.redlinks)
+		if args[key] == nil then args[key] = default end
 	local nav = mw.html.create( 'div' )
@@ Line 55: / Line 35: @@
 		:attr( 'role', 'navigation' )
 		:attr( 'aria-label' , 'Portals' )
-	local trackingEnabled = true
+	local trackingEnabled = args.tracking and checkTracking()
 	-- scan for nonexistent portals, if they exist remove them from the portals
 	-- table. If redlinks=yes, then don't remove
 	local trackingCats = ''
-	if not args.redlinks then
+	if not args.redlinks or trackingEnabled then
 		local existingPortals = {}
 		for _, portal in ipairs(portals) do
@@ Line 83: / Line 52: @@
 			end
 		end
-		if #existingPortals == 0 then
+		if #portals == 0 then
 			return trackingEnabled and "[[Category:Portal templates with all redlinked portals]]" or ""
 		end
 	end
@@ Line 124: / Line 93: @@
 function p.main( frame )
 	local origArgs = getArgs(frame)
 	return p._main( portals, args )
 end