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

Fix styles of ActionsListItem #2587

Merged
merged 1 commit into from
Jul 6, 2021
Merged

Conversation

ArmandoGraterol
Copy link
Contributor

Updated styles of ActionsListItem status

FireShot Capture 374 - Colony - localhost

Resolves DEV-450

@ArmandoGraterol ArmandoGraterol requested a review from a team July 5, 2021 18:26
@ArmandoGraterol ArmandoGraterol self-assigned this Jul 5, 2021
@linear
Copy link

linear bot commented Jul 5, 2021

Copy link
Contributor

@alicjakujawa alicjakujawa left a comment

Choose a reason for hiding this comment

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

Screenshot 2021-07-06 at 10 42 52

Did we implement those status lines for motions and we lost them as well? Or we didint implement it at all? @rdig

Copy link
Contributor

@chinins chinins left a comment

Choose a reason for hiding this comment

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

All good!

@rdig
Copy link
Member

rdig commented Jul 6, 2021

Screenshot 2021-07-06 at 10 42 52

Did we implement those status lines for motions and we lost them as well? Or we didint implement it at all? @rdig

No we did not. Only recovery mode has them for now. When I get around to putting Motions on the Graph I'll see if I can add those notifications too.

In the mean time, this is all fine

@ArmandoGraterol ArmandoGraterol force-pushed the bug/action-list-item-status branch from 1eed547 to b3d44e6 Compare July 6, 2021 14:36
@ArmandoGraterol ArmandoGraterol merged commit ef4eb20 into master Jul 6, 2021
@ArmandoGraterol ArmandoGraterol deleted the bug/action-list-item-status branch July 6, 2021 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants