Skip to content

Fix data grid full screen stacking#267339

Merged
tcalopes merged 2 commits into
elastic:mainfrom
tcalopes:265746-agent-chat-with-entities-table-in-full-screen
May 6, 2026
Merged

Fix data grid full screen stacking#267339
tcalopes merged 2 commits into
elastic:mainfrom
tcalopes:265746-agent-chat-with-entities-table-in-full-screen

Conversation

@tcalopes
Copy link
Copy Markdown
Contributor

@tcalopes tcalopes commented May 1, 2026

Summary

When entering full-screen mode on the entities table and opening the Agent Builder chat, the chant sidebar and its resize handle were hidden behind the DataGrid overlay but still account for when positioning the flyout

  • use_full_screen_watcher.ts + layout_sidebar.tsx: introduced a data-kbn-preserve-zindex attribute to exclude elements from the full-screen z-index reset. This is added to the Chrome sidebar to preserve its stacking context
  • grid_global_app_style.tsx: added a constrain to the grid's right edge (var(--kbn-layout--sidebar-width)), so the full-screen DataGrid no longer covers the sidebar column when the Agent Builder chat is open

How to test

  1. Start Elasticsearch locally

  2. Start Kibana locally

    feature flags used
     uiSettings:
       overrides:
         agentBuilder:experimentalFeatures: true
         securitySolution:entityStoreEnableV2: true
     
     xpack.securitySolution.enableExperimental:
       - entityAnalyticsEntityStoreV2
       - entityAnalyticsNewHomePageEnabled
    
  3. Add at least one entity to test

    Kibana Dev Tools
       POST .entities.v2.latest.security_default-00001/_doc
       {
         "@timestamp": "2026-05-01T12:00:00.000Z",
         "user": { "name": "test.user" },
         "entity": {
           "name": "test.user",
           "type": "Identity",
           "id": "user:test.user",
           "EngineMetadata": { "Type": "user", "UntypedId": "test.user" }
         }
       }
    
  4. Navigate to Security → Entity Analytics

  5. Scroll to the entities table and click the full-screen icon

  6. Click any entity row to open the entity flyout

  7. Click Add to chat: click on expand details -> risk contribution to see the button

  8. Confirm that the Agent Builder sidebar is visible and the resize handle on the sidebar's left edge is visible and draggable

Before

Screenshot 2026-05-01 at 5 08 58 PM

After

Screen.Recording.2026-05-01.at.4.56.21.PM.mov

@tcalopes tcalopes self-assigned this May 1, 2026
@tcalopes tcalopes added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:Entity Analytics Security Entity Analytics Team labels May 1, 2026
@tcalopes tcalopes marked this pull request as ready for review May 4, 2026 14:18
@tcalopes tcalopes requested review from a team as code owners May 4, 2026 14:18
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/security-entity-analytics (Team:Entity Analytics)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 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
alertingVTwo 724.1KB 724.1KB +72.0B
cloudSecurityPosture 613.7KB 613.7KB +72.0B
discover 1.9MB 1.9MB +72.0B
esql 896.0KB 896.1KB +72.0B
esqlDataGrid 153.6KB 153.7KB +72.0B
osquery 1.3MB 1.3MB +72.0B
securitySolution 12.0MB 12.0MB +72.0B
slo 1.2MB 1.2MB +72.0B
total +576.0B

Page load bundle

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

id before after diff
core 522.1KB 522.1KB +76.0B

History

cc @tcalopes

Copy link
Copy Markdown
Contributor

@davismcphee davismcphee left a comment

Choose a reason for hiding this comment

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

Code-only review, Data Discovery changes LGTM 👍 Unfortunate full screen data grid handling is so tricky, but not sure what else could be done here.

@bhavyarm bhavyarm self-requested a review May 5, 2026 14:05
Copy link
Copy Markdown
Contributor

@bhavyarm bhavyarm left a comment

