2014-07-13, 05:30 PM
After revamping forumdisplay templates i have noticed that my div with class hiddenrow shows style="display: table-row;". Now since it's a tableless revamp i had to make a edit in inline_moderation.js. My suggestion is to use .show() / hide() instead of .css("display", "none") and .css("display", "table-row"). show() will display property value equal to selected element (table-cell for td, table row for tr, block for div, inline for span).
Following lines in inline_moderation.js have .css("display", "table-row")
- Line 82
- Line 90
- Line 130
- Line 326
- Line 366
Following lines have .css("display", "none"):
- Line 153
- Line 162
- Line 187
- Line 193
- Line 330
- Line 340
- Line 360
To shorten it: is it possible to replace
with
and
with
Same effect but this edit is more friendly to tableless themes.
Following lines in inline_moderation.js have .css("display", "table-row")
- Line 82
allSelectedRow.css('display', 'table-row');
- Line 90
selectRow.css('display', 'table-row');
- Line 130
allSelectedRow.css('display', 'table-row');
- Line 326
selectRow.css('display', 'table-row');
- Line 366
allSelectedRow.css('display', 'table-row');
Following lines have .css("display", "none"):
- Line 153
selectRow.css('display', 'none');
- Line 162
allSelectedRow.css('display', 'none');
- Line 187
selectRow.css('display', 'none');
- Line 193
allSelectedRow.css('display', 'none');
- Line 330
selectRow.css('display', 'none');
- Line 340
allSelectedRow.css('display', 'none');
- Line 360
selectRow.css('display', 'none');
To shorten it: is it possible to replace
.css('display', 'table-row');
with
.show();
and
.css('display', 'none');
with
.hide();
Same effect but this edit is more friendly to tableless themes.