-
Notifications
You must be signed in to change notification settings - Fork 6
Added listpkgs.html and did some cleaning up. #2
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
Conversation
This page will list all available packages.
Clean up code.
Cleaned up code.
Added ability to read packages.json data and output into a table. (Not tested yet.)
Added view all applications / packages to the navbar.
Added link to list all packages / applications.
Added listpgks.html and did some cleaning up.
|
Cool. I'll clone this locally and have a look :) |
|
Thanks, could you look at allpkgs.html. Having trouble getting json to display on the table. |
|
Just comitted 1fd07c1. Fixes table rendering. Getting this error in console:
Seems weird. |
|
@andersevenrud Thanks.
Getting that too. 910 is pointing to line 910 in that js file. I'll go file a issue with the Materialize Project. |
|
First check if there is a newer version available :) |
|
Will do :) |
|
Yea 0.97 is the current, and that's what were running right now. I haven't made any changes to the source files except the color of the navbar in the CSS file You can view the issue here: Dogfalo/materialize#1776 |
|
Did you build this materialize lib yourself ? Or was it just a prebuilt download ? |
|
It was a prebuilt download. Maybe I did something wrong in the tags?
|
|
It's a syntax error, so it's a problem in the build. I added a patch in the issue you made. |
|
Yeaaaaa, It's such a small syntax error that no wonder I assumed it was on there end. Lol. |
Did some clean up and added a list all applications / pkgs. Still need to work out a API but were getting there.