Module:Sidebar: Difference between revisions

← Older edit Newer edit →

@@ Line 1: / Line 1: @@
---
--- This module implements {{Sidebar}}
---
 require('strict')
 local cfg = mw.loadData('Module:Sidebar/configuration')
@@ Line 54: / Line 51: @@
 		return false
 	end
+end
+-- do we want a navbar
+-- will be wanted later for finding hlist
+local function has_navbar(navbar_mode, sidebar_name)
+	return navbar_mode ~= cfg.i18n.navbar_none and
+		navbar_mode ~= cfg.i18n.navbar_off and
+		(
+			sidebar_name or
+			mw.getCurrentFrame():getParent():getTitle():gsub(cfg.i18n.pattern.sandbox, '') ~=
+			cfg.i18n.title_not_to_add_navbar
+		)
+end
+local function has_list_class(args, htmlclass)
+	local patterns = {
+		'^' .. htmlclass .. '$',
+		'%s' .. htmlclass .. '$',
+		'^' .. htmlclass .. '%s',
+		'%s' .. htmlclass .. '%s'
+	}
+	for arg, value in pairs(args) do
+		if type(arg) == 'string' and mw.ustring.find(arg, 'class') then
+			for _, pattern in ipairs(patterns) do
+				if mw.ustring.find(args[arg] or '', pattern) then
+					return true
+				end
+			end
+		end
+	end
+	return false
+end
+-- there are a lot of list classes in the wild, so we add their TemplateStyles
+local function add_list_styles(args)
+	local function add_list_templatestyles(htmlclass, templatestyles)
+		local frame = mw.getCurrentFrame()
+		if has_list_class(args, htmlclass) then
+			return frame:extensionTag{
+				name = 'templatestyles', args = { src = templatestyles }
+			}
+		else
+			return ''
+		end
+	end
+	-- TODO: get hlist to the point where we can turn this on
+	-- see [[MediaWiki talk:Common.css/to do#Hlist]]
+	local plainlist_styles = add_list_templatestyles('plainlist', 'Plainlist/styles.css')
+	return plainlist_styles
+end
+-- work around [[phab:T303378]]
+-- for each arg: find all the templatestyles strip markers, insert them into a
+-- table. then remove all templatestyles markers from the arg
+local function move_hiding_templatestyles(args)
+	local gfind = string.gfind
+	local gsub = string.gsub
+	local templatestyles_markers = {}
+	local strip_marker_pattern = '(\127[^\127]*UNIQ%-%-templatestyles%-%x+%-QINU[^\127]*\127)'
+	for k, arg in pairs(args) do
+		for marker in gfind(arg, strip_marker_pattern) do
+			table.insert(templatestyles_markers, marker)
+		end
+		args[k] = gsub(arg, strip_marker_pattern, '')
+	end
+	return templatestyles_markers
 end
@@ Line 65: / Line 130: @@
 		args = getArgs(frame)
 	end
+	local hiding_templatestyles = table.concat(move_hiding_templatestyles(args))
 	local root = mw.html.create()
 	local child = args.child and mw.text.trim(args.child) == cfg.i18n.child_yes
@@ Line 235: / Line 301: @@
 	end
-	if not child then
+	if not child and has_navbar(args.navbar, args.name) then
-		if args.navbar ~= cfg.i18n.navbar_none and args.navbar ~= cfg.i18n.navbar_off and
+		root
-			(args.name or frame:getParent():getTitle():gsub(cfg.i18n.pattern.sandbox, '') ~=
+			:tag('tr')
-			cfg.i18n.title_not_to_add_navbar) then
+				:tag('td')
-			root
+					:addClass(cfg.i18n.class.navbar)
-				:tag('tr')
+					:cssText(args.navbarstyle)
-					:tag('td')
+					:wikitext(require('Module:Navbar')._navbar{
-						:addClass(cfg.i18n.class.navbar)
+						args.name,
-						:cssText(args.navbarstyle)
+						mini = 1,
-						:wikitext(require('Module:Navbar')._navbar{
+						fontstyle = args.navbarfontstyle
-							args.name,
+					})
-							mini = 1,
-							fontstyle = args.navbarfontstyle
-						})
-		end
 	end
@@ Line 282: / Line 344: @@
 		child_templatestyles,
 		grandchild_templatestyles,
+		add_list_styles(args),
+		hiding_templatestyles,
 		tostring(root),
 		(child and cfg.i18n.category.child or ''),
@@ Line 333: / Line 397: @@
 	local contentArgs = {}
-	local is_centered_list_titles
+	local is_centered_list_titles = false
 	if args['centered list titles'] and args['centered list titles'] ~= '' then
 		is_centered_list_titles = true
-	else
-		is_centered_list_titles = false
 	end