-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
move jquery-datetimepicker to npm/webpack #10713
Conversation
silverwind
commented
Mar 11, 2020
- update to latest version and move to npm
- adapt for api changes and css class rename
- add specificity to arc-green rules as dependency css now loads later
- use imports-loader to make it load correctly
- fix some wrong paths in librejs
Resulting bundle is 335kB because of the dependencies this plugin has. Not sure if/how it's possible to prevent that second jQuery from loading. It's only used on a single page (create milestone) and I think we should just use Fomantic's Calendar to replace it entirely later. |
@silverwind how hard is it to switch to "Fomantic's Calendar" directly instead? |
Probably not too hard but I've never worked with Fomantic stuff yet. |
Another issue with using |
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.
LGTM
- update to latest version and move to npm - adapt for api changes and css class rename - add specificity to arc-green rules as dependency css now loads later - use imports-loader to make it load correctly - fix some wrong paths in librejs
ping LG-TM |