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

[Integrations UI] Add updated integrations subtitle #102738

Merged
merged 3 commits into from
Jun 21, 2021

Conversation

kpollich
Copy link
Member

Summary

Adds an updated subtitle for the Integrations UI per @hbharding's direction in #102317 (comment)

Screenshot

Screen Shot 2021-06-21 at 8 33 27 AM

Closes #102317

@kpollich kpollich added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.14.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Jun 21, 2021
@kpollich kpollich requested a review from hbharding June 21, 2021 12:48
@kpollich kpollich requested a review from a team as a code owner June 21, 2021 12:48
@kpollich kpollich self-assigned this Jun 21, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

const HeroImage = memo(() => {
const { toAssets } = useLinks();
const { uiSettings } = useStartServices();
const IS_DARK_THEME = uiSettings.get('theme:darkMode');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am wondering what is the best way to get that settings, you can also get it like this useTheme().darkMode

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied this component from the previous header.tsx file which was unused. I will update this to use the hook approach.

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
fleet 470 469 -1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fleet 679.2KB 679.9KB +753.0B

History

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

cc @kpollich

@kpollich kpollich merged commit 773647e into elastic:master Jun 21, 2021
@kpollich kpollich deleted the 102317-re-add-integrations-subtitle branch June 21, 2021 16:13
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 21, 2021
* Add updated integrations subtitle

* Fix header image declarations

* Use theme hook for dark mode setting
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Jun 21, 2021
* Add updated integrations subtitle

* Fix header image declarations

* Use theme hook for dark mode setting

Co-authored-by: Kyle Pollich <[email protected]>
@amolnater-qasource
Copy link

Hi @EricDavisX
We have revalidated this on 7.14.0 Snapshot and found it fixed now.

  • New description is available under Integrations tab.

Screenshot:
5

Thanks
QAS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.14.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fleet] Description not available under Integrations tab.
5 participants