Module:Chessboard - Revision history

SD0001: use clearer variable names - took me a while to figure out what these mean

2025-01-20T12:43:41Z

use clearer variable names - took me a while to figure out what these mean

← Previous revision		Revision as of 12:43, 20 January 2025
Line 47:		Line 47:
	end		end

	local ~~letters_tp~~ = letters:match( 'both' ) or letters:match( 'top' )		local letters_top = letters:match( 'both' ) or letters:match( 'top' )
	local ~~letters_bt~~ = letters:match( 'both' ) or letters:match( 'bottom' )		local letters_bottom = letters:match( 'both' ) or letters:match( 'bottom' )
	local ~~numbers_lt~~ = numbers:match( 'both' ) or numbers:match( 'left' )		local numbers_left = numbers:match( 'both' ) or numbers:match( 'left' )
	local ~~numbers_rt~~ = numbers:match( 'both' ) or numbers:match( 'right' )		local numbers_right = numbers:match( 'both' ) or numbers:match( 'right' )
	local width = ncols * size + 2		local width = ncols * size + 2
	if ( ~~numbers_lt~~ ) then width = width + 18 end		if ( numbers_left ) then width = width + 18 end
	if ( ~~numbers_rt~~ ) then width = width + 18 end		if ( numbers_right ) then width = width + 18 end

	local root = mw.html.create('div')		local root = mw.html.create('div')
