Module:List - Revision history

Kalnook: 29 revisions imported

2018-07-14T20:04:09Z

29 revisions imported

Sunspots: 1 revision imported: Importing Infobox

2017-12-16T21:40:43Z

1 revision imported: Importing Infobox

Jdlrobson: Minerva skin needs whether an hlist is separated to be explicit (see https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Hlist_bullets_not_shown_in_mobile)

2017-08-08T13:55:25Z

Minerva skin needs whether an hlist is separated to be explicit (see https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Hlist_bullets_not_shown_in_mobile)

Jc86035: rm for now, could be unexpected behaviour? should wait for WMF to fix regardless

2017-06-30T11:38:06Z

rm for now, could be unexpected behaviour? should wait for WMF to fix regardless

Jc86035: according to MediaWiki:Mobile.css, this is preferred?

2017-06-30T10:31:23Z

according to MediaWiki:Mobile.css, this is preferred?

Mr. Stradivarius: remove li_style, ul_style and ol_style parameters, allow itemn_style and itemn_value parameters, and add a tracking category for item_stylen and item_valuen parameters

2014-06-12T09:21:46Z

remove li_style, ul_style and ol_style parameters, allow itemn_style and itemn_value parameters, and add a tracking category for item_stylen and item_valuen parameters

Mr. Stradivarius: don't mark item_stylen parameters as deprecated yet - needs to be saved for the next stage of conversion

2014-06-11T12:19:07Z

don't mark item_stylen parameters as deprecated yet - needs to be saved for the next stage of conversion

Mr. Stradivarius: Changed protection level of Module:List: High-risk Lua module: used by Template:Unbulleted list ([Edit=Allow only template editors and admins] (indefinite) [Move=Allow only template editors and admins] (indefinite))

2014-05-26T01:22:22Z

Changed protection level of Module:List: High-risk Lua module: used by Template:Unbulleted list ([Edit=Allow only template editors and admins] (indefinite) [Move=Allow only template editors and admins] (indefinite))

Mr. Stradivarius: split the HTML rendering code out into a separate function, add a tracking category for deprecated parameters, ignore blank/whitespace arguments, and replace getArgNums with the equivalent function in Module:TableTools

2014-05-26T01:15:39Z

split the HTML rendering code out into a separate function, add a tracking category for deprecated parameters, ignore blank/whitespace arguments, and replace getArgNums with the equivalent function in Module:TableTools

Show changes

Mr. Stradivarius: add fix to make the start parameter work with horizontal ordered lists, and switch to Module:Arguments for argument processing

2014-01-07T03:15:57Z

add fix to make the start parameter work with horizontal ordered lists, and switch to Module:Arguments for argument processing

@@ Line 23: / Line 23: @@
 	data.classes = {}
 	if listType == 'horizontal' or listType == 'horizontal_ordered' then
-		table.insert(data.classes, 'hlist')
+		table.insert(data.classes, 'hlist hlist-separated')
 	elseif listType == 'unbulleted' then
 		table.insert(data.classes, 'plainlist')

@@ Line 23: / Line 23: @@
 	data.classes = {}
 	if listType == 'horizontal' or listType == 'horizontal_ordered' then
-		table.insert(data.classes, 'hlist hlist-separated')
+		table.insert(data.classes, 'hlist')
 	elseif listType == 'unbulleted' then
 		table.insert(data.classes, 'plainlist')

@@ Line 23: / Line 23: @@
 	data.classes = {}
 	if listType == 'horizontal' or listType == 'horizontal_ordered' then
-		table.insert(data.classes, 'hlist')
+		table.insert(data.classes, 'hlist hlist-separated')
 	elseif listType == 'unbulleted' then
 		table.insert(data.classes, 'plainlist')

@@ Line 80: / Line 80: @@
 	 -- ul_style and ol_style are included for backwards compatibility. No
 	 -- distinction is made for ordered or unordered lists.
-	data.listStyle = args.list_style or args.ul_style or args.ol_style
+	data.listStyle = args.list_style
 	-- List items
@@ Line 90: / Line 90: @@
 		local item = {}
 		item.content = args[num]
-		item.style = args['item_style' .. tostring(num)]
+		item.style = args['item' .. tostring(num) .. '_style']
-			or args['li_style' .. tostring(num)]
+			or args['item_style' .. tostring(num)]
-		item.value = args['item_value' .. tostring(num)]
+		item.value = args['item' .. tostring(num) .. '_value']
 		table.insert(data.items, item)
 	end
@@ Line 148: / Line 149: @@
 function p.renderTrackingCategories(args)
 	local isDeprecated = false -- Tracks deprecated parameters.
-	for i, param in ipairs{'ul_style', 'ol_style', 'li_style'} do
+	for k, v in pairs(args) do
-		if args[param] then
+		k = tostring(k)
 			isDeprecated = true
 			break
 		end
 	end

@@ Line 157: / Line 157: @@
 		for k, v in pairs(args) do
 			k = tostring(k)
-			if k:find('^item_style%d+$') or k:find('^li_style%d+$') then
+			if k:find('^li_style%d+$') then
 				isDeprecated = true
 				break

← Older revision	Revision as of 20:04, 14 July 2018
(No difference)

← Older revision	Revision as of 21:40, 16 December 2017
(No difference)

← Older revision	Revision as of 01:22, 26 May 2014
(No difference)

@@ Line 4: / Line 4: @@
 local p = {}
 local htmlBuilder = require('Module:HtmlBuilder')
@@ Line 22: / Line 23: @@
 local function getArgNums(args)
-	-- Returns an array containing the keys of all positional arguments
+	-- Returns an array containing the keys of all positional arguments that contain data (i.e. non-whitespace values).
 	local nums = {}
 	for k, v in pairs(args) do
@@ Line 84: / Line 84: @@
 	-- Build the list tags and list items.
 	local list = root.tag((listType == 'ordered' or listType == 'horizontal_ordered') and 'ol' or 'ul')
 	list
 		.attr('type', typeAttr)
 		.css('list-style-type', listStyleType)
@@ Line 95: / Line 104: @@
 local function makeWrapper(listType)
 	return function(frame)
-		local origArgs
+		local args = getArgs(frame, {
-		if frame == mw.getCurrentFrame() then
+			valueFunc = function (key, value)
-			origArgs = frame:getParent().args
+				if type(key) == 'number' or value ~= '' then
-			for k, v in pairs(frame.args) do
+					return value
-				origArgs = frame.args
+				end
 			end
-		end
+		})
 		return p.makeList(listType, args)
 	end