Skip to content

[9.4] [One Workflow] Use EUI primary tint for word occurrence highlight in YAML editor (#267667)#268532

Merged
kibanamachine merged 1 commit into
elastic:9.4from
kibanamachine:backport/9.4/pr-267667
May 8, 2026
Merged

[9.4] [One Workflow] Use EUI primary tint for word occurrence highlight in YAML editor (#267667)#268532
kibanamachine merged 1 commit into
elastic:9.4from
kibanamachine:backport/9.4/pr-267667

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.4:

Questions ?

Please refer to the Backport tool documentation

…YAML editor (elastic#267667)

Before
<img width="370" height="235" alt="image"
src="https://github.com/user-attachments/assets/98e6bc67-e6ab-40a0-98eb-c012774a26c4"
/>

After
<img width="410" height="264" alt="image"
src="https://github.com/user-attachments/assets/639634ef-717f-4b59-bc69-5e03c416cfea"
/>

## Summary

Replaces the default gray Monaco word-occurrence highlight (visible in
the workflows YAML editor when the cursor sits on a token like `name` or
`type`) with `euiTheme.colors.backgroundLightPrimary`, so the highlight
matches the EUI primary accents already used elsewhere in the editor
(selection, hover) instead of reading as a generic gray box.

The override is scoped via `[data-test-subj='workflowYamlEditor']` so it
only affects the workflows YAML editor — no impact on other Monaco
editors in Kibana.

### Before / After

- **Before:** Monaco's default neutral gray `wordHighlightText`
background.
- **After:** EUI `backgroundLightPrimary`, visually consistent with the
rest of the editor's primary-tinted highlights.

## Test plan

- [ ] Open a workflow in the workflows YAML editor.
- [ ] Place the cursor on a YAML key (e.g. `name`, `type`) — all
matching occurrences should be highlighted in soft EUI primary blue, not
gray.
- [ ] Verify the change does NOT bleed into other Monaco-based editors
in Kibana (Console, Lens formula bar, etc.).
- [ ] Verify in both light and dark EUI themes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
(cherry picked from commit 090eedf)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 8, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) May 8, 2026 18:43
@botelastic botelastic Bot added the Team:One Workflow Team label for One Workflow (Workflow automation) label May 8, 2026
@kibanamachine
Copy link
Copy Markdown
Contributor Author

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Scout Lane #6 - stateful-classic / default / local-stateful-classic - Discover app - errors - should render invalid scripted field error
  • [job] [logs] Scout Lane #8 - stateful-classic / default / local-stateful-classic - Observability Landing Page (discover.isEsqlDefault enabled) - redirects to onboarding when no logs data exists

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
workflowsManagement 2.3MB 2.3MB +268.0B

cc @r4zr32d3k1l

@kibanamachine kibanamachine merged commit cd4de85 into elastic:9.4 May 8, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:One Workflow Team label for One Workflow (Workflow automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants