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

Fix multi-line reactions blocking message content #785

Merged
merged 6 commits into from
Jul 6, 2023

Conversation

jonnyandrew
Copy link
Contributor

@jonnyandrew jonnyandrew commented Jul 5, 2023

Changes

  • Use a column to lay out timeline item components (sender, message bubble and reactions), ensuring overlapping happens at specified offests and doesn't depend on the item size
  • Add preview for timeline item with reactions spanning multiple lines

Context

Screenshots

Before After
before image

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2023

📱 Scan the QR code below to install the build (arm64 only) for this PR.
QR code
If you can't scan the QR code you can install the build via this link:

@jonnyandrew jonnyandrew force-pushed the jonny/multi-line-reaction-bug branch from dbbbbd9 to 5225b45 Compare July 5, 2023 16:05
@codecov
Copy link

codecov bot commented Jul 5, 2023

Codecov Report

Patch coverage: 54.92% and project coverage change: -0.11 ⚠️

Comparison is base (cbf3206) 56.88% compared to head (7cf9c4d) 56.78%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #785      +/-   ##
===========================================
- Coverage    56.88%   56.78%   -0.11%     
===========================================
  Files          913      913              
  Lines        23159    23194      +35     
  Branches      4682     4682              
===========================================
- Hits         13175    13170       -5     
- Misses        7906     7945      +39     
- Partials      2078     2079       +1     
Impacted Files Coverage Δ
...s/impl/timeline/components/TimelineItemEventRow.kt 55.07% <49.20%> (-6.81%) ⬇️
...es/messages/impl/timeline/TimelineStateProvider.kt 98.88% <100.00%> (+0.02%) ⬆️
...l/timeline/components/TimelineItemReactionsView.kt 45.83% <100.00%> (+4.92%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jonnyandrew jonnyandrew force-pushed the jonny/multi-line-reaction-bug branch from 35998bc to d97cb8d Compare July 6, 2023 10:42
@jonnyandrew jonnyandrew force-pushed the jonny/multi-line-reaction-bug branch from 3d065be to a80dc2d Compare July 6, 2023 11:04
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The space between the emoji buttons has increased

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is the new screenshot test

@jonnyandrew jonnyandrew marked this pull request as ready for review July 6, 2023 11:24
@jonnyandrew jonnyandrew requested a review from a team as a code owner July 6, 2023 11:24
@jonnyandrew jonnyandrew requested review from julioromano and removed request for a team July 6, 2023 11:24
@jonnyandrew jonnyandrew enabled auto-merge (squash) July 6, 2023 12:24
@sonarcloud
Copy link

sonarcloud bot commented Jul 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@jonnyandrew jonnyandrew merged commit c87e14a into develop Jul 6, 2023
8 of 10 checks passed
@jonnyandrew jonnyandrew deleted the jonny/multi-line-reaction-bug branch July 6, 2023 12:44
MatMaul pushed a commit that referenced this pull request Jul 6, 2023
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.

Multi-line reactions block message content
2 participants