Skip to content
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

fix: webpack build and ie11 issue #63

Closed
wants to merge 6 commits into from
Closed

fix: webpack build and ie11 issue #63

wants to merge 6 commits into from

Conversation

tma001
Copy link
Contributor

@tma001 tma001 commented Nov 22, 2016

-made jQuery external dependency in webpack (looks for global / script tag)
-export cbp-theme.js as UMD module
-added polyfill for ie11 object.asign (for input mask)

See issues #24, #56

-made jQuery external dependency in webpack (looks for global / script tag)
-export cbp-theme.js as UMD module
-added polyfill for ie11 object.asign (for input mask)

See issues #24, #56
@tma001
Copy link
Contributor Author

tma001 commented Nov 22, 2016

try out the changes I made by testing date picker in ie;

also create a new project using cbp-theme and you should be able to add in cbp-theme.js and any jquery (as opposed to the cbp-theme specific one)

@davezen1
Copy link
Contributor

davezen1 commented Nov 22, 2016

You will need to add the parameters:

output: {
library: 'cbp-theme',
libraryTarget: 'umd',
umdNamedDefine: true,
}

-minify, remove comments from bundles
-reorder script imports in example example kitchen sink app
-added jQuery in 3rd party folder in example kitchen sink app
-made jQuery external dependency in webpack (looks for global / script tag)
-export cbp-theme.js as UMD module
-remove jQuery from package.json dependencies (looking for it as global script var in browser now)

see issues #24, #56, #59
…pers. Also moved alternative installation methods (webjars/gradle) to separate file.

See issue #57.
@davezen1
Copy link
Contributor

Closing per discussion with @tma001 regarding new PR for upstream changes.

@davezen1 davezen1 closed this Nov 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants