Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: github-merge-queue <[email protected]>

…uld be hidden (matrix-org#12434)

* use the same logic in prevews as the timeline to hide events that should not be shown

* skip any events we should hide, don't need to special case Replace events as we should be ignoring them(getContent() will use replacingEvent)

* lint

* `DecryptionFailureTracker`: stronger typing

Use `DecryptionFailureCode` rather than string

* `DecryptionFailureTracker`: remove use of `DecryptionError`

The second argument to `MatrixEventEvent.Decrypted` callbacks is deprecatedf,
and we can get the info we need direct from the event. This means that we no
longer need to reference the internal `DecryptionError` class in the js-sdk.

* `DecryptionFailureTracker`: use a different Posthog code for historical UTDs

* Update for new UTD error codes

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>

* Collect decryption failure translations together

They are not specific to threads.

* Revert changes to non-english languages

…tip-update

Use new compound tooltip

Co-authored-by: github-merge-queue <[email protected]>

* Move the TAC to above the button

This isn't what the designs say, but there is a (bug/feature)* in
radix where it will turn a mouseup event within the menu into a
click so that it activates the menu option that's under your cursor
when you lift the button, even if you pressed the button down on
a different item. Unfortunately it forgot to check that the mouse
down happened in the menu, so if the menu appears under the cursor,
the cursor, it will immediately select whatever option appeared under
the cursor.

I think the simplest solution here is to just move the menu above the
TAC button. This also makes it consistent† with quick settings.

*delete as applicable
†Not actually consistent because it fades in a different way

* Update snapshots

…#12445)

Signed-off-by: Johannes Marbach <[email protected]>

* Add analytics to activity toggles

Requires matrix-org/matrix-analytics-events#101

* Add test

* Fix comment a bit

* Update to new analytics events package

* show the local echo in previews

* a bit more coverage

* Deprecate AccessibleTooltipButton

Signed-off-by: Michael Telatynski <[email protected]>

* Fix tests

Signed-off-by: Michael Telatynski <[email protected]>

* Fix tests

Signed-off-by: Michael Telatynski <[email protected]>

* Deprecate AccessibleTooltipButton

Signed-off-by: Michael Telatynski <[email protected]>

* Fix tests

Signed-off-by: Michael Telatynski <[email protected]>

* Iterate

Signed-off-by: Michael Telatynski <[email protected]>

* Fix `UserInfo-test.tsx`

* Update `LoginWithQRFlow-test.tsx` snapshot

* Remove tooltip provider from test

* Fix `AccessibleButton`

* Update snapshots

* Revert to original import

* Use title to populate aria-label

* Rollback AccessibleButton or Tooltip changes. Will come in another PR

* Rollback en.json change

* Update snapshots

* Fix `UserInfo`

* Update snapshots

* Use label instead of title in test

* Use label instead of title in TAC test

* Use label instead of title in read-receipt test

* Remove tooltip for ContextMenu

* Add extra information for caption field

---------

Signed-off-by: Michael Telatynski <[email protected]>
Co-authored-by: Michael Telatynski <[email protected]>

…trix-org#12391)

* Use different messages for UTDs sent before login

* Playwright test for historical events

* Add some tests

* Don't show "verify" message if backup is working

* Apply suggestions from code review

* Update snapshots

* Add extra attribute to `AccessibleButton`

* Add placement and fix additional span for `AccessibleButton`

* Use only `RovingAccessibleButton` in `MenuItem`

* Replace `tooltip` by `caption`

* Update snapshots

* Simplify Playwright CI

and apply the least privilege principle

Signed-off-by: Michael Telatynski <[email protected]>

* Fix playwright.needs

Signed-off-by: Michael Telatynski <[email protected]>

* Remove spurious report.needs

Signed-off-by: Michael Telatynski <[email protected]>

* Add a completion stage to simplify branch protections

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>

* Take the TAC out of labs!

Requires matrix-org#12438
and ideally matrix-org#12418

* i18n

* Add test method

That's needed now we we don't include threads in the notif count in the tests

* One less labs setting

* Update snapshot

* Disable release announcement

* Unused import

* Fix some screenshots

* Fix all the unread test cases now room unreads don't include threads

* Fix more tests

* Even more test fixes

* Still more test fixes

* Oh goodness, it's more test fixes

* Fix selectors now there are 2 buttons called Threads

* Disable some tests that aren't passing

for reasons that don't appear releated to any of the TAC work, as
per the comment.

* Remove debugging

* Oops, removed too much

…atrix-org#12453)

* Use different messages for UTDs sent before login

* Playwright test for historical events

* Add some tests

* initial work on pre-join UTDs

* add playwright tests and remove old pre-join UTD logic

* run i18n script

* fix type error

* use different error code in PostHog for pre-join UTD

* don't remove old pre-join UTD logic yet

---------

Co-authored-by: Richard van der Hoff <[email protected]>

* Update reusable elements

* Update tests

* Make right as default tooltip placement

* Add tests

* Update dialogs

* Use default right placement

…atrix-org#12458)

* Use `AccessibleButton` in `RovingAccessibleTooltipButton`

* Update snapshots

* Update @vector-im/compound-web

* Update playwright

* Remove placement

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update all non-major dependencies

* Update posthog user state access

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>

* Update dependency typescript to v5.4.5

* Fix types

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Michael Telatynski <[email protected]>

* Move LoginWithQRSection to the top of the settings tab

Signed-off-by: Michael Telatynski <[email protected]>

* Refactor LoginWithQRSection to a Functional Component

Signed-off-by: Michael Telatynski <[email protected]>

* Extract LoginWithQR types

Signed-off-by: Michael Telatynski <[email protected]>

* Update LoginWithQRFlow styling & copy

Signed-off-by: Michael Telatynski <[email protected]>

* Re-add missing buttons and update snapshots

Signed-off-by: Michael Telatynski <[email protected]>

* Use compound spacings

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>

* Initial commit

Signed-off-by: Ed Geraghty <[email protected]>

* Remove commented code

Signed-off-by: Ed Geraghty <[email protected]>

* Change function to reflect it's proxy not native support

Signed-off-by: Ed Geraghty <[email protected]>

* Re-add check for servers with native support

Signed-off-by: Ed Geraghty <[email protected]>

* Add native support check back in

Signed-off-by: Ed Geraghty <[email protected]>

* Re-add endpoint health check function

Signed-off-by: Ed Geraghty <[email protected]>

* Use inbuilt `getWellKnown` function

Signed-off-by: Ed Geraghty <[email protected]>

* Change the error message to the correct function

Signed-off-by: Ed Geraghty <[email protected]>

* Stop storing the proxyurl in the settings for now

Signed-off-by: Ed Geraghty <[email protected]>

* Make the logger messages more useful

Signed-off-by: Ed Geraghty <[email protected]>

* Start moving the checking logic directly into the controller

Signed-off-by: Ed Geraghty <[email protected]>

* Add missing import

Signed-off-by: Ed Geraghty <[email protected]>

* Get the client rather than passing it in to the functions

Signed-off-by: Ed Geraghty <[email protected]>

* remove invalid `function` keyword

Signed-off-by: Ed Geraghty <[email protected]>

* Fix imports

Signed-off-by: Ed Geraghty <[email protected]>

* Our new functions are private

We shouldn't(?) have to use these check in future elsewhere

Signed-off-by: Ed Geraghty <[email protected]>

* Change our proxy check function to return a boolean

Signed-off-by: Ed Geraghty <[email protected]>

* Make `nativeSlidingSyncSupport` also return boolean, add in health check

Signed-off-by: Ed Geraghty <[email protected]>

* Disable the sliding sync option if the server doesn't support

Signed-off-by: Ed Geraghty <[email protected]>

* Only enable the setting if it passes (again)

Signed-off-by: Ed Geraghty <[email protected]>

* Update our comments to better match what's going on

Signed-off-by: Ed Geraghty <[email protected]>

* Remove unused dialog

Signed-off-by: Ed Geraghty <[email protected]>

* Add a well-known check on start-up, if sliding sync has been enabled

Signed-off-by: Ed Geraghty <[email protected]>

* Check against the correct endpoint...

Signed-off-by: Ed Geraghty <[email protected]>

* Extract baseUrl as we'll reuse it

Signed-off-by: Ed Geraghty <[email protected]>

* Make the logs differentiate between the types of proxy

Signed-off-by: Ed Geraghty <[email protected]>

* Grab the client well-known directly for use

Can't use the client object at this point, it hasn't read in the well-known

Signed-off-by: Ed Geraghty <[email protected]>

* Add myself to the copyright assignation

I wrote the majority of this file...

Signed-off-by: Ed Geraghty <[email protected]>

* Only return `true` if it's actually there

Signed-off-by: Ed Geraghty <[email protected]>

* Correct the `proxySlidingSyncSupport` function comment to match the code

Signed-off-by: Ed Geraghty <[email protected]>

* Correct the `nativeSlidingSyncSupport`function comment to match the code

Signed-off-by: Ed Geraghty <[email protected]>

* Another comment/functionality paring

Signed-off-by: Ed Geraghty <[email protected]>

* Remove duplicated types from the doc

Signed-off-by: Ed Geraghty <[email protected]>

* Move await to the previous line

Removes brackets, and corrects `wellKnown` from being a `Promise`
Signed-off-by: Ed Geraghty <[email protected]>

* use `waitForClientWellKnown` to avoid a race condition with the request

Signed-off-by: Ed Geraghty <[email protected]>

* Move getting the client out of the `if`, use `waitForClientWellKnown`

Signed-off-by: Ed Geraghty <[email protected]>

* Remove `beforeChange` override

Signed-off-by: Ed Geraghty <[email protected]>

* Move proxy setup logic into `SlidingSyncManager`

Signed-off-by: Ed Geraghty <[email protected]>

* Swap `configure` to private, we call it from `setup` which handles proxy

Signed-off-by: Ed Geraghty <[email protected]>

* Promises are always `true`

TIL.
Signed-off-by: Ed Geraghty <[email protected]>

* use `timeoutSignal`

Signed-off-by: Ed Geraghty <[email protected]>

* Change message when there's no server support

Signed-off-by: Ed Geraghty <[email protected]>

* Refactor `slidingSyncHealthCheck`

Signed-off-by: Ed Geraghty <[email protected]>

* Refactor `nativeSlidingSyncSupport` with try/catch

Signed-off-by: Ed Geraghty <[email protected]>

* Change comment to hotlink

Signed-off-by: Ed Geraghty <[email protected]>

* Try and make the toggle disabled when there's no endpoint

Signed-off-by: Ed Geraghty <[email protected]>

* Move the if statement outside the refactored fn to avoid an await

Signed-off-by: Ed Geraghty <[email protected]>

* Revert "Swap `configure` to private, we call it from `setup` which handles proxy"

This reverts commit c80a00b.

* Remove unused import

Signed-off-by: Ed Geraghty <[email protected]>

* Further refactor `slidingSyncHealthCheck`

`proxySlidingSyncSupport` already checks the client well-known is there
Signed-off-by: Ed Geraghty <[email protected]>

* Make `proxySlidingSyncSupport` log on success

Signed-off-by: Ed Geraghty <[email protected]>

* Clarify log message for proxy being up

Signed-off-by: Ed Geraghty <[email protected]>

* Move the logic into SlidingSyncManager

All so we can set a static variable because the disabled check isn't asynchronous :)

Signed-off-by: Ed Geraghty <[email protected]>

* Obviously this isn't a return so don't overwrite with false!

Signed-off-by: Ed Geraghty <[email protected]>

* Remove outdated comment

Signed-off-by: Ed Geraghty <[email protected]>

* No need to pass in the client

Signed-off-by: Ed Geraghty <[email protected]>

* Activating SS should probably be info level logs

Signed-off-by: Ed Geraghty <[email protected]>

* If we've not enabled sliding sync, push the logs down a bit

Signed-off-by: Ed Geraghty <[email protected]>

* Update i18n error message

Signed-off-by: Ed Geraghty <[email protected]>

* Remove unused i18n strings

Signed-off-by: Ed Geraghty <[email protected]>

* Correct log message

Signed-off-by: Ed Geraghty <[email protected]>

* Prettier

Signed-off-by: Ed Geraghty <[email protected]>

* Remove many of the log messages

Signed-off-by: Ed Geraghty <[email protected]>

* Short out of `checkSupport` if it's `true`

Signed-off-by: Ed Geraghty <[email protected]>

* Add the endpoint back into the log when we're enabling it

Signed-off-by: Ed Geraghty <[email protected]>

* Note in the comment that `feature_sliding_sync_proxy_url` is legacy

Signed-off-by: Ed Geraghty <[email protected]>

* Expand the well-known liveness check log

Signed-off-by: Ed Geraghty <[email protected]>

* No need to stall the client waiting for sliding sync support

* `AutoDiscovery.findClientConfig` throws if the baseUrl is blank

* Fix `getProxyFromWellKnown` (?)

* Add missing semicolon

Sorry, linter!
Signed-off-by: Ed Geraghty <[email protected]>

* Pass our `MatrixClient` through instead of trying to grab it

Signed-off-by: Ed Geraghty <[email protected]>

* Add missing return in function comment

Signed-off-by: Ed Geraghty <[email protected]>

* Actually pass through our Client, not the Peg object

Signed-off-by: Ed Geraghty <[email protected]>

* Remove SonarCube smell complaint

Signed-off-by: Ed Geraghty <[email protected]>

* Neew to make our other two methods public to test

Signed-off-by: Ed Geraghty <[email protected]>

* First passing test

Hurrah!
Signed-off-by: Ed Geraghty <[email protected]>

* Two more tests, this time on `checkSupport`

Signed-off-by: Ed Geraghty <[email protected]>

* Reset our `serverSupportsSlidingSync` between tests

Signed-off-by: Ed Geraghty <[email protected]>

* Check the static member is being set

Signed-off-by: Ed Geraghty <[email protected]>

* Move the static assignation down to the relevant tests

Signed-off-by: Ed Geraghty <[email protected]>

* Pull getProxyFromWellKnown mocking up

Signed-off-by: Ed Geraghty <[email protected]>

* Check we /haven't/ shorted out

Signed-off-by: Ed Geraghty <[email protected]>

* Move our spy up so we can reuse it

Signed-off-by: Ed Geraghty <[email protected]>

* Check spidering  is being called

Signed-off-by: Ed Geraghty <[email protected]>

* Test the proxy is declared

Signed-off-by: Ed Geraghty <[email protected]>

* Test entered manually

Signed-off-by: Ed Geraghty <[email protected]>

* Sorry, linter

* I guess these strings are wrong?

* Replace any with string

Co-authored-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Ed Geraghty <[email protected]>
Co-authored-by: Michael Telatynski <[email protected]>

* Convert UserSettings to functional component

* Put comment back

* Fix test

* Unused imports

Co-authored-by: github-merge-queue <[email protected]>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Signed-off-by: Michael Telatynski <[email protected]>

…g#12475)

* Close RA when TAC button is clicked

* Update snapshots

…ed (matrix-org#12482)

* Update `@vector-im/compound-web`

* Update TAC snapshots

* Update space panel snapshots

...and remove the code that causes them to be retried in CI. Most of
these were just lack of waiting for async things to happen, mostly
lazy loading components, hence whythey worked on the retry: because
the code had been loaded by then.

… clicked (matrix-org#12485)

…uncement is displayed (matrix-org#12484)

Co-authored-by: Florian Duros <[email protected]>

…laywright test (when supported) (matrix-org#12414)

* Send user credentials to service worker for MSC3916 authentication

* appease linter

* Add initial test

The test fails, seemingly because the service worker isn't being installed or because the network mock can't reach that far.

* Remove unsafe access token code

* Split out base IDB operations to avoid importing `document` in serviceworkers

* Use safe crypto access for service workers

* Fix tests/unsafe access

* Remove backwards compatibility layer & appease linter

* Add docs

* Fix tests

* Appease the linter

* Iterate tests

* Factor out pickle key handling for service workers

* Enable everything we can about service workers

* Appease the linter

* Add docs

* Rename win32 image to linux in hopes of it just working

* Use actual image

* Apply suggestions from code review

Co-authored-by: Richard van der Hoff <[email protected]>

* Improve documentation

* Document `??` not working

* Try to appease the tests

* Add some notes

---------

Co-authored-by: Richard van der Hoff <[email protected]>

* Use OPTIONS for sliding sync detection poke

This avoids unintended consequences, including high resource usage, which would accompany a "full" sync request. Instead, we just grab headers and enough information for CORS to pass, revealing likely support.

Fixes element-hq/element-web#27426

* Appease the linter

* Reset for each test

* Move to `AccessibleButton`

* Update snapshots

* Update playwright snapshots

* Convert tabbedview to functional component

The 'Tab' is still a class, so now it's a functional component that
has a supporting class, which is maybe a bit... jarring, but I think
is actually perfectly logical.

* put comment back

* Fix bad tab ID behaviour

* Change to sub-components

and use contitional call syntax

* Comments

* Fix element IDs

* Migrate to `AccessibleButtons`

* Update snapshots

* Convert tabbedview to functional component

The 'Tab' is still a class, so now it's a functional component that
has a supporting class, which is maybe a bit... jarring, but I think
is actually perfectly logical.

* put comment back

* Fix bad tab ID behaviour

* Make TabbedView a controlled component

This does mean the logic of keeping what tab is active is now in each
container component, but for a functional component, this is a single
line. It makes TabbedView simpler and the container components always
know exactly what tab is being displayed rather than having to effectively
keep the state separately themselves if they wanted it.

Based on matrix-org#12478

* Fix some types & unused prop

* Remove weird behaviour of using first tab is active isn't valid

* Don't pass initialTabID here now it no longer has the prop

* Fix test

* bleh... id, not icon

* Change to sub-components

and use contitional call syntax

* Comments

* Fix element IDs

* Fix merge

* Test DesktopCapturerSourcePicker

to make sonarcloud the right colour

* Use custom hook for the fllback tab behaviour

…atrix-org#12498)

* Check native sliding sync support against an unstable feature flag

The `OPTIONS` approach from matrix-org#12492 doesn't work because Synapse *always* responds with 204 (success) to `OPTIONS` requests, as described here: element-hq/synapse#17153

We further can't use `HEAD` because it's not part of the allowed CORS methods, meaning the browser will mask the exact status code and error message from us, and the proxy hangs on the request anyways: matrix-org/sliding-sync#429

To avoid these problems, we instead search for an unstable feature flag to be exposed by the server. Presence of this flag denotes native support. See https://github.com/matrix-org/matrix-spec-proposals/pull/3575/files#r1588877046 for details.

Implementations which support sliding sync natively will need to update to support this new unstable feature flag usage.

* Appease the linter

* Appease the tests

Co-authored-by: Florian Duros <[email protected]>

* Update `@vector-im/compound-web`

* Update snapshots

* Update jest snapshotss

* Update snapshots

…12481)

* Convert tabbedview to functional component

The 'Tab' is still a class, so now it's a functional component that
has a supporting class, which is maybe a bit... jarring, but I think
is actually perfectly logical.

* put comment back

* Fix bad tab ID behaviour

* Make TabbedView a controlled component

This does mean the logic of keeping what tab is active is now in each
container component, but for a functional component, this is a single
line. It makes TabbedView simpler and the container components always
know exactly what tab is being displayed rather than having to effectively
keep the state separately themselves if they wanted it.

Based on matrix-org#12478

* Move the active tab in user settings to the dialog title

Separated by a colon, as per the new design.

* Update snapshots

* Update a playwright test

* Fix more tests / snapshots

* Attempt to test all the cases of titleForTabID

* More tests

* Migrate to `AccessibleButton`

* Update snapshots

* Update snapshots

…g#12489)

* Move to `AccessibilityButton`

* Update snapshots

* Add tests

* Update snapshots

…on (matrix-org#12499)

* Store id_token rather than just id_token_claims

Signed-off-by: Michael Telatynski <[email protected]>

* Pass id_token via `id_token_hint` on `Manage Account` interaction

Signed-off-by: Michael Telatynski <[email protected]>

* Fix tests

Signed-off-by: Michael Telatynski <[email protected]>

---------

Signed-off-by: Michael Telatynski <[email protected]>

Co-authored-by: github-merge-queue <[email protected]>

* Change b tag to h3 tag

* Remove margin

* Update snapshots