Skip to content
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

[ES|QL] Improves the comments color in dark mode #203965

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

stratoula
Copy link
Contributor

@stratoula stratoula commented Dec 12, 2024

Summary

Closes #202152

Fixes the comments color in dark mode

image

@stratoula stratoula marked this pull request as ready for review December 12, 2024 07:58
@stratoula stratoula requested a review from a team as a code owner December 12, 2024 07:58
@stratoula stratoula added v8.17.0 v8.18.0 v9.0.0 Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana backport:version Backport to applied version labels and removed v8.17.0 labels Dec 12, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

@stratoula stratoula added the release_note:skip Skip the PR/issue when compiling release notes label Dec 12, 2024
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #3 / UserName should not open any flyout or panels if context in not defined

Metrics [docs]

Page load bundle

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

id before after diff
kbnUiSharedDeps-srcJs 3.5MB 3.5MB +3.0B

@stratoula stratoula merged commit 6921de3 into elastic:main Dec 12, 2024
24 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 203965

Questions ?

Please refer to the Backport tool documentation

@stratoula
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

stratoula added a commit to stratoula/kibana that referenced this pull request Dec 12, 2024
## Summary

Closes elastic#202152

Fixes the comments color in dark mode

<img width="928" alt="image"
src="https://github.com/user-attachments/assets/7d007cfa-f6d4-45e9-817d-57178c64d0a4"
/>

(cherry picked from commit 6921de3)

# Conflicts:
#	packages/kbn-monaco/src/esql/lib/esql_theme.ts
stratoula added a commit that referenced this pull request Dec 13, 2024
…4055)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Improves the comments color in dark mode
(#203965)](#203965)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Stratoula
Kalafateli","email":"efstratia.kalafateli@elastic.co"},"sourceCommit":{"committedDate":"2024-12-12T11:08:55Z","message":"[ES|QL]
Improves the comments color in dark mode (#203965)\n\n##
Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/202152\r\n\r\nFixes the
comments color in dark mode \r\n\r\n<img width=\"928\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7d007cfa-f6d4-45e9-817d-57178c64d0a4\"\r\n/>","sha":"6921de34988b43614438f7ca318ed9f3f109b1d7","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Feature:ES|QL","Team:ESQL","backport:version","v8.18.0"],"number":203965,"url":"https://github.com/elastic/kibana/pull/203965","mergeCommit":{"message":"[ES|QL]
Improves the comments color in dark mode (#203965)\n\n##
Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/202152\r\n\r\nFixes the
comments color in dark mode \r\n\r\n<img width=\"928\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7d007cfa-f6d4-45e9-817d-57178c64d0a4\"\r\n/>","sha":"6921de34988b43614438f7ca318ed9f3f109b1d7"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203965","number":203965,"mergeCommit":{"message":"[ES|QL]
Improves the comments color in dark mode (#203965)\n\n##
Summary\r\n\r\nCloses
https://github.com/elastic/kibana/issues/202152\r\n\r\nFixes the
comments color in dark mode \r\n\r\n<img width=\"928\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7d007cfa-f6d4-45e9-817d-57178c64d0a4\"\r\n/>","sha":"6921de34988b43614438f7ca318ed9f3f109b1d7"}},{"branch":"8.x","label":"v8.18.0","labelRegex":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 13, 2025
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:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes Team:ESQL ES|QL related features in Kibana v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Dashboards][ES|QL visualization] ES|QL comments in dark mode are difficult to read
4 participants