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

fix: [Search:AppSearch:Engines page]Data from page disappears after clicking on Last updated date #201768

Merged
merged 10 commits into from
Dec 12, 2024

Conversation

alexwizp
Copy link
Contributor

@alexwizp alexwizp commented Nov 26, 2024

Closes: #200545
Closes: #8911

Description

When user clicks date button it should activate correct action, or open the new page. In this case, all data disappears from the page which is unexpected. Found while testing for accessibility, but applicable to all users.

What was changed?:

  1. Since LogsStream still uses euiStyled, and we use it in multiple places, we need to set up EuiThemeProvider and CellActionsProvider.

Screen:

image

@alexwizp alexwizp added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting v9.0.0 Project:Accessibility Team:Search labels Nov 26, 2024
@alexwizp
Copy link
Contributor Author

/ci

@elastic elastic deleted a comment from elasticmachine Nov 26, 2024
@alexwizp
Copy link
Contributor Author

/ci

@alexwizp alexwizp marked this pull request as ready for review November 26, 2024 15:43
@alexwizp alexwizp requested a review from a team as a code owner November 26, 2024 15:43
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-accessibility (Project:Accessibility)

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Nov 26, 2024
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@alexwizp alexwizp added backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) and removed Team:Search backport:skip This commit does not require backporting labels Nov 26, 2024
@alexwizp
Copy link
Contributor Author

@elastic/obs-ux-logs-team please review

@alexwizp alexwizp marked this pull request as draft December 10, 2024 12:21
@alexwizp
Copy link
Contributor Author

/ci

@elastic elastic deleted a comment from elasticmachine Dec 10, 2024
@elastic elastic deleted a comment from elasticmachine Dec 10, 2024
@alexwizp
Copy link
Contributor Author

/ci

@alexwizp alexwizp marked this pull request as ready for review December 10, 2024 22:00
@alexwizp alexwizp requested review from a team as code owners December 10, 2024 22:00
@alexwizp alexwizp requested a review from tonyghiani December 10, 2024 22:00
Copy link
Contributor

@tonyghiani tonyghiani left a comment

Choose a reason for hiding this comment

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

LGTM

@alexwizp
Copy link
Contributor Author

@elastic/kibana-core please review

@elasticmachine
Copy link
Contributor

elasticmachine commented Dec 12, 2024

💚 Build Succeeded

  • Buildkite Build
  • Commit: c701a03
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-201768-c701a03bd373

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
enterpriseSearch 2288 2309 +21

Async chunks

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

id before after diff
enterpriseSearch 2.6MB 2.6MB +1.9KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
enterpriseSearch 52.1KB 52.3KB +193.0B

History

@alexwizp alexwizp merged commit 359ac08 into elastic:main Dec 12, 2024
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12298122446

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 12, 2024
…licking on Last updated date (elastic#201768)

Closes: elastic#200545
Closes: [elastic#8911](elastic/search-team#8911)

## Description
When user clicks date button it should activate correct action, or open
the new page. In this case, all data disappears from the page which is
unexpected. Found while testing for accessibility, but applicable to all
users.

## What was changed?:

1. Since LogsStream still uses `euiStyled`, and we use it in multiple
places, we need to set up `EuiThemeProvider` and `CellActionsProvider`.

## Screen:

<img width="1226" alt="image"
src="https://github.com/user-attachments/assets/cab55d3b-b694-4228-a0a2-7cfa9e0ac16d">

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 359ac08)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

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 Dec 12, 2024
…fter clicking on Last updated date (#201768) (#204031)

# Backport

This will backport the following commits from `main` to `8.x`:
- [fix: [Search:AppSearch:Engines page]Data from page disappears after
clicking on Last updated date
(#201768)](#201768)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Alexey
Antonov","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-12T14:10:06Z","message":"fix:
[Search:AppSearch:Engines page]Data from page disappears after clicking
on Last updated date (#201768)\n\nCloses: #200545\r\nCloses:
[#8911](https://github.com/elastic/search-team/issues/8911)\r\n\r\n##
Description\r\nWhen user clicks date button it should activate correct
action, or open\r\nthe new page. In this case, all data disappears from
the page which is\r\nunexpected. Found while testing for accessibility,
but applicable to all\r\nusers.\r\n\r\n## What was changed?:\r\n\r\n1.
Since LogsStream still uses `euiStyled`, and we use it in
multiple\r\nplaces, we need to set up `EuiThemeProvider` and
`CellActionsProvider`.\r\n\r\n## Screen: \r\n\r\n<img width=\"1226\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/cab55d3b-b694-4228-a0a2-7cfa9e0ac16d\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"359ac089de749c792fb1910229e94d5155d67921","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Project:Accessibility","release_note:skip","v9.0.0","Team:Search","backport:prev-minor","ci:project-deploy-observability"],"title":"fix:
[Search:AppSearch:Engines page]Data from page disappears after clicking
on Last updated
date","number":201768,"url":"https://github.com/elastic/kibana/pull/201768","mergeCommit":{"message":"fix:
[Search:AppSearch:Engines page]Data from page disappears after clicking
on Last updated date (#201768)\n\nCloses: #200545\r\nCloses:
[#8911](https://github.com/elastic/search-team/issues/8911)\r\n\r\n##
Description\r\nWhen user clicks date button it should activate correct
action, or open\r\nthe new page. In this case, all data disappears from
the page which is\r\nunexpected. Found while testing for accessibility,
but applicable to all\r\nusers.\r\n\r\n## What was changed?:\r\n\r\n1.
Since LogsStream still uses `euiStyled`, and we use it in
multiple\r\nplaces, we need to set up `EuiThemeProvider` and
`CellActionsProvider`.\r\n\r\n## Screen: \r\n\r\n<img width=\"1226\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/cab55d3b-b694-4228-a0a2-7cfa9e0ac16d\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"359ac089de749c792fb1910229e94d5155d67921"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201768","number":201768,"mergeCommit":{"message":"fix:
[Search:AppSearch:Engines page]Data from page disappears after clicking
on Last updated date (#201768)\n\nCloses: #200545\r\nCloses:
[#8911](https://github.com/elastic/search-team/issues/8911)\r\n\r\n##
Description\r\nWhen user clicks date button it should activate correct
action, or open\r\nthe new page. In this case, all data disappears from
the page which is\r\nunexpected. Found while testing for accessibility,
but applicable to all\r\nusers.\r\n\r\n## What was changed?:\r\n\r\n1.
Since LogsStream still uses `euiStyled`, and we use it in
multiple\r\nplaces, we need to set up `EuiThemeProvider` and
`CellActionsProvider`.\r\n\r\n## Screen: \r\n\r\n<img width=\"1226\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/cab55d3b-b694-4228-a0a2-7cfa9e0ac16d\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"359ac089de749c792fb1910229e94d5155d67921"}}]}]
BACKPORT-->

Co-authored-by: Alexey Antonov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:project-deploy-observability Create an Observability project Project:Accessibility release_note:skip Skip the PR/issue when compiling release notes Team:Search v8.18.0 v9.0.0
Projects
None yet
6 participants