OpenMage / magento-lts

Official OpenMage LTS codebase | Migrate easily from Magento Community Edition in minutes! Download the source code for free or contribute to OpenMage LTS | Security vulnerability patches, bug fixes, performance improvements and more.
https://www.openmage.org
Open Software License 3.0
868 stars 436 forks source link

Template Path Hints issues with menus in Frontend/Backend #2199

Open ADDISON74 opened 2 years ago

ADDISON74 commented 2 years ago

Go to Backend > System > Configuration > Developer. In Debug section set

Reload Backend and try to access Sales in navigation menu. The list appears behind the template path hints and you cannot select any links.

backend

Reload Frontend and try to access Women, Men from navigation menu. The list appears again behind the template path hints and you cannot select any link.

frontend

I don't think the problem is limited to menus, It is obvious that these div containers that contain template file names have a higher z-index than important elements on the page.

elidrissidev commented 2 years ago

For this issue, there's a PR of mine that's already merged that fixes it, but it's not a part of the latest release. Not sure about the frontend.

ADDISON74 commented 2 years ago

Your PR seems to fix only the Backend issue. It was merged after releasing OM-19.

elidrissidev commented 2 years ago

Yeah, I didn't address the frontend part.

ADDISON74 commented 2 years ago

In the Frontend not only the menu has an issue but also the other elements on the page that are dropdown. Here's what happens when you click on the Account menu

user_menu

and on the Cart container

cart

fballiano commented 2 years ago

how does this relate to https://github.com/OpenMage/magento-lts/issues/2226?

ADDISON74 commented 2 years ago

It is an addition to that PR. It missed OM backend theme and needs a fix for it. Not only in css files but in scss too.