[Forwardport] M2.2.4 missing meta title tag and doesn't show#16608
[Forwardport] M2.2.4 missing meta title tag and doesn't show#16608gelanivishal wants to merge 3 commits intomagento:2.3-developfrom
Conversation
|
Hi @gelanivishal. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
|
Hi @miguelbalparda, thank you for the review. |
| protected function processMetadataContent($name, $content) | ||
| { | ||
| $method = 'get' . $this->string->upperCaseWords($name, '_', ''); | ||
| if ($name === 'title') { |
There was a problem hiding this comment.
Looks like additional logic for title attribute should be here, please, revert.
There was a problem hiding this comment.
@sidolov Do you want to keep old logic for 2.3 ?
There was a problem hiding this comment.
@gelanivishal , yes, this logic should be here due to bugfix
|
I am closing this PR now due to inactivity. Feel free to reopen and update if you wish to continue. |
Original Pull Request
#15532
FIX for issue #15501 - M2.2.4 missing meta title tag and doesn't show product name if meta title is empty
Description
The current Magento behaviour does not allow to specify a title meta tag and the "Meta Title" option in the product's page is binded to the page's title.
This PR adds the meta tile tag and allows to use title and meta title independently.
Fixed Issues (if relevant)
Contribution checklist