Skip to content

[4.0] Fix markup#30353

Merged
wilsonge merged 1 commit intojoomla:4.0-devfrom
Quy:menus-markup
Aug 12, 2020
Merged

[4.0] Fix markup#30353
wilsonge merged 1 commit intojoomla:4.0-devfrom
Quy:menus-markup

Conversation

@Quy
Copy link
Contributor

@Quy Quy commented Aug 12, 2020

Summary of Changes

Add closing quote.

Testing Instructions

Code review
or
Go to Menus > All Menu Items
View page source
Find jgrid without the closing quote

Actual result BEFORE applying this Pull Request

<span class="tbody-icon jgrid aria-labelledby="cbunsetDefault0-desc">

Expected result AFTER applying this Pull Request

<span class="tbody-icon jgrid" aria-labelledby="cbunsetDefault0-desc">

@wilsonge wilsonge merged commit dc5dff7 into joomla:4.0-dev Aug 12, 2020
@wilsonge wilsonge added this to the Joomla 4.0 milestone Aug 12, 2020
@Quy Quy deleted the menus-markup branch August 12, 2020 17:34
dgrammatiko added a commit to dgrammatiko/joomla-cms that referenced this pull request Aug 13, 2020
…om_templates

* '4.0-dev' of github.com:joomla/joomla-cms:
  Add closing quote (joomla#30353)
  [4.0] Make sure the renderer does not manipulate the inline CSS and JS (joomla#28719)
  [4.0] Atum and Cassiopea Change alerts styling - space saving + (joomla#30294)
  Update editor versions (joomla#30340)
  [4.0][mod_menu] Add space for 'menu class' (joomla#30341)
  Optimize code for aria-current (joomla#30328)
  [4.0] Composer and npm updates 10 Aug (joomla#30334)
  [4.0] fix js code style (joomla#30335)
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments