Skip to content

Commit

Permalink
[a11y] Headings consecutive order (#20166)
Browse files Browse the repository at this point in the history
* [WIP] [a11y] Headings consecutive order

> Headings communicate the organization of the content on the page. Web browsers, plug-ins, and assistive technologies can use them to provide in-page navigation.

> Skipping heading ranks can be confusing and should be avoided where possible: Make sure that a <h2> is not followed directly by an <h4>, for example.

Source (https://www.w3.org/WAI/tutorials/page-structure/headings/)

This PR changes the heading in the plugin and modules from h3 to h2 and in the template styles to h4

### todo
joomla.edit.item_title layout uses h4 but before I change it I need to check everywhere that it is being used

* layout
  • Loading branch information
brianteeman authored and Michael Babker committed Apr 22, 2018
1 parent 28e46b1 commit 034beb4
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
Expand Up @@ -174,7 +174,7 @@
<div class="span9">
<?php if ($this->item->xml) : ?>
<?php if ($this->item->xml->description) : ?>
<h3>
<h2>
<?php
if ($this->item->xml)
{
Expand All @@ -185,7 +185,7 @@
echo JText::_('COM_MODULES_ERR_XML');
}
?>
</h3>
</h2>
<div class="info-labels">
<span class="label hasTooltip" title="<?php echo JHtml::_('tooltipText', 'COM_MODULES_FIELD_CLIENT_ID_LABEL'); ?>">
<?php echo $this->item->client_id == 0 ? JText::_('JSITE') : JText::_('JADMINISTRATOR'); ?>
Expand Down
Expand Up @@ -51,7 +51,7 @@
<div class="span9">
<?php if ($this->item->xml) : ?>
<?php if ($this->item->xml->description) : ?>
<h3>
<h2>
<?php
if ($this->item->xml)
{
Expand All @@ -62,7 +62,7 @@
echo JText::_('COM_PLUGINS_XML_ERR');
}
?>
</h3>
</h2>
<div class="info-labels">
<span class="label hasTooltip" title="<?php echo JHtml::_('tooltipText', 'COM_PLUGINS_FIELD_FOLDER_LABEL', 'COM_PLUGINS_FIELD_FOLDER_DESC'); ?>">
<?php echo $this->form->getValue('folder'); ?>
Expand Down
Expand Up @@ -37,9 +37,9 @@

<div class="row-fluid">
<div class="span9">
<h3>
<h2>
<?php echo JText::_($this->item->template); ?>
</h3>
</h2>
<div class="info-labels">
<span class="label hasTooltip" title="<?php echo JHtml::_('tooltipText', 'COM_TEMPLATES_FIELD_CLIENT_LABEL'); ?>">
<?php echo $this->item->client_id == 0 ? JText::_('JSITE') : JText::_('JADMINISTRATOR'); ?>
Expand Down
4 changes: 2 additions & 2 deletions layouts/joomla/edit/item_title.php
Expand Up @@ -16,9 +16,9 @@
?>

<?php if ($title) : ?>
<h4><?php echo $title; ?></h4>
<h2><?php echo $title; ?></h2>
<?php endif; ?>

<?php if ($name) : ?>
<h4><?php echo $name; ?></h4>
<h2><?php echo $name; ?></h2>
<?php endif;

0 comments on commit 034beb4

Please sign in to comment.