Skip to content

[TSVB/SCSS] Migrate _annotations_editor.scss file#220900

Merged
mariairiartef merged 6 commits intoelastic:mainfrom
mariairiartef:tsvb/scss-js/annotations_editor
May 29, 2025
Merged

[TSVB/SCSS] Migrate _annotations_editor.scss file#220900
mariairiartef merged 6 commits intoelastic:mainfrom
mariairiartef:tsvb/scss-js/annotations_editor

Conversation

@mariairiartef
Copy link
Contributor

@mariairiartef mariairiartef commented May 19, 2025 •
edited by kibanamachine
Loading

Summary

Part of #220095

Migrates _annotations_editor.scss file.

Considerations

For details on considerations, read this.

Checklist

  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
@mariairiartef mariairiartef self-assigned this May 19, 2025
@mariairiartef mariairiartef added Feature:TSVB TSVB (Time Series Visual Builder) Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// scss-removal labels May 19, 2025
@mariairiartef mariairiartef mentioned this pull request May 20, 2025
Closed
16 tasks
@mariairiartef mariairiartef added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting technical debt Improvement of the software architecture and operational architecture labels May 21, 2025
@mariairiartef mariairiartef added backport:version Backport to applied version labels v9.1.0 v8.19.0 and removed backport:skip This PR does not require backporting labels May 26, 2025
@mariairiartef mariairiartef marked this pull request as ready for review May 26, 2025 10:36
@mariairiartef mariairiartef requested review from a team as code owners May 26, 2025 10:36
@elasticmachine
Copy link
Contributor

elasticmachine commented May 26, 2025

Pinging @elastic/kibana-visualizations (Team:Visualizations)
markov00
markov00 approved these changes May 27, 2025
View reviewed changes
Copy link
Contributor

@markov00 markov00 left a comment

Choose a reason for hiding this comment

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

Tested locally, no visible style changes identified!
@mariairiartef mariairiartef enabled auto-merge (squash) May 28, 2025 15:48
@mariairiartef mariairiartef merged commit 7e8f578 into elastic:main May 29, 2025
10 checks passed
@kibanamachine
Copy link
Contributor

kibanamachine commented May 29, 2025

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/15321864901
@elasticmachine
Copy link
Contributor

elasticmachine commented May 29, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] x-pack/test/alerting_api_integration/security_and_spaces/group1/config.ts / alerting api integration security and spaces enabled Alerts - Group 1 alerts backfill rule runs ad hoc backfill task should handle timeouts

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
visTypeTimeseries 481 476 -5

Async chunks

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

id before after diff
visTypeTimeseries 451.4KB 450.4KB -1.0KB

History

cc @mariairiartef
@kibanamachine
Copy link
Contributor

kibanamachine commented May 29, 2025

💔 All backports failed

