Autolab
Autolab copied to clipboard
Requiring bootstrap date picker in asset pipeline
Currently we are doing
//= require bootstrap-datetimepicker
in application.js
Is there a reason why the time picker has to be loaded in every page?
I was wondering this as well. Perhaps it has something to do with the custom form builder 'FormBuilderWithDateTimeInput' ? It was added with it in 38c7733d, and the author explicitly mentioned adding this to the global application.js
@zyx-billy oh yes, this isn't necessary though. However I don't think its a very important issue at this point.
Furthermore, if @devanshk moves the date picker to materialize then we wouldn't need to inject it into every page like this.
The UI's been updated to Materialize, but we need to make sure removing this won't break anything.