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

[Lens] Metric trend PoC - toggle switch #138804

Closed
wants to merge 22 commits into from

Conversation

drewdaemon
Copy link
Contributor

@drewdaemon drewdaemon commented Aug 15, 2022

Summary

This is a PoC for adding trendlines to the metric visualization in Lens. It is the toggle switch version of the UI.

Screen Shot 2022-08-12 at 4 01 41 PM

@drewdaemon drewdaemon changed the title Metric trend poc [Lens] Metric trend poc Aug 15, 2022
@drewdaemon drewdaemon added the ci:cloud-deploy Create or update a Cloud deployment label Aug 15, 2022
@kibana-ci
Copy link
Collaborator

kibana-ci commented Aug 15, 2022

⏳ Build in-progress, with failures

Failed CI Steps

History

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

@drewdaemon drewdaemon changed the title [Lens] Metric trend poc [Lens] Metric trend PoC - toggle switch Aug 15, 2022
@KOTungseth KOTungseth added the ui-copy Review of UI copy with docs team is recommended label Aug 18, 2022
@KOTungseth
Copy link
Contributor

184644142-32b63953-05c2-46b2-bee4-9fcdb3cdc55c

Instead of Enable trendline, can we simplify to Trendline?

For novice users, this setting could benefit from a tooltip.

cc: @gchaps

@drewdaemon
Copy link
Contributor Author

@KOTungseth thank you for the feedback! This is just a proof of concept so it will not be merged. I have linked to your suggestion from the official trend lines issue so we don't forget when we actually do this: #135882.

Feel free to leave any other feedback on that issue or on the "real" PR in the future.

@drewdaemon
Copy link
Contributor Author

Followed this strategy in #141851

@drewdaemon drewdaemon closed this Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:cloud-deploy Create or update a Cloud deployment skip-ci ui-copy Review of UI copy with docs team is recommended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants