Module:Gallery/sandbox - Revision history

Arthurfragoso: Automatically detecting if in main or Sandbox. Reducing the difference between versions helps working with it and prevents mistakes. (I also tested on a local mediawiki)

2025-02-13T22:38:33Z

Automatically detecting if in main or Sandbox. Reducing the difference between versions helps working with it and prevents mistakes. (I also tested on a local mediawiki)

← Previous revision		Revision as of 22:38, 13 February 2025
Line 3:		Line 3:
	local p = {}		local p = {}

			local moduleName = mw.getCurrentFrame():getTitle()
⚫	local templatestyles = 'Module:Gallery/sandbox/styles.css'
			local isSandbox = mw.ustring.match(moduleName, "sandbox") ~= nil
	-- local templatestyles = 'Module:Gallery/styles.css'

		⚫	local templatestyles = 'Module:Gallery'.. (isSandbox and '/sandbox' or '') ..'/styles.css'
			local cssRootClass = 'mod-gallery' .. (isSandbox and '-sb' or '')

	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')
	local plaintextModule = require('Module:Plain text')		local plaintextModule = require('Module:Plain text')
Line 101:		Line 105:

	local tbl = mw.html.create('div')		local tbl = mw.html.create('div')
	tbl:addClass(~~'mod-gallery-sb'~~)		tbl:addClass(cssRootClass)
		⚫
	--tbl:addClass('mod-gallery')

	if args.state then		if args.state then
	tbl		tbl
	:addClass('~~mod-gallery~~-collapsible')		:addClass(cssRootClass .. '-collapsible')
	:addClass('collapsible')		:addClass('collapsible')
	:addClass(args.state)		:addClass(args.state)
Line 114:		Line 117:
	tbl:cssText(args.style)		tbl:cssText(args.style)
	else		else
	tbl:addClass('~~mod-gallery~~-default')		tbl:addClass(cssRootClass .. '-default')
	end		end

	if args.align then		if args.align then
	tbl:addClass('~~mod-gallery~~-' .. args.align:lower())		tbl:addClass(cssRootClass .. '-' .. args.align:lower())
	end		end

Line 214:		Line 217:
	end		end
	end		end
⚫

	tbl:tag('div')		tbl:tag('div')

Arthurfragoso: Fixing reported problem when using external links in the caption

2025-02-10T10:07:16Z

Fixing reported problem when using external links in the caption

← Previous revision		Revision as of 10:07, 10 February 2025
Line 3:		Line 3:
	local p = {}		local p = {}

	local templatestyles = 'Module:Gallery/styles.css'		local templatestyles = 'Module:Gallery/sandbox/styles.css'
			-- local templatestyles = 'Module:Gallery/styles.css'
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')
			local plaintextModule = require('Module:Plain text')

			local function plaintext(text)
			-- stips out external links without labels,
			-- and then passes to the Plain_text module to clean the rest
			return plaintextModule.main({ args = {
			text:gsub("([^%[])%[([^%[%]%s]+)%]", '%1')
			, encode = "no" } })
			end

	local function trim(s)		local function trim(s)
Line 16:		Line 26:

	-- Check if it starts with "File:", "Image:", or "Media:"		-- Check if it starts with "File:", "Image:", or "Media:"
	prefix = file:match("^(%a+):")		local prefix = file:match("^(%a+):")
	if prefix and (prefix == "file" or prefix == "image" or prefix == "media") then		if prefix and (prefix == "file" or prefix == "image" or prefix == "media") then
	return true		return true
Line 91:		Line 101:

	local tbl = mw.html.create('div')		local tbl = mw.html.create('div')
	tbl:addClass('mod-gallery')		tbl:addClass('mod-gallery-sb')
			--tbl:addClass('mod-gallery')


	if args.state then		if args.state then
	tbl		tbl
Line 174:		Line 185:
	-- It is necessary because we add classes codes in the next step,		-- It is necessary because we add classes codes in the next step,
	-- we do not want to let the alt empty with just the non-printing characters.		-- we do not want to let the alt empty with just the non-printing characters.
	alt = caption		alt = (caption ~= '') and plaintext(caption) or ''
	end		end

Line 187:		Line 198:
	end		end

			-- Some space between the arguments and the pipe to prevent unexpected behaviors.
			-- for example: in some cases the parser interpret the pipe as part of urls
	table.insert(gallery, img ..		table.insert(gallery, img ..
	(alt ~= '' and ('\|alt=' .. alt) or '') ..		(alt ~= '' and (' \|alt=' .. alt) or '') ..
	(caption ~= '' and ('\|' .. caption) or ''))		(caption ~= '' and (' \|' .. caption) or ''))
	end		end

