Skip to content

[Lens] Improve no data popover styles#70594

Merged
flash1293 merged 7 commits intoelastic:masterfrom
flash1293:lens/style-fixes
Jul 6, 2020
Merged

[Lens] Improve no data popover styles#70594
flash1293 merged 7 commits intoelastic:masterfrom
flash1293:lens/style-fixes

Conversation

@flash1293
Copy link
Contributor

@flash1293 flash1293 commented Jul 2, 2020

Super small follow up (#69147 (comment))

Improves styles of the "no data" popover dismiss button

Before
Screenshot 2020-07-02 at 15 27 51

After
Screenshot 2020-07-03 at 09 53 12

@flash1293 flash1293 added release_note:skip Skip the PR/issue when compiling release notes v7.9.0 v8.0.0 labels Jul 2, 2020
@flash1293 flash1293 marked this pull request as ready for review July 2, 2020 15:09
@flash1293 flash1293 requested a review from a team as a code owner July 2, 2020 15:09
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@flash1293 flash1293 requested a review from cchaos July 2, 2020 15:09
Copy link
Contributor

@cchaos cchaos left a comment

Choose a reason for hiding this comment

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

Woo, thanks!! Maybe we should also add in a title="Empty dataset" or whatever we should call the currently selected timerange of data.

@cchaos
Copy link
Contributor

cchaos commented Jul 3, 2020 via email

@cchaos
Copy link
Contributor

cchaos commented Jul 3, 2020 via email

@flash1293
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

Copy link
Contributor

@ppisljar ppisljar left a comment

Choose a reason for hiding this comment

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

code LGTM

@flash1293 flash1293 merged commit 3bddc4a into elastic:master Jul 6, 2020
flash1293 added a commit to flash1293/kibana that referenced this pull request Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes v7.9.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Comments