Message theme v2 #300
Merged
Message theme v2 #300
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
szuperaz
force-pushed
the
message-theme-v2
branch
from
1e3abc2 to
19fcdaa
Compare
petyosi
approved these changes
Jun 8, 2022
|
🎉 This PR is included in version 4.0.0-theming-v2.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 4.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Leftover issue with enhancements: GetStream/stream-chat-css#115
Text message
Figma
Implementation
Read state
Figma
Implementation
Note: Matching the read state UI is part of the enhancement issue GetStream/stream-chat-css#115
Error message
Figma
Implementation
Note: Error message is displayed for UX reasons
Inline reply
Figma
Implementation
Note We should apply "own message" styling to the original message when user replies to own message, this is part of the enhancement issue: GetStream/stream-chat-css#115
System message
Figma
Implementation
Context menu
Figma
Implementation
Note Missing parts are included in the enhancement issue: GetStream/stream-chat-css#115, context menu currently has a static position -> it can easily overflow -> I'll spend some limited time to implement dynamic positioning, this will come in a separate PR if successful
Image attachment
Figma
Implementation
Note Image carousel modal will be part of this task: GetStream/stream-chat-css#97
File attachment
Figma
Implementation
Note Angular SDK doesn't yet support different file icons for different mime-types
Video attachment + mixed attachments
Figma
Implementation
Giphy
Figma
Implementation
Note Hiding message bubble is part of the enhancement issue GetStream/stream-chat-css#115
Links
Figma
Implementation
Note Links are attachments, and attachments are displayed above the message text, apart from that the missing design elements are part of the enhancement issue GetStream/stream-chat-css#115
Thread links
Figma
Implementation
Note Read data is not available to thread replies
Thread header
Figma
Implementation
closes #282