@sourabhsisodia_

Hii @riddhi.addweb , If you are facing some other issue , please create a new issue for it . The issue you are mentioning is not due to above MR . And for testing above issue you have to create nested menus then test the MR .

@brooke_heaton Please review the MR .

sourabhsisodia_ → made their first commit to this issue’s fork.

sourabhsisodia_ → made their first commit to this issue’s fork.

sourabhsisodia_ → made their first commit to this issue’s fork.

Applied this MR 8 , MR works fine . Redirect specification is optional and it works well for specified users . RTBC++.

Tested the MR!, user after logging in is redirected to the current path. The MR is also working for other paths. RTBC++

Tested the MR! Tags are designed properly in both mobile and desktop view . RTBC++.

Tested the MR! , MR applied successfully . Custom text option for 'Home' link is there on the config page . The value in the custom text field is visible on the breadcrumb . Attached screenshot for refrence. RTBC++.

Hii @Kanchan Bhogade, Please move the issue to RTBC state.

Updated the MR , Please review

Updated the MR according to review .

Fixed some spacing issue , now it should work fine .

@mortona2k can you share a video or image , of what issue you are facing

@mortona2k Tried the patch in a fresh setup , its working for me,

@Grimreaper Made the changes , please review

Heyy @arousseau I am not getting nay error , the module is working fine for me.

Module working fine for me , just don't manually type the link in the field instead copy the whole link from browser.

@mortona2k , I have solved and tested the issue but the MR is getting a phpunit error , can you please look into it ??

Its wroking fine for me, can you share a video or image of issue , you are facing

@Shubham_Kumar have you tried this on fresh setup ??

The module is working good , all check boxes are getting selected.

@DieterHolvoet I think the above error is due to this - https://www.drupal.org/project/drupal/issues/3396099 🐛 The core/drupal.message library requires a status_messages render element Active , after the mentioned core issue get resolved , we can make a js function that show status messages as a toast

@dll416 The admin configure page works fine for me. I am not getting any errror.

sourabhsisodia_ → made their first commit to this issue’s fork.

Tested the MR , it successfully solves the above mentioned issue

Heyy @EstherTempel the issue is replicating in claro and all other themes also and the error message are not getting any css and there is no template in the module which describes how the errors should be shown in validation, by default they are taking css (if available) from the default theme .

Tested the MR , it works fine

Tested the patch and it works fine !!

Reference video for fixed bug.

Fixed the issue and created a MR for the same .

sourabhsisodia_ → made their first commit to this issue’s fork.

Solved issue demo

@divya.sejekan Solved the error , and remove slide function is asynchronous and i think we don't need to make the add slide functionality asynchronous , because it will require to change many correlated things to it

@darkdim Updated the MR.