Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Feature/COR-1088-archive-button-styling #4508

Merged
merged 7 commits into from
Nov 28, 2022

Conversation

Amber-Taal-Work
Copy link
Contributor

@Amber-Taal-Work Amber-Taal-Work commented Nov 22, 2022

Summary

This PR addresses the changes described in COR-1088.
In this PR, the styling of the archived graphs button has been changed.

Before

Default state
Schermafbeelding 2022-11-22 om 15 56 49

Active state
Schermafbeelding 2022-11-22 om 15 56 54

Active state on focus

old_archived_button_active_focus

After:

Default state
Schermafbeelding 2022-11-22 om 15 57 30

Default state on hover
Schermafbeelding 2022-11-22 om 15 57 34

Default state on focus

archived_button_default_focus

Default state on hover with focus
archived_button_default_hover_focus

Active state
Schermafbeelding 2022-11-22 om 15 57 39

Active state on hover
Schermafbeelding 2022-11-22 om 15 57 42

Active state on focus
archived_button_active_focus

Active state on hover with focus
archived_button_active_hover_focus

@Amber-Taal-Work Amber-Taal-Work changed the title Feature/cor 1088 archive button styling Feature/COR-1088 archive button styling Nov 22, 2022
@Amber-Taal-Work Amber-Taal-Work changed the title Feature/COR-1088 archive button styling Feature/COR-1088-archive-button-styling Nov 22, 2022
@hasan-ozaynaci
Copy link
Contributor

hasan-ozaynaci commented Nov 23, 2022

Is the keyboard focus state visible for the light-blue and the dark-blue button? Because I can’t see it in the “after” screenshot.

@hasan-ozaynaci hasan-ozaynaci added the Hold This should not be merged (yet) label Nov 23, 2022
@hasan-ozaynaci hasan-ozaynaci removed the Hold This should not be merged (yet) label Nov 28, 2022
@hasan-ozaynaci hasan-ozaynaci merged commit 4293098 into develop Nov 28, 2022
@hasan-ozaynaci hasan-ozaynaci deleted the feature/COR-1088-archive-button-styling branch November 28, 2022 08:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants