- Issue created by @olekstkach
- Status changed to Needs review
over 1 year ago 2:23pm 24 April 2023 - last update
over 1 year ago 2 fail The last submitted patch, 2: 3356041-2.patch, failed testing. View results →
We have checker isEnabled on sitemorse_lite/src/MenuLinkBuilder::buildMenuLink and if pages use admin route we add sitemorse-inactive class, but css which were added to this class do not hide menu item and as result we have broken menu item which is good for debuggin but not for ui
<div class="toolbar-tab">
<div class="toolbar-icon toolbar-item sitemorse-check-url sitemorse-inactive" title="Sitemorse is disabled on administrative URLs.">Sitemorse check</div>
<div><nav class="toolbar-lining clearfix" role="navigation"></nav></div> </div>Needs review
1.4
Code
The last submitted patch, 2: 3356041-2.patch, failed testing. View results →