Choose a reason for hiding this comment

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

LGTM

@tcalopes tcalopes added backport:version Backport to applied version labels v9.4.1 and removed backport:skip This PR does not require backporting labels May 6, 2026
@tcalopes tcalopes merged commit 083666e into elastic:main May 6, 2026
60 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.4

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

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
9.4

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 May 6, 2026
# Backport

This will backport the following commits from `main` to `9.4`:
- [Fix data grid full screen stacking
(#267339)](#267339)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT
[{"author":{"name":"tcalopes","email":"tatiana.lopes@elastic.co"},"sourceCommit":{"committedDate":"2026-05-06T13:22:07Z","message":"Fix
data grid full screen stacking (#267339)\n\n## Summary\nWhen entering
full-screen mode on the entities table and opening the\nAgent Builder
chat, the chant sidebar and its resize handle were hidden\nbehind the
DataGrid overlay but still account for when positioning the\nflyout\n\n-
`use_full_screen_watcher.ts` + `layout_sidebar.tsx`: introduced
a\n`data-kbn-preserve-zindex` attribute to exclude elements from
the\nfull-screen z-index reset. This is added to the Chrome sidebar
to\npreserve its stacking context\n- `grid_global_app_style.tsx`: added
a constrain to the grid's right\nedge
(_var(--kbn-layout--sidebar-width)_), so the full-screen DataGrid\nno
longer covers the sidebar column when the Agent Builder chat is
open\n\n## How to test\n\n1. Start Elasticsearch locally\n2. Start
Kibana locally\n <details>\n <summary>feature flags used</summary>\n \n
uiSettings:\n overrides:\n agentBuilder:experimentalFeatures: true\n
securitySolution:entityStoreEnableV2: true\n \n
xpack.securitySolution.enableExperimental:\n -
entityAnalyticsEntityStoreV2\n - entityAnalyticsNewHomePageEnabled\n
</details>\n3. Add at least one entity to test\n <details>\n
<summary>Kibana Dev Tools</summary>\n\n POST
.entities.v2.latest.security_default-00001/_doc\n {\n \"@timestamp\":
\"2026-05-01T12:00:00.000Z\",\n \"user\": { \"name\": \"test.user\" },\n
\"entity\": {\n \"name\": \"test.user\",\n \"type\": \"Identity\",\n
\"id\": \"user:test.user\",\n\"EngineMetadata\": { \"Type\": \"user\",
\"UntypedId\": \"test.user\" }\n }\n }\n </details>\n4. Navigate to
Security → Entity Analytics\n5. Scroll to the entities table and click
the full-screen icon\n6. Click any entity row to open the entity
flyout\n7. Click _Add to chat_: click on expand details -> risk
contribution to\nsee the button\n8. Confirm that the Agent Builder
sidebar is visible and the resize\nhandle on the sidebar's left edge is
visible and draggable\n\n### Before\n\n<img width=\"1917\"
height=\"842\" alt=\"Screenshot 2026-05-01 at 5 08
58 PM\"\nsrc=\"https://github.com/user-attachments/assets/1deaa2f3-b753-4092-9fc9-ade31879f3f3\"\n/>\n\n###
After\n\n\nhttps://github.com/user-attachments/assets/829a60fc-80cc-4318-98a8-a69a65a4f89e","sha":"083666ed433da868f10af3c2d780c9f65d84f232","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Entity
Analytics","backport:version","v9.5.0","v9.4.1"],"title":"Fix data grid
full screen
stacking","number":267339,"url":"https://github.com/elastic/kibana/pull/267339","mergeCommit":{"message":"Fix
data grid full screen stacking (#267339)\n\n## Summary\nWhen entering
full-screen mode on the entities table and opening the\nAgent Builder
chat, the chant sidebar and its resize handle were hidden\nbehind the
DataGrid overlay but still account for when positioning the\nflyout\n\n-
`use_full_screen_watcher.ts` + `layout_sidebar.tsx`: introduced
a\n`data-kbn-preserve-zindex` attribute to exclude elements from
the\nfull-screen z-index reset. This is added to the Chrome sidebar
to\npreserve its stacking context\n- `grid_global_app_style.tsx`: added
a constrain to the grid's right\nedge
(_var(--kbn-layout--sidebar-width)_), so the full-screen DataGrid\nno
longer covers the sidebar column when the Agent Builder chat is
open\n\n## How to test\n\n1. Start Elasticsearch locally\n2. Start
Kibana locally\n <details>\n <summary>feature flags used</summary>\n \n
uiSettings:\n overrides:\n agentBuilder:experimentalFeatures: true\n
securitySolution:entityStoreEnableV2: true\n \n
xpack.securitySolution.enableExperimental:\n -
entityAnalyticsEntityStoreV2\n - entityAnalyticsNewHomePageEnabled\n
</details>\n3. Add at least one entity to test\n <details>\n
<summary>Kibana Dev Tools</summary>\n\n POST
.entities.v2.latest.security_default-00001/_doc\n {\n \"@timestamp\":
\"2026-05-01T12:00:00.000Z\",\n \"user\": { \"name\": \"test.user\" },\n
\"entity\": {\n \"name\": \"test.user\",\n \"type\": \"Identity\",\n
\"id\": \"user:test.user\",\n\"EngineMetadata\": { \"Type\": \"user\",
\"UntypedId\": \"test.user\" }\n }\n }\n </details>\n4. Navigate to
Security → Entity Analytics\n5. Scroll to the entities table and click
the full-screen icon\n6. Click any entity row to open the entity
flyout\n7. Click _Add to chat_: click on expand details -> risk
contribution to\nsee the button\n8. Confirm that the Agent Builder
sidebar is visible and the resize\nhandle on the sidebar's left edge is
visible and draggable\n\n### Before\n\n<img width=\"1917\"
height=\"842\" alt=\"Screenshot 2026-05-01 at 5 08
58 PM\"\nsrc=\"https://github.com/user-attachments/assets/1deaa2f3-b753-4092-9fc9-ade31879f3f3\"\n/>\n\n###
After\n\n\nhttps://github.com/user-attachments/assets/829a60fc-80cc-4318-98a8-a69a65a4f89e","sha":"083666ed433da868f10af3c2d780c9f65d84f232"}},"sourceBranch":"main","suggestedTargetBranches":["9.4"],"targetPullRequestStates":[{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/267339","number":267339,"mergeCommit":{"message":"Fix
data grid full screen stacking (#267339)\n\n## Summary\nWhen entering
full-screen mode on the entities table and opening the\nAgent Builder
chat, the chant sidebar and its resize handle were hidden\nbehind the
DataGrid overlay but still account for when positioning the\nflyout\n\n-
`use_full_screen_watcher.ts` + `layout_sidebar.tsx`: introduced
a\n`data-kbn-preserve-zindex` attribute to exclude elements from
the\nfull-screen z-index reset. This is added to the Chrome sidebar
to\npreserve its stacking context\n- `grid_global_app_style.tsx`: added
a constrain to the grid's right\nedge
(_var(--kbn-layout--sidebar-width)_), so the full-screen DataGrid\nno
longer covers the sidebar column when the Agent Builder chat is
open\n\n## How to test\n\n1. Start Elasticsearch locally\n2. Start
Kibana locally\n <details>\n <summary>feature flags used</summary>\n \n
uiSettings:\n overrides:\n agentBuilder:experimentalFeatures: true\n
securitySolution:entityStoreEnableV2: true\n \n
xpack.securitySolution.enableExperimental:\n -
entityAnalyticsEntityStoreV2\n - entityAnalyticsNewHomePageEnabled\n
</details>\n3. Add at least one entity to test\n <details>\n
<summary>Kibana Dev Tools</summary>\n\n POST
.entities.v2.latest.security_default-00001/_doc\n {\n \"@timestamp\":
\"2026-05-01T12:00:00.000Z\",\n \"user\": { \"name\": \"test.user\" },\n
\"entity\": {\n \"name\": \"test.user\",\n \"type\": \"Identity\",\n
\"id\": \"user:test.user\",\n\"EngineMetadata\": { \"Type\": \"user\",
\"UntypedId\": \"test.user\" }\n }\n }\n </details>\n4. Navigate to
Security → Entity Analytics\n5. Scroll to the entities table and click
the full-screen icon\n6. Click any entity row to open the entity
flyout\n7. Click _Add to chat_: click on expand details -> risk
contribution to\nsee the button\n8. Confirm that the Agent Builder
sidebar is visible and the resize\nhandle on the sidebar's left edge is
visible and draggable\n\n### Before\n\n<img width=\"1917\"
height=\"842\" alt=\"Screenshot 2026-05-01 at 5 08
58 PM\"\nsrc=\"https://github.com/user-attachments/assets/1deaa2f3-b753-4092-9fc9-ade31879f3f3\"\n/>\n\n###
After\n\n\nhttps://github.com/user-attachments/assets/829a60fc-80cc-4318-98a8-a69a65a4f89e","sha":"083666ed433da868f10af3c2d780c9f65d84f232"}},{"branch":"9.4","label":"v9.4.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: tcalopes <tatiana.lopes@elastic.co>
ersin-erdal pushed a commit to ersin-erdal/kibana that referenced this pull request May 6, 2026
## Summary
When entering full-screen mode on the entities table and opening the
Agent Builder chat, the chant sidebar and its resize handle were hidden
behind the DataGrid overlay but still account for when positioning the
flyout

- `use_full_screen_watcher.ts` + `layout_sidebar.tsx`: introduced a
`data-kbn-preserve-zindex` attribute to exclude elements from the
full-screen z-index reset. This is added to the Chrome sidebar to
preserve its stacking context
- `grid_global_app_style.tsx`: added a constrain to the grid's right
edge (_var(--kbn-layout--sidebar-width)_), so the full-screen DataGrid
no longer covers the sidebar column when the Agent Builder chat is open

## How to test

1. Start Elasticsearch locally
2. Start Kibana locally
      <details>
        <summary>feature flags used</summary>
   
        uiSettings:
          overrides:
            agentBuilder:experimentalFeatures: true
            securitySolution:entityStoreEnableV2: true
        
        xpack.securitySolution.enableExperimental:
          - entityAnalyticsEntityStoreV2
          - entityAnalyticsNewHomePageEnabled
      </details>
3. Add at least one entity to test
    <details>
          <summary>Kibana Dev Tools</summary>

          POST .entities.v2.latest.security_default-00001/_doc
          {
            "@timestamp": "2026-05-01T12:00:00.000Z",
            "user": { "name": "test.user" },
            "entity": {
              "name": "test.user",
              "type": "Identity",
              "id": "user:test.user",
"EngineMetadata": { "Type": "user", "UntypedId": "test.user" }
            }
          }
      </details>
4. Navigate to Security → Entity Analytics
5. Scroll to the entities table and click the full-screen icon
6. Click any entity row to open the entity flyout
7. Click _Add to chat_: click on expand details -> risk contribution to
see the button
8. Confirm that the Agent Builder sidebar is visible and the resize
handle on the sidebar's left edge is visible and draggable

### Before

<img width="1917" height="842" alt="Screenshot 2026-05-01 at 5 08 58 PM"
src="https://github.com/user-attachments/assets/1deaa2f3-b753-4092-9fc9-ade31879f3f3"
/>

### After


https://github.com/user-attachments/assets/829a60fc-80cc-4318-98a8-a69a65a4f89e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Entity Analytics Security Entity Analytics Team v9.4.0 v9.4.1 v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants