-
Notifications
You must be signed in to change notification settings - Fork 39
Javascript/HTML/CSS improvements #73
base: public
Are you sure you want to change the base?
Conversation
Hi @C-Lodder, these changes look great if you're interested in working on these :) |
Another thing I'd like to at some point is moving the CSS out of the HTML files as well |
@HaydenMeloche good news. I'll continue working on this tomorrow. |
@HaydenMeloche On the "Textures" page, should the download button work in a test environment? |
@C-Lodder As of right now we don't really have any 'test' environment really. The download button as of right now would just hit the same server our live application would |
Hmm, in which case that's the main feature that's not working. I'll continue looking into this today |
@HaydenMeloche I've take the PR out of draft and it's ready for review. There's still a lot of jQuery code in the package, but I'll rewrite the rest of this in perhaps the next version |
@HaydenMeloche is there any update on this? |
hi @C-Lodder sorry, I'm out of my normal country at the moment for about another week or so. I can take a better look when I am back! |
Thanks for your hardwork! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the config change from me
@HaydenMeloche @burnss9 I've reverted the |
This is currently a draft PR.
I will continue working on this if one of the maintainers agrees with the changes.
Changes so far: