Skip to content

[7.x] [Uptime] Expand synthetic journey step thumbnail on hover (#89179)#89970

Merged
justinkambic merged 1 commit intoelastic:7.xfrom
justinkambic:backport/7.x/pr-89179
Feb 2, 2021
Merged

[7.x] [Uptime] Expand synthetic journey step thumbnail on hover (#89179)#89970
justinkambic merged 1 commit intoelastic:7.xfrom
justinkambic:backport/7.x/pr-89179

Conversation

@justinkambic
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

)

* Add desired hover functionality and a test.

* Switch render from img to EuiImage for step view.

* Create new module for ping_timestamp. Extract a function. Add a test.

* Extract nav buttons, translations. Add tests.

* Fix a typo.

* Extract caption to own file. Add tests.

* Extract no image display to dedicated file. Add aria label. Add tests.

* Make import path more explicit.

* Move step image popover to dedicated file. Add tests.

* Clean up inline code in timestamp component.

* Explicit var names.

* Simplicity.

* Fix refactoring issues in test files.

* Move translations to central file.

* Rename test for better accuracy.
@justinkambic justinkambic added the backport This PR is a backport of another PR label Feb 2, 2021
@justinkambic justinkambic self-assigned this Feb 2, 2021
@justinkambic justinkambic added Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.12.0 labels Feb 2, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/uptime (Team:uptime)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
uptime 574 581 +7

Async chunks

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

id before after diff
uptime 892.6KB 895.7KB +3.0KB

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

@justinkambic justinkambic merged commit 3528b47 into elastic:7.x Feb 2, 2021
@justinkambic justinkambic deleted the backport/7.x/pr-89179 branch February 2, 2021 13:25
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 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.12.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants