- Revert "plasmoid: Port to pragma ComponentBehavior: Bound"
This reverts commit 9ee0b23727.
- Revert "plasmoid: Flatten plugin controllers, menu and other non-graphical components"
This reverts commit d2ee2bfdd8.
- Revert "plasmoid: Basic code cleanup"
This reverts commit 90946829bf.
BUG: 485829
## Summary
Currently, the plugin just fails silently if the local device is missing the `krfb` package or if the remote device misses an `vnc://` protocol/scheme handler. You click the button and nothing happens.
One issue is, that the plugin is considered `virtualmonitor.available` in the `DeviceDelegate.qml`, even if the check for `krfb-virtualmonitor` fails and no dbus-path is provided. I investigated the behavior a bit, but ignored it in the end as this MR benefits from being shown for device constellations that _could_ provide this feature.
A warning is shown with brief instructions, how to get the plugin working correctly.
- Check if krfb-virtualmonitor is available locally
- Check default scheme handler for vnc:// on device (Linux)
- Show warnings / reasons, if no connection could be established
## Test Plan
Regarding if the devices have mentioned packages installed, we should see different behaviors.
If the remote device has no VNC client, it can not connect to out server. _A warning should be shown._
If the local device hasn't the `krfb-virtualmonitor` available, the remote device couldn't connect. _A warning should be shown._
If both problems are present, _both warnings should be shown._
If none of these are present, no warning should be shown and we should try to establish a connection.
The connection attempts failed? _A warning should be shown._
Summary:
Show the plugin's icons in the list
Depends on D14567
{F6172885}
Reviewers: #kde_connect, apol
Reviewed By: #kde_connect, apol
Subscribers: kdeconnect
Tags: #kde_connect
Differential Revision: https://phabricator.kde.org/D14571
Summary: It's not only used by the plasmoid any more but also SFOS and the app use it.
Test Plan: Builds, App runs fine
Reviewers: #kde_connect, piggz, apol
Reviewed By: #kde_connect, piggz, apol
Subscribers: piggz, kdeconnect
Tags: #kde_connect
Differential Revision: https://phabricator.kde.org/D14567
2018-08-03 02:24:58 +02:00
Renamed from plasmoid/declarativeplugin/qml/PluginChecker.qml (Browse further)
