-
Notifications
You must be signed in to change notification settings - Fork 55
fix(ChatMessage): prevent hover over invisible actions area #953
Conversation
Generated by 🚫 dangerJS |
Codecov Report
@@ Coverage Diff @@
## master #953 +/- ##
==========================================
+ Coverage 80.5% 80.51% +<.01%
==========================================
Files 659 659
Lines 8450 8447 -3
Branches 1493 1428 -65
==========================================
- Hits 6803 6801 -2
+ Misses 1632 1631 -1
Partials 15 15
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #953 +/- ##
==========================================
- Coverage 80.96% 80.94% -0.02%
==========================================
Files 664 664
Lines 8496 8498 +2
Branches 1431 1433 +2
==========================================
Hits 6879 6879
- Misses 1602 1604 +2
Partials 15 15
Continue to review full report at Codecov.
|
@jurokapsiar, thanks a lot for spotting that - have updated styles to address these issues: submenu's accessibility |
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.
Don't forget about a changelog entry 🐱
TODO
This PR fixes undesired effect when actions menu is showed up while user hovering over invisible to her area. This has resulted in weird user experience (consider on-hover appearance of actions menu for the lower chat message):