-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Closed
Labels
Fixed in 2.2.xThe issue has been fixed in 2.2 release lineThe issue has been fixed in 2.2 release lineFixed in 2.3.xThe issue has been fixed in 2.3 release lineThe issue has been fixed in 2.3 release lineIssue: Format is validGate 1 Passed. Automatic verification of issue format passedGate 1 Passed. Automatic verification of issue format passed
Description
Preconditions
- Magento 2.2.4 only
Steps to reproduce
- Open
app/code/Magento/Theme/view/frontend/templates/html/topmenu.phtml
- Remove the following code from the template:
data-mage-init='{"menu":{"responsive":true, "expanded":true, "position":{"my":"left top","at":"left bottom"}}}'
- Navigate to the product page
Expected result
- Breadcrumbs should remain visible
Actual result
- Breadcrumbs are not rendered
Explanation
2.2.4 release uses js component to render breadcrumbs on the product page.
Here is its init method:
magento2/app/code/Magento/Catalog/view/frontend/web/js/product/breadcrumbs.js
Lines 23 to 36 in 3bd8afb
_init: function () { | |
var menu; | |
// render breadcrumbs after navigation menu is loaded. | |
menu = $(this.options.menuContainer).data('mageMenu'); | |
if (typeof menu === 'undefined') { | |
this._on($(this.options.menuContainer), { | |
'menucreate': this._super | |
}); | |
} else { | |
this._super(); | |
} | |
}, |
I don't really know what is this check for if (typeof menu === 'undefined') {
, and why do we even need to get mageMenu
, maybe @ameysar can help with explanation?
iparmentier, roma84, rommmka, dmatthew, VincentMarmiesse and 1 more
Metadata
Metadata
Assignees
Labels
Fixed in 2.2.xThe issue has been fixed in 2.2 release lineThe issue has been fixed in 2.2 release lineFixed in 2.3.xThe issue has been fixed in 2.3 release lineThe issue has been fixed in 2.3 release lineIssue: Format is validGate 1 Passed. Automatic verification of issue format passedGate 1 Passed. Automatic verification of issue format passed