Fixed menu cursor

[midlan]

Description (*)

Fixes the menu cursor for items that is link and have childs simultaneously, e.g.:

Related Pull Requests

Fixed Issues (if relevant)

1. Fixes OpenMage/magento-lts#<issue_number>

Manual testing scenarios (*)

1. ...
2. ...

Questions or comments

Contribution checklist (*)

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All automated tests passed successfully (all builds are green)

[kiatng]

I cannot replicate the cursor fix in this PR. However, I can fix it by changing line 90 from { cursor:default; } to { cursor:pointer }

magento-lts/skin/adminhtml/default/default/menu.css

Lines 88 to 90 in 4b17235

	#nav li.parent a,
	#nav li.parent li.parent a,
	#nav li.parent li.parent li.parent a { cursor:default; }

Screenshots of the 2 cursors:

Is that what this PR is about? Or am I missing something?

[midlan]

@kiatng it fixes the situation the parent is link itself (they have href="someurl") - it should have pointer cursor. Otherwise there is parents that are not links (they have href="#") so they shouldn't have pointer cursor.

[github-actions]

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
2 runs  ±0  2 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit f84c396. ± Comparison against base commit 4b17235.