Bug 443557 - Emoji selector: expanding a collapsed sidebar can make the title text overlap with other elements
Summary: Emoji selector: expanding a collapsed sidebar can make the title text overlap...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Emoji Selector (show other bugs)
Version: 5.26.4
Platform: Arch Linux Linux
: NOR minor
Target Milestone: 1.0
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-10 16:11 UTC by Gabriel
Modified: 2024-07-30 21:56 UTC (History)
2 users (show)

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


Attachments
Screenshot showing the overlap between Recent and Clear history (48.06 KB, image/png)
2021-10-10 16:11 UTC, Gabriel
Details
"Search" text touching the search box (97.55 KB, image/png)
2022-12-22 08:44 UTC, Gabriel
Details
bug showcase with 0 spacing (153.10 KB, image/jpeg)
2022-12-22 10:19 UTC, Matej Starc
Details
bug showcase with 10px spacing (153.50 KB, image/jpeg)
2022-12-22 10:20 UTC, Matej Starc
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gabriel 2021-10-10 16:11:16 UTC
Created attachment 142303 [details]
Screenshot showing the overlap between Recent and Clear history

SUMMARY
When expanding the sidebar of the emoji selector, the title text will move accordingly. In the cases where there is something there it will cause an overlap. "Recent" overlaps with "Clear history" (see screenshot) and "Search" gets partially pushed under the search field.

STEPS TO REPRODUCE
1. Open the emoji selector
2. Resize to the minimum width (as it is by default)
3. Click the sidebar button in the bottom left corner

OBSERVED RESULT
"Recent" overlaps with "Clear history", "Search" is overlapped by the search field

EXPECTED RESULT
No text will overlap with other elements

SOFTWARE
KDE Plasma Version: 5.22.5
KDE Frameworks Version: 5.86.0
Qt Version: 5.15.3

ADDITIONAL INFORMATION
Either the sidebar could be made narrower, or the minimum width of the Emoji selector could be increased (however, that might still cause the problem in other languages). Those are just suggestions :^)
Comment 1 Matej Starc 2022-12-16 18:11:47 UTC
Not reproducible on current master. Can you test again because it might be a problem on machines with lower resolutions (I tested this on 1440p and 1080p).
Comment 2 Gabriel 2022-12-22 08:44:35 UTC
Created attachment 154743 [details]
"Search" text touching the search box

> Not reproducible on current master. Can you test again because it might be a
> problem on machines with lower resolutions
Hi, sorry for the delayed reply. I wanted to also test on master, but struggled to do so because of compilation errors. I can however say that the problem persists until fairly recently as it is still present in the neon Testing edition. I'm adding a screenshot for you to compare with. This is on a 1080p screen.

It's admittedly much better since the "Recent" page now has enough margin (at least in English). What's remaining is that the "Search" page is still a bit crammed. Perhaps it could get worse in other languages with longer words? I'm not sure of how the size adapts to increased text length.
Comment 3 Matej Starc 2022-12-22 09:21:55 UTC
It seems I can reproduce it too. Forgot to set my application style to breeze.
Comment 4 Matej Starc 2022-12-22 10:19:22 UTC
Created attachment 154748 [details]
bug showcase with 0 spacing

Setting the layout's spacing to 0px, that the emoji selector sets as page's title delegate to 0 produces this result.
Comment 5 Matej Starc 2022-12-22 10:20:47 UTC
Created attachment 154749 [details]
bug showcase with 10px spacing

Setting the layout's spacing to 10px, that the emoji selector sets as page's title delegate, produces this result.

This is probably the intended look.
Comment 6 Nate Graham 2024-07-30 21:56:41 UTC
With the default Breeze theme, the issue no longer seems reproducible in current git master, which will become Plasma 6.2 in a few months. Might also have been fixed in Kirigami code.