-
Notifications
You must be signed in to change notification settings - Fork 290
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: Improvements for Delivered state and add tests #17611
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #17611 +/- ##
==========================================
- Coverage 46.16% 46.16% -0.01%
==========================================
Files 761 762 +1
Lines 25023 25023
Branches 5724 5723 -1
==========================================
- Hits 11552 11551 -1
- Misses 12025 12028 +3
+ Partials 1446 1444 -2 |
@@ -31,7 +31,7 @@ export const messageBodyActions: CSSObject = { | |||
minHeight: '32px', | |||
minWidth: '40px', | |||
position: 'absolute', | |||
right: '16px', | |||
right: '-40px', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you explain what is happening here 🤔
Feels like a weird value
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's like that there, because there is added new container, and this MessageActions is displayed on message-body, not on whole container.
src/script/components/MessagesList/Message/ContentMessage/ContentMessage.tsx
Outdated
Show resolved
Hide resolved
|
Description
Added data-uie-values for qa for check if delivery message is working correctly.
Screenshots/Screencast (for UI changes)
Checklist
Important details for the reviewers
(Delete this section if unnecessary)