Module:Chart - Revision history

Ergzay: Hopefully fix problems caused by wikipedia-wide changes that added overflow:auto settings to all pages that caused chart clipping and chart scrollbars

2024-06-03T09:00:17Z

Hopefully fix problems caused by wikipedia-wide changes that added overflow:auto settings to all pages that caused chart clipping and chart scrollbars

← Previous revision		Revision as of 09:00, 3 June 2024
Line 48:		Line 48:
	table.insert( tab,		table.insert( tab,
	mw.text.tag( 'ul',		mw.text.tag( 'ul',
	{style="~~width:100%;~~list-style:none;column-width:12em;"},		{style="list-style:none;column-width:12em;"},
	table.concat( list, '\n' )		table.concat( list, '\n' )
	)		)
Line 412:		Line 412:

	local function drawChart()		local function drawChart()
	table.insert( res, mw.text.tag( 'div', { class = 'chart noresize', style = string.format( 'margin-top:1em;max-width:%spx;', width ) } ) )		table.insert( res, mw.text.tag( 'div', { class = 'chart noresize', style = string.format( 'padding-top:10px;margin-top:1em;max-width:%spx;', width ) } ) )
	table.insert( res, mw.text.tag( 'div', { style = string.format("position:relative;min-height:%spx;min-width:%spx;max-width:%spx;", height, width, width ) } ) )		table.insert( res, mw.text.tag( 'div', { style = string.format("position:relative;min-height:%spx;min-width:%spx;max-width:%spx;", height, width, width ) } ) )

MusikBot II: Changed protection settings for "Module:Chart": High-risk template or module: 2578 transclusions (more info) ([Edit=Require extended confirmed access] (indefinite) [Move=Require extended confirmed access] (indefinite))

2024-05-05T18:00:30Z

Changed protection settings for "Module:Chart": High-risk template or module: 2578 transclusions (more info) ([Edit=Require extended confirmed access] (indefinite) [Move=Require extended confirmed access] (indefinite))

← Previous revision	Revision as of 18:00, 5 May 2024
(No difference)

Ergzay: Fix strange white gaps and wrong-sized bars in graphs which were caused by rounding errors

2024-03-22T06:35:35Z

Fix strange white gaps and wrong-sized bars in graphs which were caused by rounding errors

← Previous revision		Revision as of 06:35, 22 March 2024
Line 314:		Line 314:

	local function calcHeights( gi, i, val )		local function calcHeights( gi, i, val )
	local barHeight = math.max(2, math.floor( val / yscales[gi] * chartHeight + 0.5 )) -- add half to make it "round" instead of "trunc", min height to 2 to avoid negative bar sizes		local barHeight = math.max( 2, math.floor( val / yscales[gi] * chartHeight + 0.5 ) ) -- add half to make it "round" instead of "trunc", min height to 2 to avoid negative bar sizes
	local top, base = chartHeight - barHeight, 0		local top, base = chartHeight - barHeight, 0
	if stack then		if stack then
	~~local~~ ~~rawbase~~ = 0		for j = 1, gi - 1 do
			if tonumber(values[j][i]) > 0 then
	for j = 1, gi - 1 do rawbase = rawbase + values[j][i] end -- sum the "i" value of all the groups below our group, gi.
	base = math.~~floor~~( ~~chartHeight~~ * ~~rawbase~~ / yscales[gi] ) -- ~~normally,~~ ~~and~~ ~~especially~~ if ~~it's~~ ~~"stack"~~, ~~all~~ the ~~yscales~~ ~~must~~ be ~~equal.~~		base = base + math.max( 2, math.floor( values[j][i] / yscales[gi] * chartHeight + 0.5 ) ) -- sum the "i" value of all the groups below our group, gi, and keep the same calculation for each bar
			end
			end
	end		end
	return barHeight, top - base		return barHeight, top - base

Ergzay: Not seeing any reason why these large buffers on the text of the x-labels are needed so I'm reducing them, seems to work fine in my own testing

2024-03-22T04:52:28Z

Not seeing any reason why these large buffers on the text of the x-labels are needed so I'm reducing them, seems to work fine in my own testing

← Previous revision		Revision as of 04:52, 22 March 2024
Line 403:		Line 403:
	setOffset, setWidth = groupBounds( i )		setOffset, setWidth = groupBounds( i )
	-- setWidth = 0.85 * setWidth		-- setWidth = 0.85 * setWidth
	table.insert( res, mw.text.tag( 'div', { style = string.format( legendDivStyleFormat, setOffset + 5, setWidth - 10, setWidth - 10 ) }, xlegends[i] or '' ) )		table.insert( res, mw.text.tag( 'div', { style = string.format( legendDivStyleFormat, setOffset + 1, setWidth - 2, setWidth - 2 ) }, xlegends[i] or '' ) )
	table.insert( res, mw.text.tag( 'div', { style = string.format( tickDivstyleFormat, setOffset + setWidth / 2 ) }, '' ) )		table.insert( res, mw.text.tag( 'div', { style = string.format( tickDivstyleFormat, setOffset + setWidth / 2 ) }, '' ) )
	end		end

Ergzay: Change location that 'barHeight' capping is done so that 'top' uses the correct value

2024-03-22T04:45:27Z

Change location that 'barHeight' capping is done so that 'top' uses the correct value

← Previous revision		Revision as of 04:45, 22 March 2024
Line 314:		Line 314:

	local function calcHeights( gi, i, val )		local function calcHeights( gi, i, val )
	local barHeight = math.floor( val / yscales[gi] * chartHeight + 0.5 ) -- add half to make it "round" instead of "trunc"		local barHeight = math.max(2, math.floor( val / yscales[gi] * chartHeight + 0.5 )) -- add half to make it "round" instead of "trunc", min height to 2 to avoid negative bar sizes
	local top, base = chartHeight - barHeight, 0		local top, base = chartHeight - barHeight, 0
	if stack then		if stack then
Line 321:		Line 321:
	base = math.floor( chartHeight * rawbase / yscales[gi] ) -- normally, and especially if it's "stack", all the yscales must be equal.		base = math.floor( chartHeight * rawbase / yscales[gi] ) -- normally, and especially if it's "stack", all the yscales must be equal.
	end		end
	if barHeight < 2 then
	barHeight = 2 -- Otherwise the template would try to create a bar with a negative height
	end
	return barHeight, top - base		return barHeight, top - base
	end		end

Andrybak: replace deprecated "Image:" with "File:"

2022-09-29T01:55:44Z

replace deprecated "Image:" with "File:"

← Previous revision		Revision as of 01:55, 29 September 2022
Line 112:		Line 112:

	local function createImageMap()		local function createImageMap()
	addRes( '{{#tag:imagemap\|', '~~Image~~:Circle frame.svg{{!}}' .. ( radius * 2 ) .. 'px' )		addRes( '{{#tag:imagemap\|', 'File:Circle frame.svg{{!}}' .. ( radius * 2 ) .. 'px' )
	addRes( unpack( imslices ) )		addRes( unpack( imslices ) )
	addRes( 'desc none', '}}' )		addRes( 'desc none', '}}' )

Pppery: Bugfix for extremely lopsided charts

2021-10-17T15:53:14Z

Bugfix for extremely lopsided charts

← Previous revision		Revision as of 15:53, 17 October 2021
Line 321:		Line 321:
	base = math.floor( chartHeight * rawbase / yscales[gi] ) -- normally, and especially if it's "stack", all the yscales must be equal.		base = math.floor( chartHeight * rawbase / yscales[gi] ) -- normally, and especially if it's "stack", all the yscales must be equal.
	end		end
			if barHeight < 2 then
			barHeight = 2 -- Otherwise the template would try to create a bar with a negative height
			end
	return barHeight, top - base		return barHeight, top - base
	end		end

Guarapiranga: Upgrade from sandbox

2021-06-02T04:04:36Z

Upgrade from sandbox

Show changes

Johnuniq: per talk, remove reference strip markers from tooltip

2021-05-14T01:21:57Z

per talk, remove reference strip markers from tooltip

Izno: remove vendor prefixes on column width

2021-01-08T05:24:31Z

remove vendor prefixes on column width

← Previous revision		Revision as of 05:24, 8 January 2021
Line 48:		Line 48:
	table.insert( tab,		table.insert( tab,
	mw.text.tag( 'ul',		mw.text.tag( 'ul',
	{style="width:100%;list-style:none~~;-webkit-column-width:12em;-moz-column-width:12em~~;column-width:12em;"},		{style="width:100%;list-style:none;column-width:12em;"},
	table.concat( list, '\n' )		table.concat( list, '\n' )
	)		)

← Previous revision		Revision as of 01:21, 14 May 2021
Line 78:		Line 78:
	links[i] = link		links[i] = link
	end		end


	radius = getArg( 'radius', 150 )		radius = getArg( 'radius', 150 )
	hideGroupLegends = not nulOrWhitespace( args[keywords.hideGroupLegends] )		hideGroupLegends = not nulOrWhitespace( args[keywords.hideGroupLegends] )
Line 91:		Line 91:
	addSlice( i, mw.ustring.match( slice, '^%(%s(.-)%s%)$' ) )		addSlice( i, mw.ustring.match( slice, '^%(%s(.-)%s%)$' ) )
	end		end


	for k, v in pairs(args) do		for k, v in pairs(args) do
	local ind = mw.ustring.match( k, '^' .. keywords.slice .. '%s+(%d+)$' )		local ind = mw.ustring.match( k, '^' .. keywords.slice .. '%s+(%d+)$' )
	if ind then addSlice( tonumber( ind ), v ) end		if ind then addSlice( tonumber( ind ), v ) end
	end		end


	for _, val in ipairs( values ) do sum = sum + val end		for _, val in ipairs( values ) do sum = sum + val end
	for i, value in ipairs( values ) do		for i, value in ipairs( values ) do
Line 308:		Line 308:
	function tooltip( gi, i, val )		function tooltip( gi, i, val )
	if tooltips and tooltips[gi] and not nulOrWhitespace( tooltips[gi][i] ) then return tooltips[gi][i], true end		if tooltips and tooltips[gi] and not nulOrWhitespace( tooltips[gi][i] ) then return tooltips[gi][i], true end
	local groupName = not nulOrWhitespace( groupNames[gi] ) and groupNames[gi] .. ': ' or ''		local groupName = mw.text.killMarkers(not nulOrWhitespace( groupNames[gi] ) and groupNames[gi] .. ': ' or '')
	local prefix = unitsPrefix[gi] or unitsPrefix[1] or ''		local prefix = unitsPrefix[gi] or unitsPrefix[1] or ''
	local suffix = unitsSuffix[gi] or unitsSuffix[1] or ''		local suffix = unitsSuffix[gi] or unitsSuffix[1] or ''
Line 349:		Line 349:
	local left, barWidth = calcx( gi, i )		local left, barWidth = calcx( gi, i )
	local barHeight, top = calcHeights( gi, i, val )		local barHeight, top = calcHeights( gi, i, val )


	-- borders so it shows up when printing		-- borders so it shows up when printing
	local style = string.format("position:absolute;left:%spx;top:%spx;height:%spx;min-width:%spx;max-width:%spx;background-color:%s;-webkit-print-color-adjust:exact;border:1px solid %s;border-bottom:none;overflow:hidden;",		local style = string.format("position:absolute;left:%spx;top:%spx;height:%spx;min-width:%spx;max-width:%spx;background-color:%s;-webkit-print-color-adjust:exact;border:1px solid %s;border-bottom:none;overflow:hidden;",