Skip to content
This repository has been archived by the owner on Dec 14, 2021. It is now read-only.

Use red trash icon and fix deleted string #1146

Merged
merged 2 commits into from
Jan 28, 2020
Merged

Conversation

eliserichards
Copy link
Contributor

@eliserichards eliserichards commented Jan 23, 2020

Changes the toast notification that is shown after you delete an item. In order to match the designs, the icon needed to be changed to the red trash can, and the string should read "deleted" and not "deleted." with a period. This PR also fixed the padding between the toast icon and text as well as the vertical placement of the toast notification being above the create + button on the Item List.

Design: https://app.zeplin.io/project/5b6895bfe4af825140aa8dbc/screen/5c548fc44e168142a92bee1d

image

Copy link
Contributor

@changecourse changecourse left a comment

Choose a reason for hiding this comment

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

padding between toasts and icon should be 8px.

@eliserichards eliserichards merged commit ade612d into master Jan 28, 2020
@eliserichards eliserichards added the QA-needed When needing QA Verification/Investigation label Jan 28, 2020
@eliserichards eliserichards added this to the 4.0.0 🆕 milestone Jan 29, 2020
@abodea
Copy link
Contributor

abodea commented Jan 31, 2020

Verified as fixed on v.4.0.0(6016) using Google Pixel 3 XL(Android 9), Samsung Galaxy S10+(Android 9).

@abodea abodea added QA-verified For the issues verified by QA and removed QA-needed When needing QA Verification/Investigation labels Jan 31, 2020
@eliserichards eliserichards deleted the delete-toast branch February 11, 2020 15:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-delete QA-verified For the issues verified by QA ux-debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants