"View only" polls in the timeline

[Florian14]

This PR renders the polls created with other clients.

Notes

• The design isn't final yet
• The poll component isn't interactive (votes cannot be submitted)
• Actions on the poll events need to be implemented/refined
• ⚠️ More work is required in the matrix-rust-sdk to be able to see the polls in the timeline. The screenshots are real, but they have been taken while using a custom development branch for the matrix-rust-sdk
• The timestamp in the bubble overlaps the total votes label, this issue should be addressed in a dedicated PR as this is not specific to polls

Screenshots

Disclosed	Undisclosed

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/8mfKDc

[codecov]

Codecov Report

Patch coverage: 53.38% and project coverage change: -0.04% ⚠️

Comparison is base (a911134) 56.91% compared to head (d4b6342) 56.87%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1031      +/-   ##
===========================================
- Coverage    56.91%   56.87%   -0.04%     
===========================================
  Files          997     1012      +15     
  Lines        25343    25624     +281     
  Branches      5077     5137      +60     
===========================================
+ Hits         14424    14574     +150     
- Misses        8696     8796     +100     
- Partials      2223     2254      +31     

Files Changed	Coverage Δ
...ndroid/features/messages/impl/MessagesPresenter.kt	82.69% <0.00%> (-0.54%)	⬇️
...eatures/messages/impl/actionlist/ActionListView.kt	58.77% <0.00%> (-0.26%)	⬇️
...meline/components/event/TimelineItemContentView.kt	72.91% <0.00%> (-6.63%)	⬇️
...tils/messagesummary/MessageSummaryFormatterImpl.kt	46.66% <0.00%> (-3.34%)	⬇️
...ndroid/features/poll/impl/DefaultPollEntryPoint.kt	0.00% <0.00%> (ø)
...tformatter/impl/DefaultRoomLastMessageFormatter.kt	95.16% <0.00%> (-1.57%)	⬇️
...entformatter/impl/DefaultTimelineEventFormatter.kt	0.00% <0.00%> (ø)
...ies/matrix/api/timeline/item/event/EventContent.kt	84.73% <0.00%> (-6.26%)	⬇️
...t/android/libraries/matrix/impl/poll/PollAnswer.kt	0.00% <0.00%> (ø)
...ent/android/libraries/matrix/impl/poll/PollKind.kt	0.00% <0.00%> (ø)
... and 16 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
5 Code Smells

0.0% Coverage
0.0% Duplication

[jmartinesp]

LGTM, but when you say:

The timestamp in the bubble overlaps the total votes label, this issue should be addressed in a dedicated PR as this is not specific to polls

Couldn't we just add an extra bottom padding to make sure they won't ever overlap?

[Florian14]

LGTM, but when you say:

The timestamp in the bubble overlaps the total votes label, this issue should be addressed in a dedicated PR as this is not specific to polls

Couldn't we just add an extra bottom padding to make sure they won't ever overlap?

Just found "how" to handle this case in the designs, this is a little trickier as we should only apply padding if the timestamp overlaps content

[jmartinesp]

Just found "how" to handle this case in the designs, this is a little trickier as we should only apply padding if the timestamp overlaps content

I think that's ok for text messages, but for complex ones as polls, location, etc. I'd expect it to be placed below the event contents. In any case we don't have the final designs, so it's not a blocker.