Skip to content

Commit

Permalink
Return EUI CSS to Shareable Runtime (elastic#72990)
Browse files Browse the repository at this point in the history
* Add Canvas Styles

* Tree-shaking in EUI omits CSS in the runtime

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
  • Loading branch information
clintandrewhall and elasticmachine committed Jul 24, 2020
1 parent faba7fe commit 4b2aeae
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion x-pack/plugins/canvas/shareable_runtime/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ There are a number of options for the build script:

### Prerequisite

Before testing or running this PR locally, you **must** run `node scripts/runtime` from `/canvas` _after_ `yarn kbn bootstrap` and _before_ starting Kibana. It is only built automatically when Kibana is built to avoid slowing down other development activities.
Before testing or running this PR locally, you **must** run `node scripts/shareable_runtime` from `/canvas` _after_ `yarn kbn bootstrap` and _before_ starting Kibana. It is only built automatically when Kibana is built to avoid slowing down other development activities.

### Webpack Dev Server

Expand Down
2 changes: 1 addition & 1 deletion x-pack/plugins/canvas/shareable_runtime/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ module.exports = {
options: {
presets: [require.resolve('@kbn/babel-preset/webpack_preset')],
},
sideEffects: false,
},
{
test: /\.tsx?$/,
Expand Down Expand Up @@ -92,6 +91,7 @@ module.exports = {
},
},
],
sideEffects: true,
},
{
test: /\.module\.s(a|c)ss$/,
Expand Down

0 comments on commit 4b2aeae

Please sign in to comment.