Line 201:		Line 214:
	end		end
	end		end


	tbl:tag('div')		tbl:tag('div')

Jonesey95: test old er revision

2025-02-10T02:31:53Z

test old er revision

← Previous revision		Revision as of 02:31, 10 February 2025
Line 3:		Line 3:
	local p = {}		local p = {}

	local templatestyles = 'Module:Gallery~~/sandbox~~/styles.css'		local templatestyles = 'Module:Gallery/styles.css'
	-- local templatestyles = 'Module:Gallery/styles.css'
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')
	local plaintextModule = require('Module:Plain text')

	local function plaintext(text)
	return plaintextModule.main({ args = { text, encode = "no" } })
	end

	local function trim(s)		local function trim(s)
Line 22:		Line 16:

	-- Check if it starts with "File:", "Image:", or "Media:"		-- Check if it starts with "File:", "Image:", or "Media:"
	~~local~~ prefix = file:match("^(%a+):")		prefix = file:match("^(%a+):")
	if prefix and (prefix == "file" or prefix == "image" or prefix == "media") then		if prefix and (prefix == "file" or prefix == "image" or prefix == "media") then
	return true		return true
Line 97:		Line 91:

	local tbl = mw.html.create('div')		local tbl = mw.html.create('div')
	tbl:addClass('mod-gallery~~-sb~~')		tbl:addClass('mod-gallery')

	--tbl:addClass('mod-gallery')

	if args.state then		if args.state then
	tbl		tbl
Line 181:		Line 174:
	-- It is necessary because we add classes codes in the next step,		-- It is necessary because we add classes codes in the next step,
	-- we do not want to let the alt empty with just the non-printing characters.		-- we do not want to let the alt empty with just the non-printing characters.
	alt = (caption ~~~= '') and plaintext(caption) or ''~~		alt = caption
	end		end

Line 208:		Line 201:
	end		end
	end		end


	tbl:tag('div')		tbl:tag('div')

Arthurfragoso: Text.getPlain don't strips wikilinks and images. But the Module:Plain text does the desired function

2025-02-07T20:02:35Z

Text.getPlain don't strips wikilinks and images. But the Module:Plain text does the desired function

← Previous revision		Revision as of 20:02, 7 February 2025
Line 6:		Line 6:
	-- local templatestyles = 'Module:Gallery/styles.css'		-- local templatestyles = 'Module:Gallery/styles.css'
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')
	local ~~Text~~ = require('Module:~~Text~~')		local plaintextModule = require('Module:Plain text')

	Text = Text.Text()
			local function plaintext(text)
			return plaintextModule.main({ args = { text, encode = "no" } })
			end

	local function trim(s)		local function trim(s)
Line 19:		Line 22:

	-- Check if it starts with "File:", "Image:", or "Media:"		-- Check if it starts with "File:", "Image:", or "Media:"
	prefix = file:match("^(%a+):")		local prefix = file:match("^(%a+):")
	if prefix and (prefix == "file" or prefix == "image" or prefix == "media") then		if prefix and (prefix == "file" or prefix == "image" or prefix == "media") then
	return true		return true
Line 49:		Line 52:
	if k and type(k) == 'string' then		if k and type(k) == 'string' then
	if k == 'align' or k == 'state' or k == 'style' or k == 'title' or		if k == 'align' or k == 'state' or k == 'style' or k == 'title' or
	k == 'width' or k == 'height~~' or k == 'lines~~' or k == 'whitebg' or		k == 'width' or k == 'height' or k == 'whitebg' or
	k == 'mode' or k == 'footer' or k == 'perrow' or k == 'noborder' or		k == 'mode' or k == 'footer' or k == 'perrow' or k == 'noborder' or
	k:match('^alt%d+$') or k:match('^class%d+$') or k:match('^%d+$') then		k:match('^alt%d+$') or k:match('^class%d+$') or k:match('^%d+$') then
Line 72:		Line 75:
	-- Otherwise, for testing purposes, assume args are being passed directly in.		-- Otherwise, for testing purposes, assume args are being passed directly in.
	local origArgs = (type(frame.getParent) == 'function') and frame:getParent().args or frame		local origArgs = (type(frame.getParent) == 'function') and frame:getParent().args or frame


	-- ParserFunctions considers the empty string to be false, so to preserve the previous		-- ParserFunctions considers the empty string to be false, so to preserve the previous
	-- behavior of {{gallery}}, change any empty arguments to nil, so Lua will consider		-- behavior of {{gallery}}, change any empty arguments to nil, so Lua will consider
	-- them false too.		-- them false too.
