-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Discover] Support grid column settings on Surrounding Documents page #177003
Conversation
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: cc @jughosta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx for fixing this 🙇 . Tested locally and resizing of columns and keeping this width works as expected. One ask could you describe what was fixed in the issue, many thx 👍
@kertal I updated the PR description. Is it what you meant? |
@jughosta yes, excactly, thx 👍 |
…elastic#177003) - Closes elastic#174485 ### Summary This PR keeps track of column resizing on Surrounding documents page: if user resizes columns and thne loads more documents, the grid will rerender accordingly and it will keep custom column widths. ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios (cherry picked from commit a482a7a)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…ts page (#177003) (#177231) # Backport This will backport the following commits from `main` to `8.13`: - [[Discover] Support grid column settings on Surrounding Documents page (#177003)](#177003) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Julia Rechkunova","email":"julia.rechkunova@elastic.co"},"sourceCommit":{"committedDate":"2024-02-19T18:56:05Z","message":"[Discover] Support grid column settings on Surrounding Documents page (#177003)\n\n- Closes https://github.com/elastic/kibana/issues/174485\r\n\r\n### Summary\r\n\r\nThis PR keeps track of column resizing on Surrounding documents page: if\r\nuser resizes columns and thne loads more documents, the grid will\r\nrerender accordingly and it will keep custom column widths.\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios","sha":"a482a7a98fc16a4efad27eac38dd67eafe4b64a6","branchLabelMapping":{"^v8.14.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:DataDiscovery","backport:prev-minor","v8.14.0"],"title":"[Discover] Support grid column settings on Surrounding Documents page","number":177003,"url":"https://github.com/elastic/kibana/pull/177003","mergeCommit":{"message":"[Discover] Support grid column settings on Surrounding Documents page (#177003)\n\n- Closes https://github.com/elastic/kibana/issues/174485\r\n\r\n### Summary\r\n\r\nThis PR keeps track of column resizing on Surrounding documents page: if\r\nuser resizes columns and thne loads more documents, the grid will\r\nrerender accordingly and it will keep custom column widths.\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios","sha":"a482a7a98fc16a4efad27eac38dd67eafe4b64a6"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.14.0","branchLabelMappingKey":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/177003","number":177003,"mergeCommit":{"message":"[Discover] Support grid column settings on Surrounding Documents page (#177003)\n\n- Closes https://github.com/elastic/kibana/issues/174485\r\n\r\n### Summary\r\n\r\nThis PR keeps track of column resizing on Surrounding documents page: if\r\nuser resizes columns and thne loads more documents, the grid will\r\nrerender accordingly and it will keep custom column widths.\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere updated or added to match the most common scenarios","sha":"a482a7a98fc16a4efad27eac38dd67eafe4b64a6"}}]}] BACKPORT--> Co-authored-by: Julia Rechkunova <julia.rechkunova@elastic.co>
…elastic#177003) - Closes elastic#174485 ### Summary This PR keeps track of column resizing on Surrounding documents page: if user resizes columns and thne loads more documents, the grid will rerender accordingly and it will keep custom column widths. ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
Summary
This PR keeps track of column resizing on Surrounding documents page: if user resizes columns and thne loads more documents, the grid will rerender accordingly and it will keep custom column widths.
Checklist