Skip to content

Conversation

@micieslak
Copy link
Member

What does the PR do

Fixes style selection for StatusSearchListPopup (CTRL+K). The issue was caused by [QTBUG-142248](https://qt-project.atlassian.net/browse/QTBUG-142248

Affected areas

StatusSearchListPopup

Architecture compliance

Screencapture of the functionality

image

How to test

  1. Log in
  2. Ensure Dark Theme
  3. Hit CTRL + K (The delegates background should be dark)

@status-im-auto
Copy link
Member

status-im-auto commented Dec 18, 2025

Jenkins Builds

Click to see older builds (11)
Commit #️⃣ Finished (UTC) Duration Platform Result
959c9c3 #1 2025-12-18 15:23:40 ~9 min macos/aarch64-nwaku 📄log
✔️ 959c9c3 #1 2025-12-18 15:24:19 ~10 min android/arm64 🤖apk 📲
✔️ 959c9c3 #1 2025-12-18 15:24:21 ~10 min tests/nim 📄log
✔️ 959c9c3 #1 2025-12-18 15:27:53 ~13 min tests/ui 📄log
✖️ 959c9c3 #1 2025-12-18 15:29:40 ~15 min ios/aarch64 📱ipa
✔️ 959c9c3 #1 2025-12-18 15:29:46 ~15 min macos/aarch64 🍎dmg
✔️ 959c9c3 #1 2025-12-18 15:34:35 ~20 min linux/x86_64 📦tgz
✔️ 959c9c3 #1 2025-12-18 15:37:47 ~23 min linux/x86_64-nwaku 📦tgz
✔️ 959c9c3 #1 2025-12-18 15:47:09 ~32 min windows/x86_64 💿exe
✔️ 959c9c3 pr19612 2025-12-18 15:52:08 ~17 min tests/e2e 📊rpt
✔️ 959c9c3 PR19612 2025-12-18 16:13:09 ~25 min tests/e2e-windows 📊rpt
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 2d9b3083 #2 2025-12-19 05:30:37 ~12 min android/arm64 🤖apk 📲
✔️ 731a1593 #3 2025-12-20 05:30:27 ~12 min android/arm64 🤖apk 📲

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 ofc :)

Comment on lines 26 to 27
ColumnLayout {
anchors.fill: parent
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
ColumnLayout {
anchors.fill: parent
contentItem: ColumnLayout {

wouldn't this help?

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.

3 participants