Status Branch Result
❌ 8.19 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.19:
- [Synthetics] Migrate synthetics alerting API tests to deployment agnostic solution (#221139)
- [Discover] Show ES|QL request URL in Inpector flyout (#221816)
- [Streams] Grok UI integration with enrichment page (#219146)
- Add windsurf to gitignore (#221801)
- [Alerting] Add snapshot telemetry for _ignored fields (#221480)
- Add conditional switching between EUI releases (#219818)
- [Security Solution] Integrations card in automatic migrations (#221493)
- [Lens] Optimize unnecessary re-renders (#220366)
- 🌊 Streams: Normalized format for wired streams (#205113)

Manual backport

To create the backport manually run:

node scripts/backport --pr 220900

Questions ?

Please refer to the Backport tool documentation
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
@mariairiartef @akowalska622
[TSVB/SCSS] Migrate _annotations_editor.scss file (elastic#220900)
d96b5e1 
## Summary

Part of elastic#220095

Migrates `_annotations_editor.scss` file.

### Considerations

For details on considerations, read
[this](elastic#220095 (comment)).

## Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
This was referenced May 29, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jun 2, 2025
@kibanamachine
Copy link
Contributor

kibanamachine commented Jun 2, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 220900 locally
cc: @mariairiartef
1 similar comment
@kibanamachine
Copy link
Contributor

kibanamachine commented Jun 3, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 220900 locally
cc: @mariairiartef
@mariairiartef mariairiartef mentioned this pull request Jun 3, 2025
Merged
@mariairiartef
Copy link
Contributor Author

mariairiartef commented Jun 3, 2025

💚 All backports created successfully

Status Branch Result
✅ 8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation
mariairiartef added a commit to mariairiartef/kibana that referenced this pull request Jun 3, 2025
@mariairiartef
[TSVB/SCSS] Migrate _annotations_editor.scss file (elastic#220900)
28357f6 
## Summary

Part of elastic#220095

Migrates `_annotations_editor.scss` file.

### Considerations

For details on considerations, read
[this](elastic#220095 (comment)).

## Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit 7e8f578)

# Conflicts:
#	src/platform/plugins/private/vis_types/timeseries/public/application/components/_annotations_editor.scss
mariairiartef added a commit that referenced this pull request Jun 3, 2025
@mariairiartef
[8.19] [TSVB/SCSS] Migrate _annotations_editor.scss file (#220900) (#…
2f87205 
…222349)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[TSVB/SCSS] Migrate `_annotations_editor.scss` file
(#220900)](#220900)

<!--- Backport version: 10.0.0 -->

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

<!--BACKPORT [{"author":{"name":"Maria
Iriarte","email":"106958839+mariairiartef@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-05-29T10:30:51Z","message":"[TSVB/SCSS]
Migrate `_annotations_editor.scss` file (#220900)\n\n## Summary\n\nPart
of https://github.com/elastic/kibana/issues/220095\n\nMigrates
`_annotations_editor.scss` file.\n\n### Considerations\n\nFor details on
considerations,
read\n[this](https://github.com/elastic/kibana/issues/220095#issuecomment-2911737679).\n\n##
Checklist\n\n- [x] The PR description includes the appropriate Release
Notes section,\nand the correct `release_note:*` label is applied per
the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"7e8f578a159dc30a81f1f0b344b2a424339e905c","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:TSVB","technical
debt","Team:Visualizations","release_note:skip","backport
missing","backport:version","v9.1.0","v8.19.0","scss-removal"],"title":"[TSVB/SCSS]
Migrate `_annotations_editor.scss`
file","number":220900,"url":"https://github.com/elastic/kibana/pull/220900","mergeCommit":{"message":"[TSVB/SCSS]
Migrate `_annotations_editor.scss` file (#220900)\n\n## Summary\n\nPart
of https://github.com/elastic/kibana/issues/220095\n\nMigrates
`_annotations_editor.scss` file.\n\n### Considerations\n\nFor details on
considerations,
read\n[this](https://github.com/elastic/kibana/issues/220095#issuecomment-2911737679).\n\n##
Checklist\n\n- [x] The PR description includes the appropriate Release
Notes section,\nand the correct `release_note:*` label is applied per
the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"7e8f578a159dc30a81f1f0b344b2a424339e905c"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/220900","number":220900,"mergeCommit":{"message":"[TSVB/SCSS]
Migrate `_annotations_editor.scss` file (#220900)\n\n## Summary\n\nPart
of https://github.com/elastic/kibana/issues/220095\n\nMigrates
`_annotations_editor.scss` file.\n\n### Considerations\n\nFor details on
considerations,
read\n[this](https://github.com/elastic/kibana/issues/220095#issuecomment-2911737679).\n\n##
Checklist\n\n- [x] The PR description includes the appropriate Release
Notes section,\nand the correct `release_note:*` label is applied per
the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"7e8f578a159dc30a81f1f0b344b2a424339e905c"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jun 3, 2025
zacharyparikh pushed a commit to zacharyparikh/kibana that referenced this pull request Jun 4, 2025
@mariairiartef @zacharyparikh
[TSVB/SCSS] Migrate _annotations_editor.scss file (elastic#220900)
009dc40 
## Summary

Part of elastic#220095

Migrates `_annotations_editor.scss` file.

### Considerations

For details on considerations, read
[this](elastic#220095 (comment)).

## Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
nickpeihl pushed a commit to nickpeihl/kibana that referenced this pull request Jun 12, 2025
@mariairiartef @nickpeihl
[TSVB/SCSS] Migrate _annotations_editor.scss file (elastic#220900)
56474cc 
## Summary

Part of elastic#220095

Migrates `_annotations_editor.scss` file.

### Considerations

For details on considerations, read
[this](elastic#220095 (comment)).

## Checklist

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
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 Feature:TSVB TSVB (Time Series Visual Builder) release_note:skip Skip the PR/issue when compiling release notes scss-removal Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// technical debt Improvement of the software architecture and operational architecture v8.19.0 v9.1.0

5 participants

@mariairiartef @elasticmachine @kibanamachine @ryankeairns @markov00