Skip to content

[7.14] [KibanaPageTemplate] Fix custom sidebar classes overriding kbnPageTemplate class (#103715)#108234

Merged
kibanamachine merged 1 commit intoelastic:7.14from
kibanamachine:backport/7.14/pr-103715
Aug 11, 2021
Merged

[7.14] [KibanaPageTemplate] Fix custom sidebar classes overriding kbnPageTemplate class (#103715)#108234
kibanamachine merged 1 commit intoelastic:7.14from
kibanamachine:backport/7.14/pr-103715

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.14:

…plate class (elastic#103715)

* Fix .kbnPageTemplate__pageSideBar getting overriden by passed classes

- due to it existing in ...rest, and ...rest coming after pageSideBarProps

* Add test

* Fix `sideBarClasses`

Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Caroline Horn <549577+cchaos@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
Contributor Author

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kibanaReact 146.8KB 146.7KB -64.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @constancecchen

@kibanamachine kibanamachine merged commit 4756cb3 into elastic:7.14 Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants