Bug 462571

Summary: Brush Editor - Sliders' horizontal width extends beyond screen edge
Product: [Applications] krita Reporter: sooz <ouestfleur>
Component: * UnknownAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: halla, ouestfleur, penguinflyer2222, pinknalu
Priority: NOR Keywords: regression
Version First Reported In: nightly build (please specify the git hash!)   
Target Milestone: ---   
Platform: Android   
OS: Android 11.x   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Screenshot of Brush Editor opened on tablet - Android 11

Description sooz 2022-12-03 01:38:15 UTC
Created attachment 154255 [details]
Screenshot of Brush Editor opened on tablet - Android 11

SUMMARY
5.2.0-prealpha git 6051d46  on Android 11
Brush editor is unusable due to sliders extending beyond screen edge. 


STEPS TO REPRODUCE
1. Open brush editor
2. Try to use scratchpad or save brush edits
3. 

OBSERVED RESULT
75% of brush editor is visible, 25% on the right is off screen.

EXPECTED RESULT
Entire brush editor displayed on screen, all functions available.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Reported by Sooz from the K-A forum.
Comment 1 Freya Lupen 2022-12-03 14:52:49 UTC
Confirming that the brush editor is much wider than before on git 6051d462.

Reverting every commit in MR!1653 "Made the widget KisResourceItemChooser support narrow horizontal layout" corrects it, specifically reverting the initial commit, 3cdfbb58 (https://invent.kde.org/graphics/krita/-/merge_requests/1653/diffs?commit_id=3cdfbb5874de10e074c749bc018446c848bd414b).
Comment 2 Halla Rempt 2022-12-07 11:15:50 UTC
I suspect that it's the use of KisPresetSelectorStrip in the brush preset editor that causes the problem. This class actually was already doing what the changes in the relevant MR did: it provided a horizontal or vertical preset chooser strip with two scroll buttons. (It is currently only used in vertical mode, and I don't know why we're still using that class instead of KisPresetChooser directly. (Though I guess that simply embedding KisPresetChooser would not solve the problem yet.)
Comment 3 Halla Rempt 2023-01-04 09:14:52 UTC
Hi Sooz,

Could you check whether the current nightly build from https://binary-factory.kde.org/ fixes the issue? Dmitry thinks it should've been fixed...
Comment 4 sooz 2023-01-04 21:33:17 UTC
Thank you. It's back to normal width.
Comment 5 Bug Janitor Service 2023-01-05 05:27:47 UTC
Thanks for your comment!

Automatically switching the status of this bug to REPORTED so that the KDE team
knows that the bug is ready to get confirmed.

In the future you may also do this yourself when providing needed information.
Comment 6 sooz 2023-01-05 06:21:52 UTC
I'm changing this to resolved as the width is back to normal. This is the 1st time I ever reported a bug so let me know if I've overstepped by changing the status.
Comment 7 Halla Rempt 2023-01-05 10:25:21 UTC
Reopening a bug developers have closed is a no-no, marking a bug as fixed after testing is great!