Bug 467242

Summary: Search "toolbar" has the wrong size and "overlaps" onto the rooms area when using non-default font size
Product: [Applications] NeoChat Reporter: Marc Deop <kde>
Component: GeneralAssignee: Tobias Fella <fella>
Status: RESOLVED FIXED    
Severity: normal CC: carl, nate, nathan
Priority: NOR Keywords: regression
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: overlap_onto_rooms_section

Description Marc Deop 2023-03-12 14:24:34 UTC
Created attachment 157213 [details]
overlap_onto_rooms_section

The search toolbar is a bit to big and overlaps onto the rooms area. See the screenshot


Linux/KDE Plasma: 
KDE Plasma Version: 5.27.2 
KDE Frameworks Version: 5.104.0
Qt Version: 5.15.8

ADDITIONAL INFORMATION

I build neochat myself and make it available on my copr: https://copr.fedorainfracloud.org/coprs/marcdeop/neochat-git/
Comment 1 Tobias Fella 2023-03-22 19:58:44 UTC
Have you seen this happen again? works perfectly fine here :/
Comment 2 Marc Deop 2023-03-23 00:17:01 UTC
(In reply to Tobias Fella from comment #1)
> Have you seen this happen again? works perfectly fine here :/

Yes, I still experience the problem.

Built Neochat from git.
Comment 3 Nate Graham 2023-04-27 14:16:35 UTC
Can reproduce with today's git master (cdf2c390cab571acfaa0771e9c7e076aeb8c4df4).

The issue appears to be dependent on font size; with the default 10pt Noto Sans font, there is no problem. With 11pt Noto Sans, the problem becomes 100% reproducible for me.
Comment 4 James Graham 2023-06-03 16:17:49 UTC
*** Bug 470541 has been marked as a duplicate of this bug. ***
Comment 5 Carl Schwan 2023-06-18 07:01:05 UTC
Git commit bd5ed0f46c1141a1b351708cdf680e6c122d38a1 by Carl Schwan.
Committed on 18/06/2023 at 07:00.
Pushed by carlschwan into branch 'master'.

Fix toolbar of RoomList when using font size 11px

We don't need to use a ToolBar for the titleDelegate since the
titleDelegate is already included inside a toolbar. This work magically
when using the default font size of 10px Noto Sans but breaks with 11px

M  +29   -32   src/qml/Page/RoomList/ExploreComponent.qml

https://invent.kde.org/network/neochat/-/commit/bd5ed0f46c1141a1b351708cdf680e6c122d38a1