Bug 460451

Summary: Apply button is inactive after selecting custom icon again
Product: [Plasma] plasmashell Reporter: ratijas <me>
Component: Application Launcher (Kickoff) widgetAssignee: Plasma Bugs List <plasma-bugs-null>
Status: CONFIRMED ---    
Severity: minor CC: bugseforuns, mikel5764, nate, noahadvs, notmart
Priority: NOR    
Version First Reported In: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description ratijas 2022-10-14 20:59:37 UTC 
SUMMARY

The «Apply» button on the General page of Application Launcher Settings may become inappropriately disabled.

STEPS TO REPRODUCE
1. Open widget settings.
2. Select some custom icon. Apply.
3. Reset to default icon or Remove icon. Apply.
2. Select the same custom icon.

OBSERVED RESULT
Apply button is disabled. Can be re-enabled by selecting some other custom icon, or re-opening the settings.

EXPECTED RESULT
Apply button should be enabled always when changing from "no icon" or "default icon" state to a "custom icon".

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE: git-master
Qt Version: 5.15.6
Kernel Version: 5.19.13-arch1-1 (64-bit)
Graphics Platform: X11
Processors: 8 × Intel® Core™ i7-6700HQ CPU @ 2.60GHz
Memory: 31.3 GiB of RAM
Graphics Processor: NVIDIA GeForce GTX 970M/PCIe/SSE2
Manufacturer: ASUSTeK COMPUTER INC.
Product Name: G752VT
System Version: 1.0
Comment 1 ratijas 2022-10-14 22:22:20 UTC 
This requires a patch to KDeclarative framework, adding at least accepted() and rejected signals to KQuickAddons.IconDialog — KIconDialog's QML wrapper.
Comment 2 Nate Graham 2023-12-08 22:59:13 UTC 
Can still reproduce in Plasma 6.
Comment 3 Marco Martin 2023-12-11 14:58:55 UTC 
can't reproduce here on today's master
Comment 4 Marco Martin 2023-12-11 15:01:55 UTC 
(In reply to Marco Martin from comment #3)
> can't reproduce here on today's master
nevermind i can reproduce