Module:Navbox - Revision history

Vincent: 83 revisions imported

2019-09-10T01:33:48Z

83 revisions imported

2019-08-18T17:49:23Z

per tper

2019-08-10T17:25:34Z

per request on talk page

2018-10-26T16:03:01Z

per discussion at WT:Lua, this needs to be initialized in a non-global context

2018-10-23T20:47:30Z

optional list1class to allow templatestyles in Module:Navbox Canada

2018-08-22T04:52:06Z

fix confusion (a test I did was not correct)

2018-08-22T04:42:54Z

2018-07-27T13:10:53Z

typo

2018-07-27T13:10:05Z

don't add title padding to navbar

2018-06-15T23:27:24Z

aria-labelledby fallbacks: title, otherwise above, otherwise lone group

@@ Line 354: / Line 354: @@
 	if args.title and (args.state ~= 'plain' and args.state ~= 'off') then
 		tbl
-			:addClass('collapsible')
+			:addClass('mw-collapsible')
 			:addClass(args.state or 'autocollapse')
 	end

@@ Line 427: / Line 427: @@
 			:attr('role', 'navigation')
 			:addClass('navbox')
 			:cssText(args.bodystyle)
 			:cssText(args.style)

@@ Line 10: / Line 10: @@
 local args
 local border
-local listnums = {}
+local listnums
 local ODD_EVEN_MARKER = '\127_ODDEVEN_\127'
 local RESTART_MARKER = '\127_ODDEVEN0_\127'
@@ Line 385: / Line 385: @@
 function p._navbox(navboxArgs)
 	args = navboxArgs
 	for k, _ in pairs(args) do

@@ Line 260: / Line 260: @@
 		:addClass('navbox-' .. oddEven)
 		:addClass(args.listclass)
 		:tag('div')
 			:css('padding', (index == 1 and args.list1padding) or args.listpadding or '0em 0.25em')

@@ Line 437: / Line 437: @@
 	end
-	if (args.nocat or 'false'):lower() ~= 'false' then
+	if (args.nocat or 'false'):lower() == 'false' then
 		renderTrackingCategories(res)
 	end

← Older revision	Revision as of 01:33, 10 September 2019
(No difference)

@@ Line 216: / Line 216: @@
 			:addClass(args.groupclass)
 			:cssText(args.basestyle)
-            :css('width', args.groupwidth or '1%') -- If groupwidth not specified, minimize width
+			:css('width', args.groupwidth or '1%') -- If groupwidth not specified, minimize width
 		groupCell
@@ Line 437: / Line 437: @@
 	end
-	renderTrackingCategories(res)
+	if (args.nocat or 'false'):lower() ~= 'false' then
+		renderTrackingCategories(res)
 	return striped(tostring(res))
 end

@@ Line 85: / Line 85: @@
 			args.name,
 			mini = 1,
-			fontstyle = (args.basestyle or '') .. ';' .. (args.titlestyle or '') .. ';background:none transparent;border:none;-moz-box-shadow:none;-webkit-box-shadow:none;box-shadow:none; padding:none;'
+			fontstyle = (args.basestyle or '') .. ';' .. (args.titlestyle or '') .. ';background:none transparent;border:none;-moz-box-shadow:none;-webkit-box-shadow:none;box-shadow:none; padding:0;'
 		})
 	end

@@ Line 134: / Line 134: @@
 	titleCell
 		:tag('div')
 			:attr('id', mw.uri.anchorEncode(args.title))
 			:addClass(args.titleclass)
@@ Line 163: / Line 164: @@
 			:attr('colspan', getAboveBelowColspan())
 			:tag('div')
 				:wikitext(processItem(args.above, args.nowrapitems))
 end
@@ Line 201: / Line 204: @@
 	if args['group' .. listnum] then
 		local groupCell = row:tag('th')
 		groupCell
@@ Line 398: / Line 407: @@
 			:attr('role', 'navigation')
 			:node(tbl)
-		if args.title then
+		-- aria-labelledby title, otherwise above, otherwise lone group
-			nav:attr('aria-labelledby', mw.uri.anchorEncode(args.title))
+		if args.title or args.above or (args.group1 and not args.group2) then
 		else
 			nav:attr('aria-label', 'Navbox')
@@ Line 419: / Line 429: @@
 			:css('padding', '3px')
 			:node(tbl)
-		if args.title then
+		-- aria-labelledby title, otherwise above, otherwise lone group
-			nav:attr('aria-labelledby', mw.uri.anchorEncode(args.title))
+		if args.title or args.above or (args.group1 and not args.group2) then
 		else
 			nav:attr('aria-label', 'Navbox')