Perfect Scrollbar causing console error: too much recursion

webservices pro premium priority commented a year ago

Yes. Thanks for pointing that out. I was attempting to try importing the min.js file as an alternative using import PerfectScrollbar from 'mdbootstrap-pro/src/mdb/js/pro/perfect-scrollbar';

The error still occurs regardless of if I use the src file or the min.js file.

Grzegorz Bujański staff commented a year ago

Please send your webpack config. I will check why this error occurs. Does this error appear when you start the dev server or after the build when you open the compiled project?

webservices pro premium priority commented a year ago

Hello Grzegorz. We are doing a very specific build for a large WordPress theme. Since WordPress handles loading of files, we are building assets and then storing them. We are not using Webpack to bundle a typical web app.

Here is a snippet with my webpack.config.js under the JS tab. We are seeing this happen with our compiled project.

webservices pro premium priority commented a year ago

Our frontend.js has all of the imports (e.g. import Button from 'mdbootstrap-pro/src/mdb/js/free/button';) at the top. Then all of our custom JS is below that.

frontend.js gets loaded in the footer of our site. I have tried loading it in the header but that did not work.

Can I message you directly somehow so I can provide a link to a simplified version of our project? That way you can see clearly how we are building MDB5? If you're able to see my contact info, feel free to email me at the address on my profile.

Thanks :-)

Grzegorz Bujański staff commented a year ago

If you can send it to me, it will help me a lot :) You can send it directly to me at g.bujanski@mdbootstrap.com

webservices pro premium priority commented a year ago

Hello. I sent the email to you on 6/7. Did you receive it?

Grzegorz Bujański staff commented a year ago

Unfortunately, I only got the one where you ask for progress. But there is no zip file in it. Can you resend the message with the attachment?

webservices pro premium priority commented a year ago

I just sent another email with a link rather than an attachment. Hopefully that will help it get delivered this time. Thanks!

webservices pro premium priority commented a year ago

Hello, I emailed you yesterday to see if you have anything to follow up with. Have you found anything?

kpienkowska staff commented a year ago

Please resend this message to k.pienkowska@mdbootstrap.com because the staff member you are contacting won't be available for some time.

webservices pro premium priority commented a year ago

Thank you. I have just sent the email.

webservices pro premium priority commented a year ago

MDB5 is adding Perfect Scrollbar automatically. Unlike other modules, I do not think it is automatically loading what is needed. I was hoping your suggestion to initialize it would be the solution.

After looking at Perfect Scrollbar documentation, I tried to set it up to initialize so that when MDB5 adds Perfect Scrollbar it will work. Unfortunately, it still does not work.

Initializing by JS

I updated our frontend.js to initialize Perfect Scrollbar by JS. I tried doing it like as shown in the docs.

...
import PerfectScrollbar from 'mdbootstrap-pro/src/mdb/js/pro/perfect-scrollbar';
...
const initByJS = document.querySelector('#bsu-mobile-nav');
// Cannot load modules using mdb.* when loading JS the lean way.
// const psInitbyJS = new mdb.PerfectScrollbar(initByJS);
const psInitbyJS = new PerfectScrollbar(initByJS);
...

This results in the same console error about too much recursion.

Initializing using Basic Example

The docs also suggest adding data-mdb-perfect-scrollbar="true" to the container. So I tried updating the container as follows.

<nav id="bsu-mobile-nav" ... data-mdb-perfect-scrollbar="true">

This resulted in a new error as shown in this screenshot of the console.

Is there is a bug?

I am loading JS as described under Optimization > Lean Javascript. But other modules do not have any issues. Not even parallax.

There seems to be a bug in Perfect Scrollbar or at least how it is implemented in MDB5. It doesn't seem to like something about loading this way.

Does it seem there is a bug or is there something else I should try next?

kpienkowska staff commented a year ago

Did you fixed html content that you were trying to init?

Changing the initiation method won't help when initiated code is incorrect.

In the files you've sent me you had classes like ps__rail-x added to your HTML. they shouldn't be there. You need to fix your HTML to see whether perfect scrollbar initiates on itself or if will it needs to be initiated with JS.

webservices pro premium priority commented a year ago

Sorry, the HTML contains some things placed by JS. I built the example in a snippet on the MDB site and some things were injected. I have removed ps__rail-x and the problem described above is still the same.

kpienkowska staff commented a year ago

That was an example, there was more unnecessary code. Does your navbar work correctly in the snippets?

webservices pro premium priority commented a year ago

Yes. It works in snippets because all JS is included. Like I originally mentioned, it also works in our project if we use import * as mdb from 'mdbootstrap-pro'; to import all JS. We are loading JS as shown in the MDB docs on lean JS. When loading it this way, everything else works except Perfect Scrollbar.I decided to start experimenting with .offcanvas which does not automatically init Perfect Scrollbar. When I use .offcanvas and use JS to manually init Perfect Scrollbar, it works fine. No matter what I do, .sidenav is automatically initializing Perfect Scrollbar and it breaks.

webservices pro premium priority commented a year ago

I found the bug. I was looking in the sidenav source file (i.e. node_modules/mdbootstrap-pro/src/mdb/js/pro/sidenav.js) and found that the import is being done as if Perfect Scrollbar is installed as an individual npm package (i.e. node_modules/perfect-scrollbar). Even doing npm install perfect-scrollbar is not a fix.

Perfect Scrollbar is bundled with MDB5 Pro. All of the other imports in sidenav.js use the path ../mdb/*. Since Perfect Scrollbar is located in ../mdb/ I updated the path in sidenav.js from

import PerfectScrollbar from 'perfect-scrollbar';

to

import PerfectScrollbar from '../mdb/perfect-scrollbar';

After running npm run build it works as expected. Now sidenav automatically adds Perfect Scrollbar and it works!

Will you please escalate this now so that a developer can fix it?

kpienkowska staff commented a year ago

I will report this, thank you for the information.