Might as well add a related question here. Another tag I'd like to deprecate is the div that wraps the buttonlist class.
// No buttons? No button strip either.
if (empty($buttons))
return;
echo '
<div class="buttonlist', !empty($direction) ? ' float' . $direction : '', '"', (empty($buttons) ? ' style="display: none;"' : ''), (!empty($strip_options['id']) ? ' id="' . $strip_options['id'] . '"': ''), '>
<ul>',
implode('', $buttons), '
Getting rid of it is fine, since the ul is just as good as a wrapper. There aren't many files to go through, so this is an easy one rather than a major revision (much like the span.new_posts).
Again, there's a slight worry about a piece of javascript, this time in topic.js.
// Make sure it can go somewhere.
if (typeof(oButtonStripDisplay) == 'object' && oButtonStripDisplay != null)
oButtonStripDisplay.style.display = "";
else
{
var oNewDiv = document.createElement('div');
var oNewList = document.createElement('ul');
oNewDiv.id = this.opt.sButtonStripDisplay;
oNewDiv.className = this.opt.sButtonStripClass ? this.opt.sButtonStripClass : 'buttonlist floatbottom';
I assume we could just change the tag name there to ul instead of div. If that's gonna work, no worries. I'm perplexed by the floatbottom bit though. That is not valid CSS, so unless it's jQuery or something it's redundant too.