Bug 479304 - No way to scroll in the "Screen Sharing" portal window
Summary: No way to scroll in the "Screen Sharing" portal window
Status: RESOLVED FIXED
Alias: None
Product: xdg-desktop-portal-kde
Classification: Plasma
Component: general (show other bugs)
Version: 5.91.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: qt6
: 479568 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-01-02 15:53 UTC by Andrej Halveland
Modified: 2024-01-13 23:46 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0
Sentry Crash Report:


Attachments
Video demonstration. (1.15 MB, video/mp4)
2024-01-02 15:53 UTC, Andrej Halveland
Details
Scrollable for me (252.64 KB, image/jpeg)
2024-01-10 21:33 UTC, Nate Graham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrej Halveland 2024-01-02 15:53:11 UTC
Created attachment 164628 [details]
Video demonstration.

STEPS TO REPRODUCE
1. Open many apps
2. Try to screen share a window in discord for example

OBSERVED RESULT
In the "Screen Sharing" portal the right side items are cut-off and also there is no way to scroll down to other windows, the solution is to resize the "Screen Sharing" portal window

EXPECTED RESULT
Items should not be cut-off / clipped and we should be able to scroll down the list of windows.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.91.0
KDE Frameworks Version: 5.247.0
Qt Version: 6.7.0 beta1
Kernel Version: 6.7.0-rc6
Graphics Platform: Wayland
Graphics Drivers: mesa 23.3.2
Processors: AMD Ryzen 9 4900HS
Memory:  24GB of RAM
Graphics Processor: Integrated: Vega 8
Manufacturer: ASUS (Zephyrus G15 GA502IV)
Comment 1 Andrej Halveland 2024-01-09 11:13:43 UTC
*** Bug 479568 has been marked as a duplicate of this bug. ***
Comment 2 Nate Graham 2024-01-10 21:33:58 UTC
Created attachment 164800 [details]
Scrollable for me

Thanks for the screen recording. However I cannot reproduce the issue; for me it's scrollable. There are various other obvious UI glitches, but it is scrollable. See attached screenshot.
Comment 3 Nate Graham 2024-01-10 21:34:26 UTC
Can you try again with Plasma 6 RC1 which was just released today?
Comment 4 Andrej Halveland 2024-01-12 15:33:21 UTC
Plasma 6.7 RC1 built against Qt6.7 beta1 made no difference.

Plasma 6.7 RC1 built against Qt6.6 also made no difference.
Comment 5 Nate Graham 2024-01-12 15:37:49 UTC
There is no Plasma 6.7 FWIW, there's only 6.0.

But if using Qt 6.6 didn't help, then I'm not sure what the problem might be.
Comment 6 Andrej Halveland 2024-01-12 15:41:22 UTC
haha, I accidentally mixed up Plasma and Qt versions together... sorry!

Yes, I meant Plasma 6.0 RC1, not Plasma 6.7 RC1.
Comment 7 fanzhuyifan 2024-01-13 00:32:29 UTC
Can reproduce on arch source build with Qt 6.7.
Comment 8 Bug Janitor Service 2024-01-13 02:38:32 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/256
Comment 9 fanzhuyifan 2024-01-13 23:44:35 UTC
Git commit bab2ef4f8a32cbc840ce98fe1b457a1de86f3059 by Yifan Zhu.
Committed on 14/01/2024 at 00:02.
Pushed by fanzhuyifan into branch 'master'.

Fix ScreenChooserDialog Layout

contentWidth and contentHeight of ScrollView must be set when more than
one item is used.
This was not needed before porting from Kirigami.CardsGridView to
Kirigami.CardsLayout in ebfeb51fa45186551aeb4a515fa59ef4202bddd0
since CardsGridView was just one item.

Set Layout.preferredHeight of PipeWireDelegate so preview image has
enough vertical space.
FIXED-IN: 6.0

M  +2    -0    src/PipeWireDelegate.qml
M  +14   -0    src/ScreenChooserDialog.qml

https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/bab2ef4f8a32cbc840ce98fe1b457a1de86f3059
Comment 10 fanzhuyifan 2024-01-13 23:46:24 UTC
Git commit f8ce748cb404a4e877a7721bd6e2b4573f657b53 by Yifan Zhu.
Committed on 14/01/2024 at 00:45.
Pushed by fanzhuyifan into branch 'Plasma/6.0'.

Fix ScreenChooserDialog Layout

contentWidth and contentHeight of ScrollView must be set when more than
one item is used.
This was not needed before porting from Kirigami.CardsGridView to
Kirigami.CardsLayout in ebfeb51fa45186551aeb4a515fa59ef4202bddd0
since CardsGridView was just one item.

Set Layout.preferredHeight of PipeWireDelegate so preview image has
enough vertical space.
FIXED-IN: 6.0


(cherry picked from commit bab2ef4f8a32cbc840ce98fe1b457a1de86f3059)

M  +2    -0    src/PipeWireDelegate.qml
M  +14   -0    src/ScreenChooserDialog.qml

https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/f8ce748cb404a4e877a7721bd6e2b4573f657b53