Joomla! Issue Tracker | Joomla! CMS #27451 - [4.0] Menu Container Setup [a11y]

Fixed in Code Base
11 Jan 2020
Medium
Build: 4.0-dev
# 27451
Diff
brianteeman:treediv

Pending

Pending Hound Hound is busy sniffing around... Details

User tests: Successful: Unsuccessful:

brianteeman
9 Jan 2020

Issue

Ensures that lists are structured correctly (list)

You can not have a div inside a ul

Target application

Menus: New Item - 4.0 Test Site - Administration

Element path

.treeselect

Snippet

<ul class="treeselect">
<div class="alert alert-info">

Solution

This PR simply moves the div to directly above the <ul>

There is no visual change

ccd083a 9 Jan 2020

[4.0] Menu Container Setup [a11y]

brianteeman - open - 9 Jan 2020

brianteeman - change - 9 Jan 2020

Status

New

⇒

Pending

joomla-cms-bot - change - 9 Jan 2020

Category

⇒

Administration com_menus

jwaisner - test_item - 9 Jan 2020 - Tested successfully

jwaisner - comment - 9 Jan 2020

I have tested this item ✅ successfully on ccd083a

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27451.}

289dc6d 11 Jan 2020

move if

brianteeman - change - 11 Jan 2020

Labels

Added: ?

wilsonge - change - 11 Jan 2020

Status	Pending	⇒	Fixed in Code Base
Closed_Date	0000-00-00 00:00:00	⇒	2020-01-11 15:36:21
Closed_By		⇒	wilsonge

wilsonge - close - 11 Jan 2020

wilsonge - merge - 11 Jan 2020

wilsonge - comment - 11 Jan 2020

Thanks!

Add a Comment

Older
Newer

Joomla! Issue Tracker - CMS

[#27451] - [4.0] Menu Container Setup [a11y]