Module:Navbox: Difference between revisions

(10 intermediate revisions by the same user not shown)

Line 7:

local p = {}

-- TODO (~~low~~ priority): add ~~an option for a dataless header row, to mirror the~~

-- TODO (medium priority): add/use navbox/styles.css

~~-- option of having a headerless data row. Requires changing row content from~~

~~-- a regular array to an associative array~~.

-- TODO (medium priority): nest and refactor helper functions

Line 28:

Line 26:

local header_style = '! style="white-space: nowrap; font-size: 0.9em; width: 5%; text-align: right; background-color: #eaecf0; padding: 0.25em 0.5em; border: 1px solid white;" |'

local data_style = is_content_navbox

and '| style="padding: 0;" |\n' -- For nested navboxes; ~~requires a newline~~ for it to work

and '| style="padding: 0;" |\n' -- For nested navboxes; endline required for nested navboxes to work properly

or '| style="font-size: 0.9em; padding: 0.25em 0.5em;" | ' -- For normal content

or '| style="font-size: 0.9em; padding: 0.25em 0.5em;" |' -- For normal content; endline not needed as adding it makes rows too thick

table.insert(row, "|-")

Line 35:

Line 33:

table.insert(row, data_style .. data)

elseif data and not header then

-- Row is a headerless data row~~...~~

-- Row is a headerless data row

local data_style = is_content_navbox

and '| style="padding: 0; ~~background-color: #eaecf0;"~~ colspan="2" |\n' -- For nested navboxes~~; requires a newline for it to work~~

and '| style="padding: 0; colspan="2" |\n' -- For nested navboxes

or '| style="font-size: 0.9em; padding: 0.25em 0.5em;" colspan="2" | ' -- For normal content

or '| style="font-size: 0.9em; padding: 0.25em 0.5em;" colspan="2" |' -- For normal content

table.insert(row, "|-")

table.insert(row, data_style .. data)

elseif header and not data then

-- Row is a dataless header row~~...~~

-- Row is a dataless header row

local data_style = is_content_navbox

and '! style="padding: 0;" colspan="2" |\n' -- For nested navboxes; ~~requires a newline for it to work~~

and '! style="padding: 0;" colspan="2" |\n' -- For nested navboxes; in case nav elements are ever added here, such as numlinks

or '! style="font-size: 0.9em; padding: 0.25em 0.5em;" colspan="2" | ' ~~-- For normal content~~

or '! style="white-space: nowrap; font-size: 0.9em; width: 5%; background-color: #eaecf0; padding: 0.25em 0.5em; border: 1px solid white;" colspan="2" |'

table.insert(row, "|-")

Line 104:

Line 102:

-- Navbox to be called by other modules; also called by wrapper function

function p._navbox(args)

local title = args["Title"]

local title = args["Title"] or "Navbox Title"

local name = args["name"]

local rows = args["Rows"]

Line 219:

Line 217:

-- Preprocess individual entries for headers and data

local rows = tiu.~~header_data_pairs_to_table~~(args, 30)

local rows = tiu.numbered_header_data_args_to_table(args, 30)

args["Rows"] = rows

@@ Line 7: / Line 7: @@
 local p = {}
--- TODO (low priority): add an option for a dataless header row, to mirror the
+-- TODO (medium priority): add/use navbox/styles.css
--- option of having a headerless data row. Requires changing row content from
--- a regular array to an associative array.
 -- TODO (medium priority): nest and refactor helper functions
@@ Line 28: / Line 26: @@
 		local header_style = '! style="white-space: nowrap; font-size: 0.9em; width: 5%; text-align: right; background-color: #eaecf0; padding: 0.25em 0.5em; border: 1px solid white;" |'
 		local data_style = is_content_navbox
-			and '| style="padding: 0;" |\n'									-- For nested navboxes; requires a newline for it to work
+			and '| style="padding: 0;" |\n'									-- For nested navboxes; endline required for nested navboxes to work properly
-			or  '| style="font-size: 0.9em; padding: 0.25em 0.5em;" | '		-- For normal content
+			or  '| style="font-size: 0.9em; padding: 0.25em 0.5em;" |'		-- For normal content; endline not needed as adding it makes rows too thick
 		table.insert(row, "|-")
@@ Line 35: / Line 33: @@
 		table.insert(row, data_style .. data)
 	elseif data and not header then
-		-- Row is a headerless data row...
+		-- Row is a headerless data row
 		local data_style = is_content_navbox
-			and '| style="padding: 0; background-color: #eaecf0;" colspan="2" |\n'		-- For nested navboxes; requires a newline for it to work
+			and '| style="padding: 0; colspan="2" |\n'									-- For nested navboxes
-			or  '| style="font-size: 0.9em; padding: 0.25em 0.5em;" colspan="2" | '		-- For normal content
+			or  '| style="font-size: 0.9em; padding: 0.25em 0.5em;" colspan="2" |'		-- For normal content
 		table.insert(row, "|-")
 		table.insert(row, data_style .. data)
 	elseif header and not data then
-		-- Row is a dataless header row...
+		-- Row is a dataless header row
 		local data_style = is_content_navbox
-			and '! style="padding: 0;" colspan="2" |\n'									-- For nested navboxes; requires a newline for it to work
+			and '! style="padding: 0;" colspan="2" |\n'							-- For nested navboxes; in case nav elements are ever added here, such as numlinks
-			or  '! style="font-size: 0.9em; padding: 0.25em 0.5em;" colspan="2" | '		-- For normal content
+			or  '! style="white-space: nowrap; font-size: 0.9em; width: 5%; background-color: #eaecf0; padding: 0.25em 0.5em; border: 1px solid white;" colspan="2" |'
 		table.insert(row, "|-")
@@ Line 104: / Line 102: @@
 -- Navbox to be called by other modules; also called by wrapper function
 function p._navbox(args)
-	local title           = args["Title"]
+	local title           = args["Title"] or "Navbox Title"
 	local name            = args["name"]
 	local rows            = args["Rows"]
@@ Line 219: / Line 217: @@
 	-- Preprocess individual entries for headers and data
-	local rows = tiu.header_data_pairs_to_table(args, 30)
+	local rows = tiu.numbered_header_data_args_to_table(args, 30)
 	args["Rows"] = rows