Bug 420416 - Volume Applet Settings allows invalid characters and textbox expands
Summary: Volume Applet Settings allows invalid characters and textbox expands
Status: RESOLVED FIXED
Alias: None
Product: plasma-pa
Classification: Unmaintained
Component: applet (other bugs)
Version First Reported In: 5.22.0
Platform: Arch Linux Linux
: NOR minor
Target Milestone: ---
Assignee: David Rosca
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-22 10:46 UTC by Nico
Modified: 2024-10-03 12:03 UTC (History)
4 users (show)

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

Attachments
Screen recording showing the behavior (3.67 MB, video/mp4)
2020-04-22 10:46 UTC, Nico 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nico 2020-04-22 10:46:15 UTC 
Created attachment 127772 [details]
Screen recording showing the behavior

SUMMARY


STEPS TO REPRODUCE
1. Right click the plasmoid and click on Configure Audio Volume
2. Enter any letter in the textboxes for "Maximum volume" or "Volume step"

OBSERVED RESULT
You can enter letter and unlimited characters which results in the textbox expanding

EXPECTED RESULT
Only numbers are allowed and it is limited to 3 digits as the maximum is 150. It should have a fixed width

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Arch Linux
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.2

ADDITIONAL INFORMATION
Comment 1 Patrick Silva 2021-06-13 00:27:50 UTC 
Can reproduce on Plasma 5.22, Arch Linux.
Comment 2 Nate Graham 2021-06-14 19:25:31 UTC 
lol

If I constrain the maximum width of this spinbox in the applet, then the text simply overflows from box itself which is arguably worse. Blocking on Bug 438650.

Alternatively, we may be able to work around these issues by limiting the maximum number of characters that can be entered in the spinbox.
Comment 3 Bug Janitor Service 2024-09-26 12:43:54 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/285
Comment 4 Bug Janitor Service 2024-10-02 12:21:12 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/288