Hello,

I note that the main link of the drop down menu which unrolls the sub links does not work. The sub-links work but not the main

Comments

nixax created an issue. See original summary.

nixax’s picture

Category: Bug report » Support request
nixax’s picture

Issue summary: View changes
sreenivas bttv’s picture

@nixax,

Thanks for raising a support request.

To allow click on top-level menu links, following changes are requested. It requires both twig and js changes.

  • Adding caret as button. On clicking, caret icon child menus will be shown and hide.
  • For a desktop breakpoint, on hover menu item child menu will be shown instead of click. (for best practices triggering caret icon click will be method)
  • On mobile and tablet breakpoints, all level menu links are clickable and Caret icon used to expand child menus.
nixax’s picture

Thanks for your following.
Do I have to wait for a patch development from you or is it an operation that I have to do personally? I have no skills in js

sreenivas bttv’s picture

@nixax,

I will try to raise the patch by this weekend if any other member did not pick the issue.

sreenivas bttv’s picture

@nixax, please unassign the ticket unless if you are working towards fixing the issue. So other contributors will pick the issue.

kiran.kadam911’s picture

Assigned: nixax » kiran.kadam911
Status: Active » Needs work

taking this up.

hansa11’s picture

Status: Needs work » Active
nixax’s picture

Status: Active » Needs work
kiran.kadam911’s picture

Assigned: kiran.kadam911 » Unassigned
Status: Needs work » Needs review

Attaching patch in next comment.

kiran.kadam911’s picture

StatusFileSize
new2.73 KB

As per #4 I did the changes, Kindly review the attached patch.

Thanks!

rahulrasgon’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new157.8 KB

@kiran.kadam911 patch is apply successfully. It is working fine in Chrome and Firefox.

RTBB + 1

sreenivas bttv’s picture

Status: Reviewed & tested by the community » Fixed

@kiran.kadam911 & rahulrasgon,

Thank you and #12 patch pushed to 8.x-3.x-dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.