Fix: Calculation of the frame for a component of a bubble

[nimau]

This PR fixes some issues with the calculation of the frame for a bubble component.

• the height of the text is not calculated with the correct width, resulting in an incorrect calculated Y position when using the bubble timeline style.
• the component's position may be calculated with an extra positioning space (to make room for the timestamp above the component), even for cells of kind TextMessageOutgoingWithoutSenderInfoBubbleCell

These issues result in the sent confirmation icon being misplaced and make it difficult to run to detect the sent confirmation icon in UITests.

Examples:

First message in a room:

Current implementation:

after a restart of the app, the sent confirmation icon has moved:

With this PR:

after a restart, the icon stays at the same position:

Second message in a room:

Current implementation: the icon position is not a the same place

With this PR: the icon remains aligned with the bottom of the component.

Long message:

Current implementation:

With this PR:

With pagination:

Current implementation:

With this PR:

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +0.14 🎉

Comparison is base (f23575e) 12.21% compared to head (e3ae9ca) 12.36%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #7512      +/-   ##
===========================================
+ Coverage    12.21%   12.36%   +0.14%     
===========================================
  Files         1644     1644              
  Lines       162969   162986      +17     
  Branches     66863    66880      +17     
===========================================
+ Hits         19905    20148     +243     
+ Misses      142425   142179     -246     
- Partials       639      659      +20     

Flag	Coverage Δ
uitests	55.26% <ø> (ø)
unittests	6.21% <0.00%> (+0.14%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
Riot/Categories/MXKRoomBubbleTableViewCell+Riot.m	0.00% <0.00%> (ø)
...ules/MatrixKit/Models/Room/MXKRoomBubbleCellData.h	0.00% <ø> (ø)
...ules/MatrixKit/Models/Room/MXKRoomBubbleCellData.m	15.88% <0.00%> (-0.15%)	⬇️
...xtMessageOutgoingWithoutSenderInfoBubbleCell.swift	0.00% <0.00%> (ø)

... and 14 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

[stefanceriu]

Very nice 👏