0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

wmlunits: Update menu markup to allow for CSS column layouts

Browse source
This commit is contained in:
Ignacio R. Morelle 2017-08-13 21:39:06 -03:00
parent 9f843423ca
commit ef406b6938
1 changed files with 7 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
data/tools/unit_tree/html_output.py
View file

@ -501,15 +501,13 @@ class HTMLOutput:
"""
html_name = cleantext(name)
html_classes = " ".join((cleantext(classes), "popuptrigger"))
# FIXME: This is legacy code needed for the Language menu, since it's
# a table and we can't make it otherwise because CSS column
# support is still hit-or-miss for some browsers still in use.
child_tag = 'ul' if not is_table_container else 'div'
label_tag = 'li' if not is_table_container else 'div'
write('<li class="popupcontainer" role="menuitem" aria-haspopup="true">')
write('<a class="' + html_classes + '" href="#">' + html_name + '</a>')
write('<' + child_tag + ' class="popupmenu" id="' + menuid + '" role="menu" aria-label="' + html_name + '">')
write('<' + label_tag + '>' + html_name + '</' + label_tag + '>')
write('<div id="%s" class="popupmenu" role="menu" aria-label="%s">' %
(menuid, html_name))
write('<div class="popupheader">' + html_name + '</div>')
if not is_table_container:
write('<ul>')
def add_menuitem_placeholder():
"""Writes a horizontal bar serving as a menu placeholder."""
@ -540,9 +538,8 @@ class HTMLOutput:
original.
"""
if not is_table_container:
write('</ul></li>\n')
else:
write('</div></li>')
write('</ul>')
write('</div></li>')
# We may not have all the required info yet so defer writing the
# campaigns/eras navigation.