[4.0] Workflow [a11y] (#21488)

Remove th and scope=col from the checkall cell
Add th and scope=row to the title cell

Author: brianteeman

administrator/components/com_workflow/tmpl/states/default.php

@@ -57,9 +57,9 @@
 								<th scope="col" style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('searchtools.sort', '', 's.ordering', $listDirn, $listOrder, null, 'asc', 'JGRID_HEADING_ORDERING', 'icon-menu-2'); ?>
 								</th>
-								<th scope="col" style="width:1%" class="nowrap text-center hidden-sm-down">
+								<td style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('grid.checkall'); ?>
-								</th>
+								</td>
 								<th scope="col" style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('searchtools.sort', 'JSTATUS', 's.condition', $listDirn, $listOrder); ?>
 								</th>
@@ -116,7 +116,7 @@
 									<td class="text-center hidden-sm-down">
 										<?php echo HTMLHelper::_('jgrid.isdefault', $item->default, $i, 'states.', $canChange); ?>
 									</td>
-									<td>
+									<th scope="row">
 										<?php if ($canEdit) : ?>
 											<?php $editIcon = '<span class="fa fa-pencil-square mr-2" aria-hidden="true"></span>'; ?>
 											<a href="<?php echo $edit; ?>" title="<?php echo Text::_('JACTION_EDIT'); ?> <?php echo $this->escape(addslashes($item->title)); ?>">
@@ -125,7 +125,7 @@
 										<?php else: ?>
 											<?php echo $item->title; ?>
 										<?php endif; ?>
-									</td>
+									</th>
 									<td class="text-center">
 										<?php echo Text::_($item->condition); ?>
 									</td>
@@ -138,7 +138,7 @@
 					</table>
 					<?php // load the pagination. ?>
 					<?php echo $this->pagination->getListFooter(); ?>
-				
+
 				<?php endif; ?>
 				<input type="hidden" name="task" value="">
 				<input type="hidden" name="boxchecked" value="0">

administrator/components/com_workflow/tmpl/transitions/default.php

@@ -56,9 +56,9 @@
 								<th scope="col" style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('searchtools.sort', '', 't.ordering', $listDirn, $listOrder, null, 'asc', 'JGRID_HEADING_ORDERING', 'icon-menu-2'); ?>
 								</th>
-								<th scope="col" style="width:1%" class="nowrap text-center hidden-sm-down">
+								<td style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('grid.checkall'); ?>
-								</th>
+								</td>
 								<th scope="col" style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('searchtools.sort', 'JSTATUS', 't.published', $listDirn, $listOrder); ?>
 								</th>
@@ -111,7 +111,7 @@
 											<?php echo HTMLHelper::_('jgrid.published', $item->published, $i, 'transitions.', $canChange); ?>
 										</div>
 									</td>
-									<td>
+									<th scope="row">
 										<?php if ($canEdit) : ?>
 											<?php $editIcon = '<span class="fa fa-pencil-square mr-2" aria-hidden="true"></span>'; ?>
 											<a href="<?php echo $edit; ?>" title="<?php echo Text::_('JACTION_EDIT'); ?> <?php echo $this->escape(addslashes($item->title)); ?>">
@@ -120,7 +120,7 @@
 										<?php else: ?>
 											<?php echo $item->title; ?>
 										<?php endif; ?>
-									</td>
+									</th>
 									<td class="text-center">
 									<?php if ($item->from_state_id < 0) : ?>
 										<?php echo Text::_('JALL'); ?>

administrator/components/com_workflow/tmpl/workflows/default.php

@@ -67,9 +67,9 @@
 								<th scope="col" style="width:1%" class="nowrap text-center d-none d-md-table-cell">
 									<?php echo HTMLHelper::_('searchtools.sort', '', 'w.ordering', $listDirn, $listOrder, null, 'asc', 'JGRID_HEADING_ORDERING', 'icon-menu-2'); ?>
 								</th>
-								<th style="width:1%" class="nowrap text-center hidden-sm-down">
+								<td style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('grid.checkall'); ?>
-								</th>
+								</td>
 								<th scope="col"  style="width:1%" class="nowrap text-center hidden-sm-down">
 									<?php echo HTMLHelper::_('searchtools.sort', 'JSTATUS', 'w.condition', $listDirn, $listOrder); ?>
 								</th>
@@ -141,7 +141,7 @@
 										<?php echo HTMLHelper::_('jgrid.published', $item->published, $i, 'workflows.', $canChange); ?>
 									</div>
 								</td>
-								<td>
+								<th scope="row">
 									<?php if ($canEdit || $canEditOwn) : ?>
 										<?php $editIcon = '<span class="fa fa-pencil-square mr-2" aria-hidden="true"></span>'; ?>
 										<a href="<?php echo $edit; ?>" title="<?php echo Text::_('JACTION_EDIT'); ?> <?php echo $this->escape(addslashes($item->title)); ?>">
@@ -150,7 +150,7 @@
 									<?php else: ?>
 										<?php echo $item->title; ?>
 									<?php endif; ?>
-								</td>
+								</th>
 								<td class="text-center">
 									<a href="<?php echo $states; ?>"><?php echo Text::_('COM_WORKFLOW_MANAGE'); ?></a>
 								</td>
@@ -182,7 +182,7 @@
 					</table>
 					<?php // load the pagination. ?>
 					<?php echo $this->pagination->getListFooter(); ?>
-				
+
 				<?php endif; ?>
 				<input type="hidden" name="task" value="">
 				<input type="hidden" name="boxchecked" value="0">