Line 75:		Line 75:
	:attr('cellspacing', '0')		:attr('cellspacing', '0')

	if ( ~~letters_tp~~ ) then		if ( letters_top ) then
	b:tag('tr')		b:tag('tr')
	:wikitext(letters_row( rev, ~~numbers_lt~~, ~~numbers_rt~~ ))		:wikitext(letters_row( rev, numbers_left, numbers_right ))
	end		end
	local tablerow = b:tag('tr')		local tablerow = b:tag('tr')
	if ( ~~numbers_lt~~ ) then		if ( numbers_left ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('width', '18px')		:css('width', '18px')
Line 91:		Line 91:
	:wikitext(innerboard(args, size, rev))		:wikitext(innerboard(args, size, rev))

	if ( ~~numbers_rt~~ ) then		if ( numbers_right ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('width', '18px')		:css('width', '18px')
Line 97:		Line 97:
	:wikitext(rev and 1 or nrows)		:wikitext(rev and 1 or nrows)
	end		end
	if ( ~~numbers_lt~~ or ~~numbers_rt~~ ) then		if ( numbers_left or numbers_right ) then
	for trow = 2, nrows do		for trow = 2, nrows do
	local idx = rev and trow or ( 1 + nrows - trow )		local idx = rev and trow or ( 1 + nrows - trow )
	tablerow = b:tag('tr')		tablerow = b:tag('tr')
	if ( ~~numbers_lt~~ ) then		if ( numbers_left ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('height', size .. 'px')		:css('height', size .. 'px')
	:wikitext(idx)		:wikitext(idx)
	end		end
	if ( ~~numbers_rt~~ ) then		if ( numbers_right ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('height', size .. 'px')		:css('height', size .. 'px')
Line 113:		Line 113:
	end		end
	end		end
	if ( ~~letters_bt~~ ) then		if ( letters_bottom ) then
	b:tag('tr')		b:tag('tr')
	:wikitext(letters_row( rev, ~~numbers_lt~~, ~~numbers_rt~~ ))		:wikitext(letters_row( rev, numbers_left, numbers_right ))
	end		end

SD0001: rename class "chess-board" to "chess-pieces", since the div contains only the pieces, and to avoid confusion with the new top-level "chessboard" class

2025-01-19T10:22:35Z

rename class "chess-board" to "chess-pieces", since the div contains only the pieces, and to avoid confusion with the new top-level "chessboard" class

← Previous revision		Revision as of 10:22, 19 January 2025
Line 6:		Line 6:
	pattern = cfg.pattern or '%w%w'		pattern = cfg.pattern or '%w%w'
	local root = mw.html.create('div')		local root = mw.html.create('div')
	root:addClass('chess-~~board~~ notheme')		root:addClass('chess-pieces notheme')
	:css('position', 'relative')		:css('position', 'relative')
	:wikitext(cfg.image_board(size))		:wikitext(cfg.image_board(size))

SD0001: move most inline styles to Module:Chessboard/styles.css (syncing from sandbox)

2025-01-18T22:24:16Z

move most inline styles to Module:Chessboard/styles.css (syncing from sandbox)

← Previous revision		Revision as of 22:24, 18 January 2025
Line 18:		Line 18:
	local img = cfg.image_square(piece:match(pattern), row, col, size )		local img = cfg.image_square(piece:match(pattern), row, col, size )
	root:tag('div')		root:tag('div')
	:css('position', 'absolute')
	:css('z-index', '3')
	:css('top', tostring(( trow - 1 ) * size) .. 'px')		:css('top', tostring(( trow - 1 ) * size) .. 'px')
	:css('left', tostring(( tcol - 1 ) * size) .. 'px')		:css('left', tostring(( tcol - 1 ) * size) .. 'px')
	:css('width', size .. 'px')
	:css('height', size .. 'px')
	:wikitext(img)		:wikitext(img)
	end		end
Line 38:		Line 34:
	if num_lt then		if num_lt then
	root:tag('td')		root:tag('td')
	:css('vertical-align', 'inherit')
	:css('padding', '0')
	end		end
	for k = 1,ncols do		for k = 1,ncols do
	root:tag('td')		root:tag('td')
	:css('padding', '0')
	:css('vertical-align', 'inherit')
	:css('text-align', 'center')
	:css('height', '18px')		:css('height', '18px')
	:css('width', size .. 'px')		:css('width', size .. 'px')
Line 52:		Line 43:
	if num_rt then		if num_rt then
	root:tag('td')		root:tag('td')
	:css('vertical-align', 'inherit')
	:css('padding', '0')
	end		end
	return tostring(root)		return tostring(root)
Line 67:		Line 56:

	local root = mw.html.create('div')		local root = mw.html.create('div')
			:addClass('chessboard')
	:addClass('thumb')		:addClass('thumb')
	:addClass('noviewer')		:addClass('noviewer')
Line 84:		Line 74:
	:attr('cellpadding', '0')		:attr('cellpadding', '0')
	:attr('cellspacing', '0')		:attr('cellspacing', '0')
	:css('font-size', '88%')
	:css('border' , '1px #c8ccd1 solid')
	:css('padding', '0')
	:css('margin', 'auto')

	if ( letters_tp ) then		if ( letters_tp ) then
	b:tag('tr')		b:tag('tr')
	:css('vertical-align', 'middle')
	:wikitext(letters_row( rev, numbers_lt, numbers_rt ))		:wikitext(letters_row( rev, numbers_lt, numbers_rt ))
	end		end
	local tablerow = b:tag('tr')		local tablerow = b:tag('tr')
	:css('vertical-align','middle')
	if ( numbers_lt ) then		if ( numbers_lt ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('padding', '0')
	:css('vertical-align', 'inherit')
	:css('text-align', 'center')
	:css('width', '18px')		:css('width', '18px')
	:css('height', size .. 'px')		:css('height', size .. 'px')
Line 108:		Line 89:
	:attr('colspan', ncols)		:attr('colspan', ncols)
	:attr('rowspan', nrows)		:attr('rowspan', nrows)
	:css('padding', '0')
	:css('vertical-align', 'inherit')
	:wikitext(innerboard(args, size, rev))		:wikitext(innerboard(args, size, rev))

	if ( numbers_rt ) then		if ( numbers_rt ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('padding', '0')
	:css('vertical-align', 'inherit')
	:css('text-align', 'center')
	:css('width', '18px')		:css('width', '18px')
	:css('height', size .. 'px')		:css('height', size .. 'px')
Line 125:		Line 101:
	local idx = rev and trow or ( 1 + nrows - trow )		local idx = rev and trow or ( 1 + nrows - trow )
	tablerow = b:tag('tr')		tablerow = b:tag('tr')
	:css('vertical-align', 'middle')
	if ( numbers_lt ) then		if ( numbers_lt ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('padding', '0')
	:css('vertical-align', 'inherit')
	:css('text-align', 'center')
	:css('height', size .. 'px')		:css('height', size .. 'px')
	:wikitext(idx)		:wikitext(idx)
Line 136:		Line 108:
	if ( numbers_rt ) then		if ( numbers_rt ) then
	tablerow:tag('td')		tablerow:tag('td')
	:css('padding', '0')
	:css('vertical-align', 'inherit')
	:css('text-align', 'center')
	:css('height', size .. 'px')		:css('height', size .. 'px')
	:wikitext(idx)		:wikitext(idx)
Line 146:		Line 115:
	if ( letters_bt ) then		if ( letters_bt ) then
	b:tag('tr')		b:tag('tr')
	:css('vertical-align', 'middle')
	:wikitext(letters_row( rev, numbers_lt, numbers_rt ))		:wikitext(letters_row( rev, numbers_lt, numbers_rt ))
	end		end
Line 156:		Line 124:
	end		end

	return tostring(root)		return tostring(root) ..
			mw.getCurrentFrame():extensionTag( 'templatestyles', '', { src = 'Module:Chessboard/styles.css' } )
	end		end

Gonnym: clearer

2024-09-13T10:42:51Z

clearer

← Previous revision		Revision as of 10:42, 13 September 2024
Line 71:		Line 71:
	:addClass(align)		:addClass(align)
	if( header and header ~= '' ) then		if( header and header ~= '' ) then
	root:tag('div'):addClass('center'):css('line-height', '130%'):css('margin', '0 auto'):css('max-width', (width + ncols) .. 'px'):wikitext(header)		root:tag('div')
			:addClass('center')
			:css('line-height', '130%')
			:css('margin', '0 auto')
			:css('max-width', (width + ncols) .. 'px')
			:wikitext(header)
	end		end
	local div = root:tag('div')		local div = root:tag('div')
Line 89:		Line 94:
	:wikitext(letters_row( rev, numbers_lt, numbers_rt ))		:wikitext(letters_row( rev, numbers_lt, numbers_rt ))
	end		end
	local tablerow = b:tag('tr'):css('vertical-align','middle')		local tablerow = b:tag('tr')
			:css('vertical-align','middle')
	if ( numbers_lt ) then		if ( numbers_lt ) then
	tablerow:tag('td')		tablerow:tag('td')

MSGJ: trim footer to prevent undesired line breaks, remove two functions which appear to be unused and undocumented

2024-08-23T08:28:00Z

trim footer to prevent undesired line breaks, remove two functions which appear to be unused and undocumented

@@ Line 144: / Line 144: @@
 	end
-	if (footer and footer ~= '') then
+	if footer and mw.text.trim(footer)~='' then
 		div:tag('div')
 			:addClass('thumbcaption')
@@ Line 229: / Line 229: @@
 		return chessboard(pargs, size, reverse, letters, numbers, header, footer, align, clear)
 	end
 end

SD0001: remove unnecessary use of an explicit white background

2024-01-26T20:03:01Z

remove unnecessary use of an explicit white background

← Previous revision		Revision as of 20:03, 26 January 2024
Line 79:		Line 79:
	:attr('cellpadding', '0')		:attr('cellpadding', '0')
	:attr('cellspacing', '0')		:attr('cellspacing', '0')
	:css('background', 'white')
	:css('font-size', '88%')		:css('font-size', '88%')
	:css('border' , '1px #c8ccd1 solid')		:css('border' , '1px #c8ccd1 solid')

SD0001: add no-theme class to board for dark mode styling in mobile apps, per talk request by User:Dmitry Brant

2024-01-26T18:10:51Z

add no-theme class to board for dark mode styling in mobile apps, per talk request by User:Dmitry Brant

← Previous revision		Revision as of 18:10, 26 January 2024
Line 6:		Line 6:
	pattern = cfg.pattern or '%w%w'		pattern = cfg.pattern or '%w%w'
	local root = mw.html.create('div')		local root = mw.html.create('div')
	root:addClass('chess-board')		root:addClass('chess-board notheme')
	:css('position', 'relative')		:css('position', 'relative')
	:wikitext(cfg.image_board(size))		:wikitext(cfg.image_board(size))

SD0001: support generating chessboard from pgn

2023-12-31T20:26:57Z

support generating chessboard from pgn

← Previous revision		Revision as of 20:26, 31 December 2023
Line 210:		Line 210:
	local clear = args.clear or pargs.clear or ( align:match('tright') and 'right' ) or 'none'		local clear = args.clear or pargs.clear or ( align:match('tright') and 'right' ) or 'none'
	local fen = args.fen or pargs.fen		local fen = args.fen or pargs.fen
			local pgn = args.pgn or pargs.pgn


	size = mw.ustring.match( size, '[%d]+' ) or '26' -- remove px from size		size = mw.ustring.match( size, '[%d]+' ) or '26' -- remove px from size
			if (pgn) then
			local pgnModule = require('Module:Pgn')
			metadata, moves = pgnModule.main(pgn)
			fen = moves[#moves]
			end
	if (fen) then		if (fen) then
	align = args.align or pargs.align or 'tright'		align = args.align or pargs.align or 'tright'

Frietjes: Undid revision 1100998670 by Frietjes (talk) not needed

2022-07-29T17:45:53Z

Undid revision 1100998670 by Frietjes (talk) not needed

← Previous revision		Revision as of 17:45, 29 July 2022
Line 10:		Line 10:
	:wikitext(cfg.image_board(size))		:wikitext(cfg.image_board(size))

	local roffset, coffset = cfg.offset()
	for trow = 1,nrows do		for trow = 1,nrows do
	local row = rev and trow or ( 1 + nrows - trow )		local row = rev and trow or ( 1 + nrows - trow )
Line 21:		Line 20:
	:css('position', 'absolute')		:css('position', 'absolute')
	:css('z-index', '3')		:css('z-index', '3')
	:css('top', tostring(( trow - 1 ~~+ roffset~~) * size) .. 'px')		:css('top', tostring(( trow - 1 ) * size) .. 'px')
	:css('left', tostring(( tcol - 1 ~~+ coffset~~) * size) .. 'px')		:css('left', tostring(( tcol - 1 ) * size) .. 'px')
	:css('width', size .. 'px')		:css('width', size .. 'px')
	:css('height', size .. 'px')		:css('height', size .. 'px')

Frietjes: optional pattern

2022-07-28T20:58:03Z

optional pattern

← Previous revision		Revision as of 20:58, 28 July 2022
Line 4:		Line 4:

	local function innerboard(args, size, rev)		local function innerboard(args, size, rev)
			pattern = cfg.pattern or '%w%w'
	local root = mw.html.create('div')		local root = mw.html.create('div')
	root:addClass('chess-board')		root:addClass('chess-board')
Line 15:		Line 16:
	local col = rev and ( 1 + ncols - tcol ) or tcol		local col = rev and ( 1 + ncols - tcol ) or tcol
	local piece = args[ncols * ( nrows - row ) + col + 2] or ''		local piece = args[ncols * ( nrows - row ) + col + 2] or ''
	if piece:match( ~~'%w%w'~~ ) then		if piece:match( pattern ) then
	local img = cfg.image_square(piece:match(~~'%w%w'~~), row, col, size )		local img = cfg.image_square(piece:match(pattern), row, col, size )
	root:tag('div')		root:tag('div')
	:css('position', 'absolute')		:css('position', 'absolute')