Update reply view border corner radii and color

[stefanceriu]

And also update relating preview test snapshots

[americanrefugee]

Why are there double sentences for sharing a location?

[stefanceriu]

Why are there double sentences for sharing a location?

That's expected, the tests also run in Double-Length Pseudolanguage to check how layouts work with longer words.

[americanrefugee]

Is it intentional to use the same border colour as the outgoing message bubble background? This basically means that messages have a border around the reply view but outgoing messages don't?

@pixlwave Yes, that's fine to me. There wasn't a lot of contrast between the reply contents and a received message, but there's certainly enough contrast between the reply contents and a sent message.

This way the border overall isn't too stark. I personally like this update. Do you agree?

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.38%. Comparing base (48794a2) to head (c438b7c).
⚠️ Report is 1 commits behind head on develop.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...eens/Timeline/View/Replies/TimelineReplyView.swift	50.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #5341      +/-   ##
===========================================
- Coverage    74.38%   74.38%   -0.01%     
===========================================
  Files          800      800              
  Lines        53519    53519              
===========================================
- Hits         39812    39811       -1     
- Misses       13707    13708       +1     

Flag	Coverage Δ
unittests	64.52% <50.00%> (-0.01%)	⬇️

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

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

[pixlwave]

Yes, that's fine to me.

In which case, LGTM 👍