| Summary: | Starting KDE causes with power widget causes screen brightness to go to 0 | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Charles H <chathaway> |
| Component: | Power and Battery widget | Assignee: | Kai Uwe Broulik <kde> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | normal | CC: | kde, peter, plasma-bugs-null |
| Priority: | NOR | Keywords: | triaged |
| Version First Reported In: | master | ||
| Target Milestone: | 1.0 | ||
| Platform: | Kubuntu | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | Set a defalt value for slider; not tested | ||
|
Description
Charles H
2015-11-30 22:55:49 UTC
Created attachment 95829 [details] Set a defalt value for slider; not tested I'm having trouble compiling the widget for testing; please test this, but I believe this solution will work based on the documentation here (http://api.kde.org/4.x-api/kde-runtime-apidocs/plasma/declarativeimports/plasmacomponents/html/classSlider.html) That patch doesn't make a lot of sense.
but the user of BrightnessItem (PopupDialog.qml) *also* sets a value:
BrightnessItem {
id: brightnessSlider
value: batterymonitor.screenBrightness
...
}
so you'd set the value of 1, then this would override it before you've started doing anything.
Currently we set it to undefined which isn't very good code tbh, but it won't do anything as we only connect the valueChanged signal on the slider after everything is loaded. We then call updateBrightness in Logic.js that updates the slider's initial value.
That initial value comes from powerdevil, I think you want to be looking there and this slider is a red herring.
I think you're looking very much in the wrong place; the slider won't do
Did you try it without the widget, what happens? if you (from a terminal) kquitapp5 plasmashell plasmashell does it reset? I'll try that as soon as I can David, thanks (left the laptop with the issue at home, I'll be back and have time Thursday night). This was not fun debugging, jumping from kdeinit5/powerdevil via dbus to plasmashell to a widget. Question: have you expanded the battery monitor widget? That seems to trigger the bad condition in plasma-workspace 5.6.4. Can you try this patch: https://git.reviewboard.kde.org/r/128098/ (and check also if the analysis is sane) Dear Bug Submitter, This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days, the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging If you have already provided the requested information, please set the bug status as REPORTED so that the KDE team knows that the bug is ready to be confirmed. Thank you for helping us make KDE software even better for everyone! Dear Bug Submitter, This bug has been in NEEDSINFO status with no change for at least 30 days. The bug is now closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging Thank you for helping us make KDE software even better for everyone! |