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

MsgBubbles: Stickers #20523

Closed
daniellekirkwood opened this issue Jan 12, 2022 · 3 comments · Fixed by matrix-org/matrix-react-sdk#7560
Closed

MsgBubbles: Stickers #20523

daniellekirkwood opened this issue Jan 12, 2022 · 3 comments · Fixed by matrix-org/matrix-react-sdk#7560

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Jan 12, 2022

Your use case

As part of the Message Bubbles project, Stickers should not into Bubbles.

@t3chguy
Copy link
Member

t3chguy commented Jan 13, 2022

One thing to consider is stickers are not unlike images with a transparent border (and a tooltip) so putting them in a bubble that may be inconsistent with sticker-like images which are sent as an m.image instead of m.sticker.
Related #20535

@amshakal
Copy link

Hi, okay based on the issue above and to remove complications. Let's move stickers outside the bubble. On doing some research on other platforms I see floating timestamp for stickers is quite common and perhaps a better tradeoff than having a gradient. The only place where it might not look good is if stick is the first message sent by a user. I am attaching below designs for the those scenarios. Aplogies for going back and forth on this issue. Thanks!

When sticker is the first message
Screenshot 2022-01-17 at 8 48 27 pm
Screenshot 2022-01-17 at 8 48 55 pm

When sticker is a message in between
Screenshot 2022-01-17 at 8 48 23 pm
Screenshot 2022-01-17 at 8 48 47 pm

When sticker is the last message
Screenshot 2022-01-17 at 8 43 29 pm
Screenshot 2022-01-17 at 8 49 02 pm

Link to figma: https://www.figma.com/file/zVtAKK0hzKu5JkpbzlE1pG/Message-Bubbles?node-id=2099%3A44213

@daniellekirkwood
Copy link
Contributor Author

Updated content of issue based on above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants