-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Controller 'mdtTable', required by directive 'mdtHeaderRow', can't be found! #94
Comments
Same Error here Did some searching , please see issue #24 Even after including jquery and lodash, the error remains |
Same Error :( the ^? char in require atribute means i need a controller with ? name as parent. |
@chungyan5 check if you have closing tags for each of your |
I have the same issue with both jquery and lodash installed |
I copied the example 1:1, just to get it running. First, i got the same error as described, but then I found a solution for my case. Right order of js-files is important:
Maybe it helps. |
hi all, It seems my fault in testing. It is worked, however, i still cannot use it. I just try the html and js files by browser in file:///folder/file.html, it works, no any error. @publicroot still have this problem even following your sequence. |
@chungyan5 Same problem here on Electron. Have you found some solution? |
Jquery was my issue, didn't install as a dependency. <script src="https://code.jquery.com/jquery-2.1.4.js"> </script>Any chance we can pull the jquery out of there entirely? |
@publicroot angular-sanitize.min.js is missing in Your list. |
Figured out how to fix it in my scenario: insert jquery.min.js BEFORE angular.js:
|
Adding jQuery at the beginning fix this issue, but I can't understand why ? I rather not load jQuery only because of this issue. More info:
|
@laurentperroteau Agreed. I've taken a quick look through the JS and realized jQuery isn't used that often (I think once?). We can almost get away without it! It has something to do with a few element lookups. I'm sure the author would happily accept a pull request for this! (it's on my backburner of tasks personally, but I'm going to try to refactor it out) @iamisti thoughts on potentially removing the dependency? Thank you for the library! It's working great so far! |
@laurentperroteau great, thanks! |
Same problem on electron, followed all instructions here, this is my scripts list:
And here my html:
Just in case, my app dependencies: Can someone tell me what's wrong? |
Which version did you try? |
Same error here with version 1.8.0. |
@markgerald @pietrovismara |
@iamisti me too version 1.8.0. |
Hi Guys, the fix is almost there, I'll have a new release by tomorrow morning and everybody can use it in their electron frontend too. |
@iamisti That's great to hear! |
@iamisti I'm not using electron. I read all the possible solutions for this error on the ordering of lodash, jquery, etc., and nothing works. These are my addictions:
The error occurs when trying to play your example: |
??? |
@markgerald sorry, I'm still working on that bug. We had a pull request for it, but then we figured out it is braking something else. So stay sharp, the fix is coming, most probably tomorrow, but latest after tomorrow. |
@iamisti I can clone the fork and test if it works on electron, just point me to the fork. |
@pietrovismara I pushed the fix to master, so now if you change your bower dependeny from: Then let me know if it's fixes the issue. |
@iamisti Done and it's working for me in electron! No more errors and the table is correctly displayed. |
Sounds greeat! 👍 Thanks for the fix from @laurentperroteau , I've used his suggestion. The fix is available with |
here now the error: Cannot read property 'appendChild' of null |
let me look into that and I'll get back when I have something for you to test |
@markgerald can u help me, describing a bit more how did u try to load my example?
Because I can't reproduce your issue, and now I'd need a bit more help in that to fix it for you. |
As this issue also refers to lodash i want to point out that in my case "md-data-table": "^2.2.0" still needs lodash to be included before. It would save a lot of time for beginners like me if it would be mentioned in the Install part of the readme. Edit: |
i found this error is similar to a closed issue "Controller 'mdDataTable', required by directive 'mdDataTableHeaderRow', can't be found!", i followed the final solution of that issue to test by adding
<script src="https://code.jquery.com/jquery-2.1.4.js"> </script> <script src="https://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.10.1/lodash.min.js"> </script>
, but i am no luck.
Could you please to have some ideas for this issue?
thanks
yan
The text was updated successfully, but these errors were encountered: