Skip to content

[ResponseOps] Fix rules list layout on narrow screens#280871

Draft
sophiec20 wants to merge 5 commits into
elastic:mainfrom
sophiec20:fix/rules-list-narrow-responsive
Draft

[ResponseOps] Fix rules list layout on narrow screens#280871
sophiec20 wants to merge 5 commits into
elastic:mainfrom
sophiec20:fix/rules-list-narrow-responsive

Conversation

@sophiec20

@sophiec20 sophiec20 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #280865.

Prevents the rules list table from switching into EUI's responsive card layout at narrow viewport widths. This keeps the rules list in table form and allows horizontal scrolling below 690px. Also tried to tidy up the controls between the filters and the table with more spacing and more sensible responsive behaviour.

After
image

Testing

  • node scripts/eslint x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/rules_list/components/rules_list_table.tsx
  • node scripts/scout.js run-tests --arch stateful --domain classic --testFiles x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rules_home_page.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rules_page_navigation.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rules_page_tabs.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rules_create_flow.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/edit_rule_flow.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rules_bulk_actions.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rules_settings.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rule_logs_spaces_switch.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rule_action_alerts_filter.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/rule_snooze_scheduler_legacy_theme.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/per_alert_snooze.spec.ts,x-pack/platform/plugins/shared/triggers_actions_ui/test/scout/ui/tests/alert_deletion.spec.ts

Manual verification

  • Verified the rules list remains a horizontally scrollable table at 500px viewport width.
  • node scripts/jest x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/rules_list/components/rules_list_auto_refresh.test.tsx x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/rules_list/components/rules_list.test.tsx

@infra-vault-gh-plugin-prod

infra-vault-gh-plugin-prod Bot commented Jul 24, 2026

Copy link
Copy Markdown
🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!
  • Click to trigger kibana-entity-store-performance-from-pr for this PR!
  • Click to trigger kibana-storybooks-from-pr for this PR!

@sophiec20 sophiec20 added the ci:project-deploy-observability Create an Observability project label Jul 24, 2026
@github-actions

Copy link
Copy Markdown
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!)

@sophiec20
sophiec20 force-pushed the fix/rules-list-narrow-responsive branch from 2a0906b to 7ff1ed7 Compare July 24, 2026 19:28
@kibanamachine

Copy link
Copy Markdown
Contributor

Project deployed, see credentials at: https://buildkite.com/elastic/kibana-deploy-project-from-pr/builds/1373

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

  • Buildkite Build
  • Commit: 2a0906b
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-280871-2a0906b4dc55

History

@elastic-vault-github-plugin-prod

This comment was marked as outdated.

@sophiec20
sophiec20 force-pushed the fix/rules-list-narrow-responsive branch from 7ff1ed7 to 21b95c4 Compare July 24, 2026 19:46
@sophiec20
sophiec20 force-pushed the fix/rules-list-narrow-responsive branch from 4956c48 to f1a8381 Compare July 25, 2026 11:05
@kibanamachine

kibanamachine commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: f1a8381
  • Build duration: 59 mins
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-280871-f1a83819c42c

Failed CI Steps

Test Failures

  • [job] [logs] Scout Lane #11 - stateful-classic / default / local-stateful-classic - Entity attachment cases – flyout add-to-case actions and Entities accordion - Entities accordion renders when entity attachments were added via API
  • [job] [logs] Scout Lane #11 - stateful-classic / default / local-stateful-classic - Entity attachment cases – flyout add-to-case actions and Entities accordion - renders no Entities accordion when a case has no entity attachments

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
triggersActionsUi 2.8MB 2.8MB +206.0B

History

@sophiec20 sophiec20 removed the ci:project-deploy-observability Create an Observability project label Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ResponseOps] Rules list not responsive at narrow widths

2 participants