Line 162:		Line 165:
	end		end

	local altCount = 0; ~~-- remove after migration~~		local altCount = 0;

	-- Run through virtualgallery and builds gallery		-- Run through virtualgallery and builds gallery
Line 178:		Line 181:
	-- It is necessary because we add classes codes in the next step,		-- It is necessary because we add classes codes in the next step,
	-- we do not want to let the alt empty with just the non-printing characters.		-- we do not want to let the alt empty with just the non-printing characters.
	alt = (caption ~= '') and ~~Text.getPlain~~(caption) or ''		alt = (caption ~= '') and plaintext(caption) or ''
	end		end

Line 213:		Line 216:
	frame:extensionTag{ name = 'gallery', content = '\n' .. table.concat(gallery,'\n'), args = gargs}		frame:extensionTag{ name = 'gallery', content = '\n' .. table.concat(gallery,'\n'), args = gargs}
	)		)


	if args.footer then		if args.footer then
	tbl:tag('div')		tbl:tag('div')

Arthurfragoso: Removes wikilinks and wikicode when adding the caption to the alt text

2025-02-07T16:49:31Z

Removes wikilinks and wikicode when adding the caption to the alt text

← Previous revision		Revision as of 16:49, 7 February 2025
Line 6:		Line 6:
	-- local templatestyles = 'Module:Gallery/styles.css'		-- local templatestyles = 'Module:Gallery/styles.css'
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')
			local Text = require('Module:Text')
			Text = Text.Text()

	local function trim(s)		local function trim(s)
Line 176:		Line 178:
	-- It is necessary because we add classes codes in the next step,		-- It is necessary because we add classes codes in the next step,
	-- we do not want to let the alt empty with just the non-printing characters.		-- we do not want to let the alt empty with just the non-printing characters.
	alt = caption		alt = (caption ~= '') and Text.getPlain(caption) or ''
	end		end

Arthurfragoso: opting for the non-joiner character

2025-02-06T18:52:49Z

opting for the non-joiner character

← Previous revision		Revision as of 18:52, 6 February 2025
Line 136:		Line 136:

	local zwsp = string.char(0xE2, 0x80, 0x8B) -- U+200B Zero Width Space		local zwsp = string.char(0xE2, 0x80, 0x8B) -- U+200B Zero Width Space
	local ~~zwj~~ = string.char(0xE2, 0x80, ~~0x8D~~) -- U+~~200D~~ Zero Width Joiner		local zwnj = string.char(0xE2, 0x80, 0x8C) -- U+200C Zero Width Non-Joiner

	-- create a coding to identify classes		-- create a coding to identify classes
Line 144:		Line 144:

	local skininvert = zwsp .. zwsp .. zwsp;		local skininvert = zwsp .. zwsp .. zwsp;
	local bgtransparent = zwsp .. zwsp .. ~~zwj~~;		local bgtransparent = zwsp .. zwsp .. zwnj;

	for i = 1, #args do		for i = 1, #args do

Arthurfragoso: improving documentation

2025-02-06T16:55:17Z

improving documentation

← Previous revision		Revision as of 16:55, 6 February 2025
Line 139:		Line 139:

	-- create a coding to identify classes		-- create a coding to identify classes
			-- using unicode non-printing characters
			-- this is a workaround until we get the class arg in the <gallery> tag
			-- https://phabricator.wikimedia.org/T344784

	local skininvert = zwsp .. zwsp .. zwsp;		local skininvert = zwsp .. zwsp .. zwsp;
	local bgtransparent = zwsp .. zwsp .. zwj;		local bgtransparent = zwsp .. zwsp .. zwj;
Line 171:		Line 175:
	-- if alt is empty, we use the caption as a alt text.		-- if alt is empty, we use the caption as a alt text.
	-- It is necessary because we add classes codes in the next step,		-- It is necessary because we add classes codes in the next step,
	-- we do not want to let the alt empty with just the ~~classes~~.		-- we do not want to let the alt empty with just the non-printing characters.
	alt = caption		alt = caption
	end		end

			-- we attach the non-printing code to the end of the alt text
	if mw.ustring.find(class, 'skin%-invert') then -- this matches both skin-invert and skin-invert-image		if mw.ustring.find(class, 'skin%-invert') then -- this matches both skin-invert and skin-invert-image
	alt = alt .. skininvert		alt = alt .. skininvert

