- Use the compact representation for indicating status
- In the expanded representation just place a button
- Use declarative bindings as much as possible
BUG: 368192
FIXED-IN: 5.8.0
Details
Details
- Reviewers
mart - Group Reviewers
Plasma: Design Plasma - Commits
- R119:708eed1a6603: [Touchpad plasmoid] Major cleanup
I just wanted to fix the icon but then decided to give it a major cleanup.
Tested:
- Enabling / disabling
- Disabling with no external mouse prompts for confirmation
- With no touchpad the icon turns gray (and the applet should just hide completely but doesn't work in tray, see below)
I stumbled upon two bugs in system tray:
- When I have an applet with preferredRepresentation being expandedRepresentation systray tries to be smart and then expands on plasmashell startup and shows an empty spot for fthe applet
- HiddenStatus applets are still shown in the popup list like PassiveStatus ones
Expanded representation in system tray, this is only accessible when in the popup and clicking the text of the applet instead of its icon. In all other cases clicking the icon just turns it on/off.
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
| kcms/touchpad/src/applet/qml/contents/ui/touchpad.qml | ||
|---|---|---|
| 37 | i don't like it that much, but will need in the future to actually fake mouse events when clicking on the label. | |