Read Receipts: Allow Timelines to be configured to hide read receipts on state events.

[pixlwave]

This is an mvp attempt at #3065 to fix element-hq/element-x-ios#2338.

The PR adds a new TimelineReadReceiptTracking enum which is used in place of the track_read_receipts boolean to decide whether to enable tracking for all events, just for message-like events or to disable tracking.

When tracking is configured for message-like only, any read receipts on state events are bundled up into the previous event, using the same logic that receipts from filtered events are handled with.

• Public API changes documented in changelogs (optional)

Before	After

[codecov]

Codecov Report

❌ Patch coverage is 99.06542% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.66%. Comparing base (bbc6df7) to head (d0eb5d6).
⚠️ Report is 9 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...ix-sdk-ui/src/timeline/controller/read_receipts.rs	94.11%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5900      +/-   ##
==========================================
- Coverage   88.66%   88.66%   -0.01%     
==========================================
  Files         363      363              
  Lines      104580   104628      +48     
  Branches   104580   104628      +48     
==========================================
+ Hits        92730    92771      +41     
- Misses       7501     7508       +7     
  Partials     4349     4349              

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

[codspeed-hq]

CodSpeed Performance Report

Merging #5900 will not alter performance

_{Comparing doug/read-receipt-filters (d0eb5d6) with main (bbc6df7)}

Summary

✅ 50 untouched

[Hywan]

Clean code, nice feature, great test, good documentation, what can I ask more? A tiny feedback to remove a PartialEq implementation, but otherwise it's really great, kudos 👏 and thanks for the great contribution!

[Hywan]

It's probably better to use matches!(settings.track_read_receipts, TimelineReadReceiptTracking::AllEvents) here, so that you can remove the PartialEq implementation on this enum. Less code.

[pixlwave]

Ah nice, updated here: https://github.com/matrix-org/matrix-rust-sdk/compare/4b903d041a6d4ddcc864a42eb258851c58d383a7..fdc747962df908135935f6cfb8b071628dfc46fa

[pixlwave]

Force pushed to fix a conflict on the changelog.