Skip to content

[7.x] [Ingest Pipelines] Processor Editor Move Tooltip (#72239)#72452

Merged
jloleysens merged 1 commit intoelastic:7.xfrom
jloleysens:backport/7.x/pr-72239
Jul 21, 2020
Merged

[7.x] [Ingest Pipelines] Processor Editor Move Tooltip (#72239)#72452
jloleysens merged 1 commit intoelastic:7.xfrom
jloleysens:backport/7.x/pr-72239

Conversation

@jloleysens
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* first implementation of tooltip

* Add processor tooltip component files

* remove init position from code for now

* colocate on change handler and make code a bit cleaner

* removed document.body.appendChild logic because EuiPortal does that for us

* use correct toggle button api

* added test to check button disabled while editing

* remove cursor not allowed

* simplify logic

* assert if against positive

* remove unused variable

* Remove unused actions const

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@jloleysens jloleysens added the backport This PR is a backport of another PR label Jul 20, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
ingestPipelines 267 +9 258

async chunks size

id value diff baseline
ingestPipelines 584.5KB +6.9KB 577.6KB

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

@jloleysens jloleysens merged commit 3b46e56 into elastic:7.x Jul 21, 2020
@jloleysens jloleysens deleted the backport/7.x/pr-72239 branch July 21, 2020 08:52
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