Bug 457584 - Suggestion: disable horizontal scrollbar for the user list (right side)
Summary: Suggestion: disable horizontal scrollbar for the user list (right side)
Status: RESOLVED FIXED
Alias: None
Product: NeoChat
Classification: Applications
Component: General (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Tobias Fella
URL: https://bugreports.qt.io/browse/QTBUG...
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-07 11:11 UTC by Matej Starc
Modified: 2022-08-08 21:06 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
scrollbar screenshot (405.33 KB, image/png)
2022-08-07 11:11 UTC, Matej Starc
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matej Starc 2022-08-07 11:11:50 UTC
Created attachment 151161 [details]
scrollbar screenshot

SUMMARY
I noticed that the user list on the right side has a horizontal scrollbar enabled.


EXPECTED RESULT
Why would we need a scrollbar for content that is not intended to be scrollable horizontally.
Comment 1 Nate Graham 2022-08-08 16:54:52 UTC
This is a Qt bug: https://bugreports.qt.io/browse/QTBUG-83890

I'll implement the traditional workaround...
Comment 2 Matej Starc 2022-08-08 19:02:37 UTC
(In reply to Nate Graham from comment #1)
> This is a Qt bug: https://bugreports.qt.io/browse/QTBUG-83890
> 
> I'll implement the traditional workaround...

How do they even manage to break that...

Thanks for the hack when it gets implemented.
Comment 3 Bug Janitor Service 2022-08-08 19:59:14 UTC
A possibly relevant merge request was started @ https://invent.kde.org/network/neochat/-/merge_requests/494
Comment 4 Nate Graham 2022-08-08 21:06:01 UTC
Git commit 619369e1487082d4be433e3bbfecf288de9ef7bd by Nate Graham.
Committed on 08/08/2022 at 21:02.
Pushed by ngraham into branch 'master'.

Add hackaround for Qt bug to all non-horizontally-scrollable scrollviews

https://bugreports.qt.io/browse/QTBUG-83890 has been open for years with
a patch that's been stalled for years. There's no indication that it's
going to be fixed anytime soon, and it generates bug reports for us.
Let's add the typical hackaround for all non-horizontally-scrollable
scrollviews.

M  +3    -0    imports/NeoChat/Component/ChatBox/ChatBar.qml
M  +1    -0    imports/NeoChat/Component/ChatBox/CompletionMenu.qml
M  +4    -0    imports/NeoChat/Component/ChatBox/ReplyPane.qml
M  +4    -0    imports/NeoChat/Menu/Timeline/MessageSourceSheet.qml
M  +3    -0    imports/NeoChat/Panel/RoomDrawer.qml
M  +8    -0    imports/NeoChat/Settings/SonnetConfigPage.qml

https://invent.kde.org/network/neochat/commit/619369e1487082d4be433e3bbfecf288de9ef7bd