Bug 462571 - Brush Editor - Sliders' horizontal width extends beyond screen edge
Summary: Brush Editor - Sliders' horizontal width extends beyond screen edge
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: * Unknown (show other bugs)
Version: nightly build (please specify the git hash!)
Platform: Android Android 11.x
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2022-12-03 01:38 UTC by sooz
Modified: 2023-01-05 10:25 UTC (History)
4 users (show)

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


Attachments
Screenshot of Brush Editor opened on tablet - Android 11 (65.83 KB, image/jpeg)
2022-12-03 01:38 UTC, sooz
Details

Note You need to log in before you can comment on or make changes to this bug.
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!