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

feat(message-document): Use mask for status indicator #555

Merged
merged 1 commit into from
Jun 17, 2023

Conversation

yuraiz
Copy link
Contributor

@yuraiz yuraiz commented Jun 11, 2023

With mask it correctly matches message background

Screenshot from 2023-06-11 15-34-42
Screenshot from 2023-06-11 15-34-22

That PR doesn't include gradient bubbles

@yuraiz yuraiz force-pushed the mask-document-icon branch from 56498dd to 251fc27 Compare June 11, 2023 13:03
@melix99 melix99 force-pushed the mask-document-icon branch from 251fc27 to efd2e4d Compare June 13, 2023 17:37
@yuraiz yuraiz force-pushed the mask-document-icon branch from efd2e4d to 3c79c94 Compare June 13, 2023 18:38
@yuraiz yuraiz requested a review from marhkb June 13, 2023 18:39
@yuraiz yuraiz force-pushed the mask-document-icon branch 2 times, most recently from 978c19c to 786c4fe Compare June 14, 2023 19:47
@yuraiz yuraiz force-pushed the mask-document-icon branch from 786c4fe to 8d04f95 Compare June 17, 2023 12:03
@yuraiz yuraiz requested a review from marhkb June 17, 2023 12:05
@marhkb
Copy link
Collaborator

marhkb commented Jun 17, 2023

LGTM. @melix99 do you want to take a look at the css stuff anyway?

With mask it correctly matches message bubble background
@yuraiz yuraiz force-pushed the mask-document-icon branch from 8d04f95 to 199cb75 Compare June 17, 2023 20:15
@yuraiz yuraiz requested a review from melix99 June 17, 2023 20:15
@marhkb marhkb merged commit 6315d0d into paper-plane-developers:main Jun 17, 2023
@yuraiz yuraiz deleted the mask-document-icon branch June 18, 2023 09:42
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

Successfully merging this pull request may close these issues.

3 participants