Disable mutliple click (parallel or serial) on a room

[bmarty]

Content

Add minimum delay between two click handling on the room list.
Protect:

• opening the setting
• opening a room
• opening the create room screen

Motivation and context

Fixes #4619

Screenshots / GIFs

Tests

• Open the room list
• Tap on several rooms simultaneously
• Observe that only one room is opened by going back to the room list
• Quickly tap on a room
• Observe that only one room is opened by going back to the room list
• Tap on a room and on the user avatar icon
• Observe that the settings screen is opened or the room is opened but not both by going back to the room list
• Tap on a room and on the floating action button to create a room
• Observe that the settings to create a room is opened or the room is opened but not both by going back to the room list

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes have been tested on an Android device or Android emulator with API 24
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request title will be used in the release note, it clearly define what will change for the user
• Pull request includes screenshots or videos if containing UI changes
• You've made a self review of your PR

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/Qp67yw

[codecov]

Codecov Report

Attention: Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.

Project coverage is 80.13%. Comparing base (31e5ae0) to head (019bad7).
Report is 37 commits behind head on develop.

Files with missing lines	Patch %	Lines
...ent/android/features/roomlist/impl/RoomListView.kt	50.00%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #4683      +/-   ##
===========================================
+ Coverage    80.11%   80.13%   +0.01%     
===========================================
  Files         2126     2126              
  Lines        56270    56269       -1     
  Branches      7014     7017       +3     
===========================================
+ Hits         45081    45089       +8     
+ Misses        8778     8768      -10     
- Partials      2411     2412       +1     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ganfra]

Maybe this should be handled directly in the compose layer? Would be simpler to handle this with a custom Modifier so it can easily be reused (and tested?)

[ganfra]

I'd call this canHandle and the other one could be getResult?

[bmarty]

I have rewritten this class.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[ganfra]

Would be handy to create a reusable Modifier so we can use this everywhere more easily, but we can do later I guess.

[bmarty]

Would be handy to create a reusable Modifier so we can use this everywhere more easily, but we can do later I guess.

I have thought about it, but I am not sure how to implement this, as the FirstThrottle is shared between several click handlers.