Arthurfragoso: adds classN for enabling darkmode skin-invert-image

2025-02-06T10:01:32Z

adds classN for enabling darkmode skin-invert-image

← Previous revision		Revision as of 10:01, 6 February 2025
Line 49:		Line 49:
	k == 'width' or k == 'height' or k == 'lines' or k == 'whitebg' or		k == 'width' or k == 'height' or k == 'lines' or k == 'whitebg' or
	k == 'mode' or k == 'footer' or k == 'perrow' or k == 'noborder' or		k == 'mode' or k == 'footer' or k == 'perrow' or k == 'noborder' or
	k:match('^alt%d+$') or k:match('^%d+$') then		k:match('^alt%d+$') or k:match('^class%d+$') or k:match('^%d+$') then
	-- valid		-- valid
	elseif k == 'captionstyle' then		elseif k == 'captionstyle' then
Line 134:		Line 134:

	local imageCount = 0		local imageCount = 0

			local zwsp = string.char(0xE2, 0x80, 0x8B) -- U+200B Zero Width Space
			local zwj = string.char(0xE2, 0x80, 0x8D) -- U+200D Zero Width Joiner

			-- create a coding to identify classes
			local skininvert = zwsp .. zwsp .. zwsp;
			local bgtransparent = zwsp .. zwsp .. zwj;

	for i = 1, #args do		for i = 1, #args do
Line 156:		Line 163:
	local caption = virtualgallery[n][2] or ''		local caption = virtualgallery[n][2] or ''
	local alt = trim(args['alt' .. n] or '')		local alt = trim(args['alt' .. n] or '')
			local class = trim(args['class' .. n] or '')

			-- we count alt text only before any modification
	--remove after migration
	if alt ~= '' then		if alt ~= '' then
	altCount = altCount + 1		altCount = altCount + 1
			else
			-- if alt is empty, we use the caption as a alt text.
			-- It is necessary because we add classes codes in the next step,
			-- we do not want to let the alt empty with just the classes.
			alt = caption
			end

			if mw.ustring.find(class, 'skin%-invert') then -- this matches both skin-invert and skin-invert-image
			alt = alt .. skininvert
			end

			-- as it is possible to combine multiple classes, we use find instead of ==
			if mw.ustring.find(class, 'bg%-transparent') then
			alt = alt .. bgtransparent
	end		end

Arthurfragoso: sandbox templatestyles

2025-02-06T09:48:01Z

sandbox templatestyles

← Previous revision		Revision as of 09:48, 6 February 2025
Line 3:		Line 3:
	local p = {}		local p = {}

	-- local templatestyles = 'Module:Gallery/sandbox/styles.css'		local templatestyles = 'Module:Gallery/sandbox/styles.css'
	local templatestyles = 'Module:Gallery/styles.css'		-- local templatestyles = 'Module:Gallery/styles.css'
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')

Line 92:		Line 92:

	local tbl = mw.html.create('div')		local tbl = mw.html.create('div')
	--tbl:addClass('mod-gallery-sb')		tbl:addClass('mod-gallery-sb')
	tbl:addClass('mod-gallery')		--tbl:addClass('mod-gallery')

	if args.state then		if args.state then

Arthurfragoso: missed ".tif" extension

2025-02-03T05:07:09Z

missed ".tif" extension

← Previous revision		Revision as of 05:07, 3 February 2025
Line 25:		Line 25:
	"apng", "djvu", "flac", "gif", "jfi", "jfif", "jif", "jpe", "jpeg", "jpg",		"apng", "djvu", "flac", "gif", "jfi", "jfif", "jif", "jpe", "jpeg", "jpg",
	"m1a", "m1v", "m2a", "m2v", "mid", "mp1", "mp2", "mp3", "mpa", "mpe", "mpeg", "mpg",		"m1a", "m1v", "m2a", "m2v", "mid", "mp1", "mp2", "mp3", "mpa", "mpe", "mpeg", "mpg",
	"mpv", "oga", "ogg", "ogv", "opus", "pdf", "png", "stl", "svg", "svgz", "tiff",		"mpv", "oga", "ogg", "ogv", "opus", "pdf", "png", "stl", "svg", "svgz", "tif", "tiff",
	"wav", "wave", "webm", "webp", "xcf"		"wav", "wave", "webm", "webp", "xcf"
	}		}