Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Update xml-instructions.md to remove unnecessary html tag #3860

Merged
merged 5 commits into from
Mar 5, 2019

Conversation

hgajjar
Copy link

@hgajjar hgajjar commented Mar 4, 2019

This PR is a:

  • New topic
  • Content update
  • Content fix or rewrite
  • Bug fix or improvement

Summary

Removed extra HTML tag which is being displayed in browser, and fixed a misleading xml tag for an xml attribute.

Additional information

List all affected URLs

https://devdocs.magento.com/guides/v2.3/frontend-dev-guide/layouts/xml-instructions.html
https://devdocs.magento.com/guides/v2.2/frontend-dev-guide/layouts/xml-instructions.html
https://devdocs.magento.com/guides/v2.1/frontend-dev-guide/layouts/xml-instructions.html

whatsnew
Corrected the syntax for the remove attribute used with the <referenceBlock> and <referenceContainer layout instructions.

@magento-cicd2
Copy link
Contributor

An admin must run tests on this PR before it can be merged.

@meker12 meker12 self-assigned this Mar 5, 2019
@meker12 meker12 self-requested a review March 5, 2019 05:10
Copy link
Contributor

@meker12 meker12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar changes needed for version 2.1 of the file https://devdocs.magento.com/guides/v2.1/frontend-dev-guide/layouts/xml-instructions.html file.

…leading tag from "remove" attribute for v2.1 guide
@hgajjar
Copy link
Author

hgajjar commented Mar 5, 2019

@meker12 Code review changes are done, kindly review, thanks.

@meker12
Copy link
Contributor

meker12 commented Mar 5, 2019

running tests

@meker12 meker12 merged commit 7240632 into magento:master Mar 5, 2019
@ghost
Copy link

ghost commented Mar 5, 2019

Hi @hgajjar, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants