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

[Content management] fix table list flashes table interface when empty #160650

Merged
merged 8 commits into from
Jun 29, 2023

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Jun 27, 2023

Closes #159507
Closes #148557

Flaky test runner https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2508

Test instructions

  • Install sample web logs data set
  • Delete dashboard that ships with sample data set
  • switch to another app, like visualize
  • Use debugger tools to throttle network connection to "slow 3g"
  • Open dashboard application again
  • Verify empty table view is never displayed and there is no flickering between views.

@nreese nreese force-pushed the listing_empty_state branch from e5ff383 to 6beb01e Compare June 27, 2023 17:23
@nreese
Copy link
Contributor Author

nreese commented Jun 27, 2023

@elasticmachine merge upstream

@nreese nreese marked this pull request as ready for review June 28, 2023 13:44
@nreese nreese requested a review from a team as a code owner June 28, 2023 13:44
@nreese nreese added v8.9.0 v8.10.0 Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) labels Jun 28, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@nreese nreese added the release_note:skip Skip the PR/issue when compiling release notes label Jun 28, 2023
@nreese
Copy link
Contributor Author

nreese commented Jun 28, 2023

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
dashboard 360.0KB 360.0KB +1.0B
eventAnnotation 120.6KB 120.6KB -33.0B
filesManagement 88.8KB 88.8KB +1.0B
graph 458.0KB 458.0KB +1.0B
maps 2.7MB 2.7MB +1.0B
visualizations 262.1KB 262.0KB -32.0B
total -61.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 14 16 +2
securitySolution 413 417 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 15 17 +2
securitySolution 492 496 +4
total +6

History

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

@sebelga sebelga self-requested a review June 29, 2023 08:03
Copy link
Contributor

@sebelga sebelga left a comment

Choose a reason for hiding this comment

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

Tested locally and works as expected 👍

Thanks for fixing this @nreese ! 🎉

@nreese nreese merged commit 9f73f94 into elastic:main Jun 29, 2023
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 29, 2023
elastic#160650)

Closes elastic#159507
Closes elastic#148557

Flaky test runner
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2508

### Test instructions
* Install sample web logs data set
* Delete dashboard that ships with sample data set
* switch to another app, like visualize
* Use debugger tools to throttle network connection to "slow 3g"
* Open dashboard application again
* Verify empty table view is never displayed and there is no flickering
between views.

---------

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 9f73f94)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.9

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jun 29, 2023
…n empty (#160650) (#160893)

# Backport

This will backport the following commits from `main` to `8.9`:
- [[Content management] fix table list flashes table interface when
empty (#160650)](#160650)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Nathan
Reese","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-06-29T13:55:16Z","message":"[Content
management] fix table list flashes table interface when empty
(#160650)\n\nCloses
https://github.com/elastic/kibana/issues/159507\r\nCloses
https://github.com/elastic/kibana/issues/148557\r\n\r\nFlaky test
runner\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2508\r\n\r\n###
Test instructions\r\n* Install sample web logs data set\r\n* Delete
dashboard that ships with sample data set\r\n* switch to another app,
like visualize\r\n* Use debugger tools to throttle network connection to
\"slow 3g\"\r\n* Open dashboard application again\r\n* Verify empty
table view is never displayed and there is no flickering\r\nbetween
views.\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"9f73f9479c1ff77fbada8305e618931b57d65796","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Presentation","release_note:skip","Team:SharedUX","v8.9.0","v8.10.0"],"number":160650,"url":"https://github.com/elastic/kibana/pull/160650","mergeCommit":{"message":"[Content
management] fix table list flashes table interface when empty
(#160650)\n\nCloses
https://github.com/elastic/kibana/issues/159507\r\nCloses
https://github.com/elastic/kibana/issues/148557\r\n\r\nFlaky test
runner\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2508\r\n\r\n###
Test instructions\r\n* Install sample web logs data set\r\n* Delete
dashboard that ships with sample data set\r\n* switch to another app,
like visualize\r\n* Use debugger tools to throttle network connection to
\"slow 3g\"\r\n* Open dashboard application again\r\n* Verify empty
table view is never displayed and there is no flickering\r\nbetween
views.\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"9f73f9479c1ff77fbada8305e618931b57d65796"}},"sourceBranch":"main","suggestedTargetBranches":["8.9"],"targetPullRequestStates":[{"branch":"8.9","label":"v8.9.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/160650","number":160650,"mergeCommit":{"message":"[Content
management] fix table list flashes table interface when empty
(#160650)\n\nCloses
https://github.com/elastic/kibana/issues/159507\r\nCloses
https://github.com/elastic/kibana/issues/148557\r\n\r\nFlaky test
runner\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2508\r\n\r\n###
Test instructions\r\n* Install sample web logs data set\r\n* Delete
dashboard that ships with sample data set\r\n* switch to another app,
like visualize\r\n* Use debugger tools to throttle network connection to
\"slow 3g\"\r\n* Open dashboard application again\r\n* Verify empty
table view is never displayed and there is no flickering\r\nbetween
views.\r\n\r\n---------\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"9f73f9479c1ff77fbada8305e618931b57d65796"}}]}]
BACKPORT-->

Co-authored-by: Nathan Reese <[email protected]>
sebelga added a commit to sebelga/kibana that referenced this pull request Jul 17, 2023
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 Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.9.0 v8.10.0
Projects
None yet
5 participants