Room list is automatically scrolled to the top after opening a room

[Claire1817]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

#481
We don't scroll to the position 0 when a room is opened/created

Motivation and context

Screenshots / GIFs

Enregistrement.de.l.ecran.2022-03-10.a.10.42.26.mov

Tests

• Open a room, no scroll before the opening.
• Create a room, no scroll before the opening

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes has been tested on an Android device or Android emulator with API 21
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/tchapgouv/tchap-android-v2/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request updates TCHAP_CHANGES.md
• Pull request includes a new file under ./changelog.d. See https://github.com/tchapgouv/tchap-android-v2/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR
• If you have modified the screen flow, or added new screens to the application, you have updated the test UiAllScreensSanityTest.allScreensTest()

[github-actions]

Unit Test Results

  15 files  ±0    15 suites  ±0   9s ⏱️ -1s
104 tests ±0  104 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 7b9e20a. ± Comparison against base commit 12dd29c.

♻️ This comment has been updated with latest results.

[github-actions]

Matrix SDK

Integration Tests Results:

• [org.matrix.android.sdk.session]
  passed=
• [org.matrix.android.sdk.account]
  passed=
• [org.matrix.android.sdk.internal]
  passed=
• [org.matrix.android.sdk.ordering]
  passed=
• [org.matrix.android.sdk.PermalinkParserTest]
  passed=

[Florian14]

The result is not ideal. There is no more scroll before opening the room, this is better but the scroll up still occur after coming back in the room list. This is probably normal as we reset the filter but we can probably prevent the reset if the filter is already empty, what do you think?

[Florian14]

SGTM