Skip to content

[Ingest Pipelines] Use sentence case for mouse tooltip #74847

Merged
jloleysens merged 1 commit intoelastic:masterfrom
jloleysens:ingest-pipelines/fix/use-sentence-case-for-mouse-tooltip
Aug 12, 2020
Merged

[Ingest Pipelines] Use sentence case for mouse tooltip #74847
jloleysens merged 1 commit intoelastic:masterfrom
jloleysens:ingest-pipelines/fix/use-sentence-case-for-mouse-tooltip

Conversation

@jloleysens
Copy link
Contributor

Summary

This contribution adds support for rendering known-processors per their user-friendly defined labels in the mouse tooltip.

How to test

  1. Go to ingest pipelines app in stack management
  2. Create a new pipeline
  3. Add a few processors (some from the list of known processors and one unknown one; e.g. my-processor)
  4. Re-order the processors and ensure the mouse tooltip renders as expected (per screenshots)

Screenshots

Known processor
Screenshot 2020-08-12 at 14 00 17

Unknown processor
Screenshot 2020-08-12 at 14 00 54

@jloleysens jloleysens added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// release_note:skip Skip the PR/issue when compiling release notes v7.10.0 Feature:Ingest Node Pipelines Ingest node pipelines management v7.9.1 labels Aug 12, 2020
@jloleysens jloleysens requested a review from a team as a code owner August 12, 2020 12:05
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui (Team:Elasticsearch UI)

@jloleysens jloleysens assigned yuliacech and unassigned sebelga Aug 12, 2020
Copy link
Contributor

@yuliacech yuliacech left a comment

Choose a reason for hiding this comment

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

Hi @jloleysens , the change looks good to me! Tested locally and the tooltip was rendered correctly for both a known and a custom processor.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

async chunks size

id value diff baseline
ingestPipelines 610.0KB +290.0B 609.7KB

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

@jloleysens jloleysens merged commit 0e5ac40 into elastic:master Aug 12, 2020
@jloleysens jloleysens deleted the ingest-pipelines/fix/use-sentence-case-for-mouse-tooltip branch August 12, 2020 15:45
jloleysens added a commit to jloleysens/kibana that referenced this pull request Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Ingest Node Pipelines Ingest node pipelines management release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v7.9.1 v7.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants