David Baker
David Baker
Closing now for as the test failure looks like it could be legitimate and doesn't look trivial to fix. Hopefully we can revisit this later as part of scroll jump...
This is now ready, code-wise, just needs a new icon for the mark as unread menu item and whatever testing we want to do before merging.
Icon now updated
So it turns out that the rust (mobile) impls will read either prefix but currently only write to the unstable one. We could do the same here, although would that...
Implementation now updated to read both, with comments on the plan and that we're okay accepting that risk.
So I didn't add playwright tests as this was not too bad to get good coverage with regular tests, so I was biasing towards only adding them for things we...
There is now a playwright test.
First split out PR: https://github.com/matrix-org/matrix-react-sdk/pull/12327
Second split out PR: https://github.com/matrix-org/matrix-react-sdk/pull/12344
OK, main bits of this now split out & merged separately. The next biggest chunk would be analytics but hopefully this doesn't add a great deal of complexity here.