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

[ResponseOps][Cases] Design Review changes PR 1 #194560

Conversation

georgianaonoleata1904
Copy link
Contributor

@georgianaonoleata1904 georgianaonoleata1904 commented Oct 1, 2024

Connected to #188187

Summary

  • Field names have been aligned on top to ensure consistency with the fields area on the right
Screenshot 2024-10-01 at 14 30 38
  • Custom field and template name text size has been reduced to align with the badge
    Screenshot 2024-10-01 at 14 34 15

Copy link

cla-checker-service bot commented Oct 1, 2024

💚 CLA has been signed

@georgianaonoleata1904 georgianaonoleata1904 self-assigned this Oct 1, 2024
@georgianaonoleata1904 georgianaonoleata1904 changed the title [ResponseOps][Cases] Design Review changes #2, #3 [ResponseOps][Cases] Design Review changes for issue number 2 and issue number 3 Oct 1, 2024
@georgianaonoleata1904 georgianaonoleata1904 added Feature:Cases Cases feature v8.16.0 backport:skip This commit does not require backporting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) release_note:skip Skip the PR/issue when compiling release notes labels Oct 1, 2024
@georgianaonoleata1904 georgianaonoleata1904 marked this pull request as ready for review October 1, 2024 13:00
@georgianaonoleata1904 georgianaonoleata1904 requested a review from a team as a code owner October 1, 2024 13:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops-cases (Feature:Cases)

@georgianaonoleata1904 georgianaonoleata1904 force-pushed the design-review-issues-188187 branch from c9554b8 to 7f69875 Compare October 1, 2024 13:56
@georgianaonoleata1904 georgianaonoleata1904 changed the title [ResponseOps][Cases] Design Review changes for issue number 2 and issue number 3 [ResponseOps][Cases] Design Review changes PR 1 Oct 1, 2024
@adcoelho adcoelho added v9.0.0 backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) and removed backport:skip This commit does not require backporting labels Oct 1, 2024
Copy link
Contributor

@adcoelho adcoelho left a comment

Choose a reason for hiding this comment

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

Tested and works as expected 👍

Screenshot 2024-10-01 at 23 25 11

Copy link
Contributor

@js-jankisalvi js-jankisalvi left a comment

Choose a reason for hiding this comment

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

Congrats on the first PR!! 🎉
Verified locally, looks good 👍

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

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
cases 491.3KB 491.4KB +148.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @georgianaonoleata1904

@georgianaonoleata1904 georgianaonoleata1904 merged commit 616c6e8 into elastic:main Oct 2, 2024
23 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 2, 2024
Connected to elastic#188187

## Summary

- Field names have been aligned on top to ensure consistency with the
fields area on the right
<img width="1495" alt="Screenshot 2024-10-01 at 14 30 38"
src="https://github.com/user-attachments/assets/51578be1-6cd8-43e0-a800-61e562aa5d77">

- Custom field and template name text size has been reduced to align
with the badge
![Screenshot 2024-10-01 at 14 34
15](https://github.com/user-attachments/assets/986f9426-640e-4158-8614-662672f4ca72)

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 616c6e8)
@kibanamachine
Copy link
Contributor

💚 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

kibanamachine added a commit that referenced this pull request Oct 2, 2024
)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ResponseOps][Cases] Design Review changes PR 1
(#194560)](#194560)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Georgiana-Andreea
Onoleață","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-02T06:48:15Z","message":"[ResponseOps][Cases]
Design Review changes PR 1 (#194560)\n\nConnected to
https://github.com/elastic/kibana/issues/188187\r\n\r\n##
Summary\r\n\r\n- Field names have been aligned on top to ensure
consistency with the\r\nfields area on the right\r\n<img width=\"1495\"
alt=\"Screenshot 2024-10-01 at 14 30
38\"\r\nsrc=\"https://github.com/user-attachments/assets/51578be1-6cd8-43e0-a800-61e562aa5d77\">\r\n\r\n-
Custom field and template name text size has been reduced to
align\r\nwith the badge\r\n![Screenshot 2024-10-01 at 14
34\r\n15](https://github.com/user-attachments/assets/986f9426-640e-4158-8614-662672f4ca72)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"616c6e837fedeb5c15d62670c48072c1d6ea542b","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","v9.0.0","Feature:Cases","backport:prev-minor","v8.16.0"],"title":"[ResponseOps][Cases]
Design Review changes PR
1","number":194560,"url":"https://github.com/elastic/kibana/pull/194560","mergeCommit":{"message":"[ResponseOps][Cases]
Design Review changes PR 1 (#194560)\n\nConnected to
https://github.com/elastic/kibana/issues/188187\r\n\r\n##
Summary\r\n\r\n- Field names have been aligned on top to ensure
consistency with the\r\nfields area on the right\r\n<img width=\"1495\"
alt=\"Screenshot 2024-10-01 at 14 30
38\"\r\nsrc=\"https://github.com/user-attachments/assets/51578be1-6cd8-43e0-a800-61e562aa5d77\">\r\n\r\n-
Custom field and template name text size has been reduced to
align\r\nwith the badge\r\n![Screenshot 2024-10-01 at 14
34\r\n15](https://github.com/user-attachments/assets/986f9426-640e-4158-8614-662672f4ca72)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"616c6e837fedeb5c15d62670c48072c1d6ea542b"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194560","number":194560,"mergeCommit":{"message":"[ResponseOps][Cases]
Design Review changes PR 1 (#194560)\n\nConnected to
https://github.com/elastic/kibana/issues/188187\r\n\r\n##
Summary\r\n\r\n- Field names have been aligned on top to ensure
consistency with the\r\nfields area on the right\r\n<img width=\"1495\"
alt=\"Screenshot 2024-10-01 at 14 30
38\"\r\nsrc=\"https://github.com/user-attachments/assets/51578be1-6cd8-43e0-a800-61e562aa5d77\">\r\n\r\n-
Custom field and template name text size has been reduced to
align\r\nwith the badge\r\n![Screenshot 2024-10-01 at 14
34\r\n15](https://github.com/user-attachments/assets/986f9426-640e-4158-8614-662672f4ca72)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"616c6e837fedeb5c15d62670c48072c1d6ea542b"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Georgiana-Andreea Onoleață <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) Feature:Cases Cases feature release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants