Summary: | Brightness keys no longer work with Plasma 5.16 | ||
---|---|---|---|
Product: | [Plasma] Powerdevil | Reporter: | Ilya Bizyaev <bizyaev> |
Component: | general | Assignee: | Bhushan Shah <bhush94> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | arojas, benklop, kishore96, matejm98mthw, plasma-bugs, rajeeshknambiar, rikmills, rom1dep, rullger, subdiff |
Priority: | NOR | ||
Version: | 5.15.90 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/powerdevil/90839c802ebc30c1fd46b54bc04a75fe1e7c3a47 | Version Fixed In: |
Description
Ilya Bizyaev
2019-05-19 11:42:49 UTC
Observed the issue with brightness up/down keys not working in Plasma 5.16 beta. Checking further, System Settings → Shortcuts → Global Shortcuts → Power Management shows that there are two entries each for Decrease and Increase Brightness of Screen/Keyboard, of which one has the correct shortcut assigned. Removing the shortcut from currently assigned entry and assigning it to the second one makes shortcut work again. Observing shortcuts not working (among other things) as the result of kded5 crashing on 5.15.90: https://invent.kde.org/snippets/215 Regression from https://cgit.kde.org/powerdevil.git/commit/?id=972eb26d0558d4c97062995eae2eb624d8cb1f38 After that commit, the settings under the [org_kde_powerdevil] section in ~/.config/kglobalshortcutsrc no longer have effect. Reassigning the shortcuts in systemsettings makes them appear under a new [powerdevil] section, and they work again. This should be done automatically with kconf_update Brightness keys are working for me but there is no brightness level pop-up. I have not reassigned the shortcuts. ~/.config/kglobalshortcutsrc contains... [$Version] update_info=powerdevil_move_shortcuts.upd:powerdevil_move_shortcuts [org_kde_powerdevil] Decrease Keyboard Brightness=Keyboard Brightness Down,Keyboard Brightness Down,Decrease Keyboard Brightness Decrease Screen Brightness=Monitor Brightness Down,Monitor Brightness Down,Decrease Screen Brightness [powerdevil] Decrease Keyboard Brightness=,Keyboard Brightness Down,Decrease Keyboard Brightness Decrease Screen Brightness=,Monitor Brightness Down,Decrease Screen Brightness Arch Linux kde_unstable repo (fix for bug 407614 has been applied) plasma 5.15.90 qt 5.13.0beta3 https://phabricator.kde.org/D21303 modifies the regression introducing code. Unfortunately kconf_update script is not useful for migrating shortcuts. Git commit 90839c802ebc30c1fd46b54bc04a75fe1e7c3a47 by Bhushan Shah. Committed on 20/05/2019 at 12:35. Pushed by bshah into branch 'Plasma/5.16'. Set application name to org_kde_powerdevil Summary: Previously application component name without any KAboutData was org_kde_powerdevil (executable name), although, 972eb26d0558d commit changed the component name to powerdevil, which had side-effects that powerdevil would write shortcuts to powerdevil component and it would conflict with old org_kde_powerdevil shortcuts, and ultimately none would work. In theory we can do migration of the shortcuts to powerdevil component using kconf_update but that doesn't work, as kglobalaccel is already started before powerdevil, resulting in kconf_update changes having no effect, more on this in commit 4ae36ddddae which introduces the PowerDevilApp::migratePre512KeyboardShortcuts() function, while in theory PowerDevilApp::migratePre516KeyboardShortcuts can be introduced, we can simply avoid this hassle by re-assigning old component name. Reviewers: davidedmundson, romangg Reviewed By: davidedmundson Subscribers: fvogt, arojas, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D21303 M +2 -1 daemon/powerdevilapp.cpp https://commits.kde.org/powerdevil/90839c802ebc30c1fd46b54bc04a75fe1e7c3a47 Commit above will fix it for other users, unfortunately for beta testers, they will still have broken "powerdevil" entry, which will conflict with the org_kde_powerdevil. Beta testers will need to cleanup the "[powerdevil]" section in ~/.config/kglobalshortcutsrc *** Bug 407710 has been marked as a duplicate of this bug. *** *** Bug 408158 has been marked as a duplicate of this bug. *** Thanks Bhushan! Also for the tip with cleaning up the [powerdevil] section. Now also all the duplicated shortcut entries are gone in the Shortcuts KCM for power management. |