Problem/Motivation

Navbar toggler button in navbar is not working as it should. Adding bs to data-target and data-toggle https://git.drupalcode.org/project/radix/-/blob/5.0.x/src/components/nav... makes it work again
as said in https://getbootstrap.com/docs/5.0/components/navbar/#nav

CommentFileSizeAuthor
#4 3311193-4-after.png5.07 KBShyam_Bhatt
#4 3311193-4-before.png2.91 KBShyam_Bhatt
#2 3311193.patch695 bytesAnchal_gupta

Issue fork radix-3311193

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aleix created an issue. See original summary.

Anchal_gupta’s picture

Anchal_gupta CreditAttribution: Anchal_gupta at Material for Drupal India Association commented
FileSize
3311193.patch695 bytes

I have uploaded the patch. Please review it

aleix opened merge request !40

Shyam_Bhatt’s picture

Shyam_Bhatt CreditAttribution: Shyam_Bhatt at QED42 commented
Status: Active » Reviewed & tested by the community
FileSize
3311193-4-before.png2.91 KB
3311193-4-after.png5.07 KB

I have checked the patch "3311193.patch" is working fine. After applying the patch once we click on the navbar button in the mobile layout it's working. please check the below images.

Before:
2022-09-23/3311193-4-before.png

After:
2022-09-23/3311193-4-after.png

Shyam_Bhatt’s picture

Shyam_Bhatt CreditAttribution: Shyam_Bhatt at QED42 commented

doxigo made their first commit to this issue’s fork.

  • doxigo committed abc4343 on 5.0.x authored by aleix 
    Issue #3311193 by Anchal_gupta, doxigo: navbar toggler button not...
doxigo’s picture

doxigo CreditAttribution: doxigo as a volunteer and at Ramsalt Lab commented
Status: Reviewed & tested by the community » Fixed

Thanks guys, reverted the commit, merged on dev

Status: Fixed » Closed (fixed)

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