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 group screen has cluttered design #2008

Closed
greatvovan opened this issue May 24, 2023 · 2 comments
Closed

Alert group screen has cluttered design #2008

greatvovan opened this issue May 24, 2023 · 2 comments

Comments

@greatvovan
Copy link

The alert group screen has

  • title repeated 3 times,
  • message repeated 2 times,
  • timestamp repeated 3 times,

which deteriorates readability and increases amount of time to understand the problem.

Example:

Screenshot 2023-05-23 at 4 55 59 PM

Suggestions:

  1. Remove the second title along with the update timestamp.
    Rationale: the title is just a duplicate and update timestamp is shown below in the grouped alerts header (it's the same).

  2. Always open alert group card with grouped alerts panel collapsed.
    Rationale: the current state is the most relevant, while historical information is needed from occasion to occasion.

  3. Reduce font (and maybe change color or other style) in the grouped alerts panel.
    Rationale: once expanded, the panel should not detract attention from the main content. Additionally, the top element of the list is always a duplicate of the current content, so it should not collide with the main content and cause confusion.

Approximate new view (besides a smaller font):

image
@iskhakov
Copy link
Contributor

This makes sense, thank you for suggestions

github-merge-queue bot pushed a commit that referenced this issue Jun 21, 2023
…nsistent (#2295)

# What this PR does

This PR changes Alert Group layout according to the feedback from this
issue #2008

Also it makes time format more consistent across OnCall

## Which issue(s) this PR fixes

## Checklist

- [ ] Unit, integration, and e2e (if applicable) tests updated
- [ ] Documentation added (or `pr:no public docs` PR label added if not
required)
- [ ] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
@iskhakov
Copy link
Contributor

iskhakov commented Apr 2, 2024

fixed by 8bfaccf

@iskhakov iskhakov closed this as completed Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants