? PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar iamrobert
iamrobert
30 Jul 2023

Summary of Changes

The W3C spec doesn't allow a div inside a button as block elements are not allowed inside inline elements.

The ATUM topbar user menu contains this code:

Image

Switching to span resolves the W3C issue.

Testing Instructions

  1. Test ATUM template at W3C Checker

Actual result BEFORE applying this Pull Request

Fails W3C validation:

image

    <button class="dropdown-toggle d-flex align-items-center ps-0 py-0" data-bs-toggle="dropdown" type="button"
        title="<?php echo Text::_('MOD_USER_MENU'); ?>">
        <div class="header-item-icon">
            <span class="icon-user-circle" aria-hidden="true"></span>
        </div>
        <div class="header-item-text">
            <?php echo Text::_('MOD_USER_MENU'); ?>
        </div>
        <span class="icon-angle-down" aria-hidden="true"></span>
    </button>

Expected result AFTER applying this Pull Request

Passes W3C validation

    <button class="dropdown-toggle d-flex align-items-center ps-0 py-0" data-bs-toggle="dropdown" type="button"
        title="<?php echo Text::_('MOD_USER_MENU'); ?>">
        <span class="header-item-icon">
            <span class="icon-user-circle" aria-hidden="true"></span>
        </span>
        <span class="header-item-text">
            <?php echo Text::_('MOD_USER_MENU'); ?>
        </span>
        <span class="icon-angle-down" aria-hidden="true"></span>
    </button>

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 30 Jul 2023
Category Modules Administration
avatar iamrobert iamrobert - open - 30 Jul 2023
avatar iamrobert iamrobert - change - 30 Jul 2023
Status New Pending
avatar brianteeman brianteeman - test_item - 30 Jul 2023 - Tested successfully
avatar brianteeman
brianteeman - comment - 30 Jul 2023

I have tested this item successfully on e1c4519

Issue confirmed
Fix resolves the issue


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41285.

avatar Quy Quy - test_item - 30 Jul 2023 - Tested successfully
avatar Quy
Quy - comment - 30 Jul 2023

I have tested this item successfully on e1c4519


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41285.

avatar Quy Quy - change - 30 Jul 2023
Status Pending Ready to Commit
Labels Added: PR-4.3-dev
avatar Quy
Quy - comment - 30 Jul 2023

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41285.

avatar wilsonge wilsonge - close - 31 Jul 2023
avatar wilsonge wilsonge - merge - 31 Jul 2023
avatar wilsonge wilsonge - change - 31 Jul 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-07-31 10:19:04
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 31 Jul 2023

Thanks!

Add a Comment

Login with GitHub to post a comment