Skip to content

Conversation

@micieslak
Copy link
Member

@micieslak micieslak commented Sep 25, 2025

What does the PR do

It makes "members" panel working as expected on mobile.

Moreover:

  • fixes goToNextPanel/goToPreviousPanel eliminating some subtle issues related to that functionality
  • minor refactor in ChatHeaderContentView
  • swipe signal exposed from StatusSectionLayout
  • unsed experimental flag 'showOnlineUsers' removed
  • localAccountSensitiveSettings usages removed from ChatView/ChatHeaderContentView

Closes: #18280

Affected areas

ChatView, ChatHeaderContentView, StatusSectionLayout

Architecture compliance

Screencapture of the functionality

Screencast.from.25.09.2025.17.46.27.webm

Risk

Low

@status-im-auto
Copy link
Member

status-im-auto commented Sep 25, 2025

Jenkins Builds

Click to see older builds (37)
Commit #️⃣ Finished (UTC) Duration Platform Result
a1816bc #1 2025-09-25 15:57:43 ~6 min macos/aarch64 📄log
a1816bc #1 2025-09-25 15:57:43 ~6 min macos/aarch64-nwaku 📄log
✔️ a1816bc #1 2025-09-25 15:59:46 ~8 min tests/nim 📄log
✔️ a1816bc #1 2025-09-25 16:01:26 ~9 min ios/aarch64 📦pkg
✔️ a1816bc #1 2025-09-25 16:06:13 ~14 min tests/ui 📄log
✔️ a1816bc #1 2025-09-25 16:08:50 ~17 min linux/x86_64 📦tgz
✔️ a1816bc #1 2025-09-25 16:14:03 ~22 min linux/x86_64-nwaku 📦tgz
✔️ a1816bc #1 2025-09-25 16:17:33 ~25 min windows/x86_64 💿exe
✔️ a1816bc pr18917 2025-09-25 16:22:58 ~14 min tests/e2e 📊rpt
✔️ c319c27a #1 2025-09-25 16:00:29 ~8 min android/arm64 🤖apk 📲
✔️ 89d4b0a1 #2 2025-09-25 17:21:20 ~6 min android/arm64 🤖apk 📲
✔️ 925efd1f #3 2025-09-26 17:22:28 ~7 min android/arm64 🤖apk 📲
✔️ c6e99db #2 2025-09-26 23:09:00 ~6 min ios/aarch64 📦pkg
✔️ c6e99db #2 2025-09-26 23:10:19 ~7 min tests/nim 📄log
✔️ c6e99db #2 2025-09-26 23:14:37 ~11 min macos/aarch64 🍎dmg
✔️ c6e99db #2 2025-09-26 23:15:37 ~12 min tests/ui 📄log
✔️ c6e99db #2 2025-09-26 23:18:45 ~15 min linux/x86_64 📦tgz
✔️ c6e99db #2 2025-09-26 23:18:56 ~15 min macos/aarch64-nwaku 🍎dmg
✔️ c6e99db #2 2025-09-26 23:21:49 ~18 min linux/x86_64-nwaku 📦tgz
✔️ c6e99db #2 2025-09-26 23:26:49 ~23 min windows/x86_64 💿exe
✔️ c6e99db pr18917 2025-09-26 23:34:30 ~15 min tests/e2e 📊rpt
✔️ 8cf8517e #4 2025-09-26 23:09:24 ~6 min android/arm64 🤖apk 📲
✔️ 2db25e3f #5 2025-09-29 17:25:38 ~10 min android/arm64 🤖apk 📲
✔️ 0fdfde4c #8 2025-10-02 13:39:23 ~6 min android/arm64 🤖apk 📲
✔️ 4f9eb7a #3 2025-10-02 15:07:50 ~7 min ios/aarch64 📦pkg
✔️ 4f9eb7a #3 2025-10-02 15:08:15 ~7 min tests/nim 📄log
✔️ 4f9eb7a #9 2025-10-02 15:10:37 ~9 min android/arm64 🤖apk 📲
✔️ 4f9eb7a #3 2025-10-02 15:15:03 ~14 min tests/ui 📄log
✔️ 4f9eb7a #3 2025-10-02 15:24:10 ~23 min linux/x86_64-nwaku 📦tgz
✔️ 4f9eb7a #3 2025-10-02 15:29:15 ~28 min windows/x86_64 💿exe
✔️ c8b5954 #4 2025-10-03 09:59:51 ~5 min tests/nim 📄log
c8b5954 #4 2025-10-03 10:02:14 ~8 min linux/x86_64-nwaku 📄log
✔️ c8b5954 #4 2025-10-03 10:06:09 ~12 min ios/aarch64 📦pkg
✔️ c8b5954 #4 2025-10-03 10:07:11 ~13 min tests/ui 📄log
✔️ c8b5954 #4 2025-10-03 10:10:14 ~16 min macos/aarch64 🍎dmg
✔️ c8b5954 #4 2025-10-03 10:13:54 ~19 min macos/aarch64-nwaku 🍎dmg
✔️ c8b5954 #4 2025-10-03 10:19:26 ~25 min windows/x86_64 💿exe
Commit #️⃣ Finished (UTC) Duration Platform Result
101abad #11 2025-10-03 11:36:21 ~4 min android/arm64 📄log
✔️ 101abad #5 2025-10-03 11:38:09 ~6 min tests/nim 📄log
✔️ 101abad #5 2025-10-03 11:38:49 ~6 min ios/aarch64 📦pkg
101abad #5 2025-10-03 11:41:11 ~9 min linux/x86_64-nwaku 📄log
✔️ 101abad #5 2025-10-03 11:43:13 ~11 min macos/aarch64 🍎dmg
✔️ 101abad #5 2025-10-03 11:45:41 ~13 min tests/ui 📄log
✔️ 101abad #5 2025-10-03 11:47:54 ~15 min macos/aarch64-nwaku 🍎dmg
✔️ 101abad #5 2025-10-03 11:55:31 ~23 min windows/x86_64 💿exe
101abad #6 2025-10-03 14:59:32 ~27 min linux/x86_64-nwaku 📄log
✔️ fea6074 #14 2025-10-06 08:06:44 ~7 min android/arm64 🤖apk 📲
✔️ fea6074 #6 2025-10-06 08:06:47 ~7 min tests/nim 📄log
✔️ fea6074 #6 2025-10-06 08:07:57 ~8 min ios/aarch64 📦pkg
✔️ fea6074 #6 2025-10-06 08:13:34 ~14 min tests/ui 📄log
✔️ fea6074 #7 2025-10-06 08:22:15 ~22 min linux/x86_64-nwaku 📦tgz
✔️ fea6074 #6 2025-10-06 08:33:39 ~34 min windows/x86_64 💿exe
✔️ fea6074 #7 2025-10-06 09:56:58 ~10 min macos/aarch64 🍎dmg
✔️ fea6074 #9 2025-10-06 10:09:36 ~23 min linux/x86_64 📦tgz
✔️ fea6074 pr18917 2025-10-06 10:24:34 ~14 min tests/e2e 📊rpt

Copy link
Member

@caybro caybro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one minor suggestion

Copy link
Contributor

@noeliaSD noeliaSD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested it and it works fine! On the other hand, I added a comment that would be nice to fix if possible.

@micieslak micieslak requested a review from a team as a code owner October 2, 2025 15:00
Copy link
Contributor

@noeliaSD noeliaSD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates! I like this new acount settings store! :)

Manual increment/decrement were leading to setting value out of range
in some cases.
It gives ability to add custom actions when the panels are swiped.
Now the usage of that ctx property is centralized in a single place in AppMain
New store added intended to encapsulate access to localAccountSensitiveSettings
ctx property.
@micieslak micieslak merged commit 9aa5c14 into master Oct 6, 2025
11 of 12 checks passed
@micieslak micieslak deleted the fix/issue-18280 branch October 6, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Mobile] “Members” header button does nothing in mobile portrait

4 participants