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

[Alert details page][Custom threshold] Adjust main chart title #174249

Closed
maryam-saeidi opened this issue Jan 4, 2024 · 2 comments · Fixed by #175865
Closed

[Alert details page][Custom threshold] Adjust main chart title #174249

maryam-saeidi opened this issue Jan 4, 2024 · 2 comments · Fixed by #175865
Assignees
Labels
Feature:Alert Details Page Observability ux management team Feature: Custom threshold Observability custom threshold rule type Team:obs-ux-management Observability Management User Experience Team v8.13.0

Comments

@maryam-saeidi
Copy link
Member

maryam-saeidi commented Jan 4, 2024

🍒 Summary

According to this design, we want to have a meaningful title for the main chart on the alert details page:

image

At the moment, we only show a custom title:

image

Note

Since in the custom threshold, we have multiple conditions/aggregation and multiple groups, we need to consider how we can make a meaningful title not only for one aggregation but also multiple ones.

Question

@maciejforcone, Can you please let us know what the title should look like for the following case?

  • Having multiple groups
    • container.id: container-0
    • container.name: container-name
    • host.name: host-0
    • host.mac: 00-00-5E-00-53-23
  • Multiple conditions (each condition will have a separate chart but the group by information is shared)
First condition Second confition
image image

🎯 Acceptance Criteria

  • Adjust main chart title
@maryam-saeidi maryam-saeidi added Feature:Alert Details Page Observability ux management team Team:obs-ux-management Observability Management User Experience Team Feature: Custom threshold Observability custom threshold rule type v8.13.0 labels Jan 4, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@maciejforcone
Copy link

I think about showing the equation as a title, and truncate the text when too long, otherwise user would need to go to the rule conditions to find it, and it takes unnecessary additional step/time.

image

@benakansara benakansara self-assigned this Jan 26, 2024
benakansara added a commit that referenced this issue Feb 7, 2024
…rts (#175865)

Resolves #174249

- Updates title of condition charts on Custom threshold alert details
page
- In case chart title length is more than 120 chars, title is shortened
with `...` at end
- Tooltip shows full title

<img width="1254" alt="Screenshot 2024-01-30 at 10 19 35"
src="https://github.com/elastic/kibana/assets/69037875/41cfe6d5-5bd4-41f7-a91f-c2f3bd25943d">
<img width="1259" alt="Screenshot 2024-01-30 at 10 19 49"
src="https://github.com/elastic/kibana/assets/69037875/a9d76bf5-7f94-445e-98bf-e3763d03bdb3">
<img width="1272" alt="Screenshot 2024-01-30 at 10 26 52"
src="https://github.com/elastic/kibana/assets/69037875/642153b1-a8c2-4992-b3fd-187ffe385ca2">
fkanout pushed a commit to fkanout/kibana that referenced this issue Feb 7, 2024
…rts (elastic#175865)

Resolves elastic#174249

- Updates title of condition charts on Custom threshold alert details
page
- In case chart title length is more than 120 chars, title is shortened
with `...` at end
- Tooltip shows full title

<img width="1254" alt="Screenshot 2024-01-30 at 10 19 35"
src="https://github.com/elastic/kibana/assets/69037875/41cfe6d5-5bd4-41f7-a91f-c2f3bd25943d">
<img width="1259" alt="Screenshot 2024-01-30 at 10 19 49"
src="https://github.com/elastic/kibana/assets/69037875/a9d76bf5-7f94-445e-98bf-e3763d03bdb3">
<img width="1272" alt="Screenshot 2024-01-30 at 10 26 52"
src="https://github.com/elastic/kibana/assets/69037875/642153b1-a8c2-4992-b3fd-187ffe385ca2">
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this issue Feb 15, 2024
…rts (elastic#175865)

Resolves elastic#174249

- Updates title of condition charts on Custom threshold alert details
page
- In case chart title length is more than 120 chars, title is shortened
with `...` at end
- Tooltip shows full title

<img width="1254" alt="Screenshot 2024-01-30 at 10 19 35"
src="https://github.com/elastic/kibana/assets/69037875/41cfe6d5-5bd4-41f7-a91f-c2f3bd25943d">
<img width="1259" alt="Screenshot 2024-01-30 at 10 19 49"
src="https://github.com/elastic/kibana/assets/69037875/a9d76bf5-7f94-445e-98bf-e3763d03bdb3">
<img width="1272" alt="Screenshot 2024-01-30 at 10 26 52"
src="https://github.com/elastic/kibana/assets/69037875/642153b1-a8c2-4992-b3fd-187ffe385ca2">
fkanout pushed a commit to fkanout/kibana that referenced this issue Mar 4, 2024
…rts (elastic#175865)

Resolves elastic#174249

- Updates title of condition charts on Custom threshold alert details
page
- In case chart title length is more than 120 chars, title is shortened
with `...` at end
- Tooltip shows full title

<img width="1254" alt="Screenshot 2024-01-30 at 10 19 35"
src="https://github.com/elastic/kibana/assets/69037875/41cfe6d5-5bd4-41f7-a91f-c2f3bd25943d">
<img width="1259" alt="Screenshot 2024-01-30 at 10 19 49"
src="https://github.com/elastic/kibana/assets/69037875/a9d76bf5-7f94-445e-98bf-e3763d03bdb3">
<img width="1272" alt="Screenshot 2024-01-30 at 10 26 52"
src="https://github.com/elastic/kibana/assets/69037875/642153b1-a8c2-4992-b3fd-187ffe385ca2">
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alert Details Page Observability ux management team Feature: Custom threshold Observability custom threshold rule type Team:obs-ux-management Observability Management User Experience Team v8.13.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants