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

Redesign colors of quality incident status and severity colors for UUX #913

Closed
ds-ashanmugavel opened this issue Apr 30, 2024 · 6 comments · Fixed by #1120
Closed

Redesign colors of quality incident status and severity colors for UUX #913

ds-ashanmugavel opened this issue Apr 30, 2024 · 6 comments · Fixed by #1120
Assignees
Labels
5 - Frontend Refactoring frontend Frontend related Issues uux UUX feedback and related topics

Comments

@ds-ashanmugavel
Copy link
Contributor

As user ,
I want to have clear colors that indicate the state ,
so that I understand the intention of the colors .

Hints / Details

  • Feedback from Julia: Redesign colors from severity and state of notification states:
    -- used colors unclear - please recheck

Image

Image

For severity use the ones from our UI Concept: Miro

Image

Acceptance Criteria

  • redesign of colors of state and severity for quality incidents so that they match the intention of the action/state

Out of Scope

  • ...
@ds-ashanmugavel
Copy link
Contributor Author

I copied the ticket as the original tickets were deleted due to a spam account.
Orginal Ticket: #663

@ds-mmaul
Copy link
Contributor

severity icons changed with catenax-ng#1264

@ds-lcapellino ds-lcapellino mentioned this issue May 22, 2024
2 tasks
@mkanal mkanal moved this from inbox to backlog in Trace-X Jun 18, 2024
@mkanal
Copy link
Contributor

mkanal commented Jun 18, 2024

Please do a proposal and we have an alignment before the colors were implemented.

@mkanal mkanal moved this from backlog to next in Trace-X Jun 26, 2024
@ds-mmaul ds-mmaul moved this from next to wip in Trace-X Jun 26, 2024
@ds-mmaul ds-mmaul self-assigned this Jun 26, 2024
@ds-mmaul ds-mmaul moved this from wip to test in Trace-X Jun 26, 2024
@ds-mmaul
Copy link
Contributor

Waiting for feedback from @mkanal and @ds-crehm before merging. Proposal:

Image

@ds-mmaul ds-mmaul moved this from test to wip in Trace-X Jun 27, 2024
ds-mmaul added a commit that referenced this issue Jun 28, 2024
@ds-mmaul ds-mmaul moved this from wip to test in Trace-X Jun 28, 2024
ds-mwesener added a commit that referenced this issue Jun 28, 2024
…n-status-uux

chore(uux): 913 notification status colors change
@ds-crehm
Copy link
Contributor

ds-crehm commented Jul 2, 2024

Tested on E2E: All colors changed according to the second proprosal:
grafik

Small note: The CLOSED status has the same background color as the messages.
grafik

Acceptance criteria are met. Ready for review.

@ds-crehm ds-crehm reopened this Jul 2, 2024
@ds-crehm ds-crehm moved this from test to review in Trace-X Jul 2, 2024
@ds-crehm ds-crehm assigned mkanal and unassigned ds-mmaul and ds-crehm Jul 2, 2024
@mkanal
Copy link
Contributor

mkanal commented Jul 4, 2024

LGFM Po acceptance in behalf of @jzbmw

@mkanal mkanal closed this as completed Jul 4, 2024
@mkanal mkanal moved this from review to done in Trace-X Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Frontend Refactoring frontend Frontend related Issues uux UUX feedback and related topics
Projects
Status: done
Development

Successfully merging a pull request may close this issue.

4 participants