Bug 424573

Summary: Sensors search stops working after I reopen the widget settings
Product: [Plasma] plasmashell Reporter: Patrick Silva <bugseforuns>
Component: System Monitor widgetsAssignee: Marco Martin <notmart>
Status: RESOLVED FIXED    
Severity: normal CC: groot, kde, nate, plasma-bugs-null
Priority: NOR    
Version First Reported In: 5.19.3   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.20
Sentry Crash Report:

Description Patrick Silva 2020-07-23 16:11:20 UTC 
SUMMARY
Reproducible on both Arch Linux and neon unstable.

STEPS TO REPRODUCE
1. add System Monitor Sensor widget to desktop
2. open the settings window of System Monitor Sensor widget and click on "Sensors details" section in the left side
3. search for some sensor, let's say "core" (search works as expected)
4. close the widget settings by clicking on "Ok" button
5. reopen the widget settings and try to search for some sensor in "Sensors details" section again

OBSERVED RESULT
it's impossible to type anything in the search field

EXPECTED RESULT
search feature should always work

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.19.3
KDE Frameworks Version: 5.72.0
Qt Version: 5.15.0

ADDITIONAL INFORMATION
sensor search works again if plasma is restared with "plasmashell --replace"
Comment 1 Nate Graham 2020-07-23 18:22:12 UTC 
Can reproduce. Weird.
Comment 2 David Redondo 2020-07-24 11:01:00 UTC 
There seems something very weird going on with the focus.
You can't tab around anymore in the broken state but shift+tab switches between ok and cancel button
Comment 3 groot 2020-07-24 21:21:56 UTC 
I can reproduce this on FreeBSD (although for me, the "search" doesn't work at all -- typing anything in that field just blanks the list of sensors, as if none of the sensors match). FreeBSD packages, I should say, which is KF5 5.71 and Plasma 5.19.1.

This is very reminiscent of .. something I recently hit, but I can't remember where or in which bugtracker. I vaguely remember the QML falling over in ksysguard  because of a missing runtime dependency.

As for tabbing around in the broken state as mentioned by David Redondo, I can click on *Sensors Details* and then tab to the sensors placed in the *Chart Sensors* box, but then it stops. And I can shift-tab back to sensors details, and then it starts cycling between OK and Cancel.
Comment 4 David Redondo 2020-07-30 09:58:32 UTC 
In plasma 5.20 this has a new ui without this text field.