Commit graph

113 commits

Author SHA1 Message Date
Nicolas Fella
7193cfc927 [plasmoid] Use PlaceholderMessage from PlasmaExtras instead of Kirigami
Otherwise the style may be incorrect

BUG: 434197
2021-03-09 19:20:05 +01:00
Nicolas Fella
1060c1a81a Add API for opening the KCM to daemon
We have a few places that open the KCM, with different arguments.

Centralize the implementation in one place.

This makes it easier to switch to invoking systemsettings5 in the future (once https://invent.kde.org/plasma/systemsettings/-/merge_requests/11 is in).

It also makes sure the relevant device is selected when clicking on a pairing notification.

The function is exposed to DBus for the Plasmoid and potential third-party users.

CCBUG: 425660
2020-10-18 18:40:48 +00:00
Nicolas Fella
f90d6a019f Rework the battery plugin
Instead of having the DBus stuff in a separate class expose the plugin class itself like we do for the other plugins.

Replace the method-based API with properties.

Change the path to <device>/battery for consistency with other plugins
2020-08-27 15:04:40 +00:00
Nicolas Fella
87db95b22f Convert license headers to SPDX expressions 2020-08-17 09:48:10 +00:00
Nicolas Fella
198789e6fc Fix file dialog usage in app and plasmoid
fileDialog.open() does *not* block, so we need to move the sending to onAccepted. Not sure why I thought this ever worked.

BUG: 423975
2020-07-18 21:57:15 +00:00
Nate Graham
a9d0cb64b7 [applet] Port to Kirigami.PlaceholderMessage
This new Kirigami component was just introduced with
https://phabricator.kde.org/D29057.

ref T13021
2020-04-22 22:49:26 +00:00
Nate Graham
8e0cfd3899 [applet] Standardize presentation on the "devices unavailable" view
Summary:
This patch makes the "devilces unavailable" page follow the general style used in other
similar views throughout Plasma and other KDE software. It also improves the wording
of various buttons and headers to make them more context-sensitive and thus more relevant
to what the user needs to do to solve the problem.

Test Plan:
{F8149680}

{F8149679}

Reviewers:
#VDG, #kde-connect

Subscribers:
2020-03-15 19:53:53 +00:00
George Vogiatzis
875cf16a40 [Plasmoid]Make device menu use PC ContextMenu
Summary:
Change the three dotted menu to use PlasmaComponents ContextMenu.
This prevents menu from staying open on other system tray's applets.
Wayland still has that issue, happens in other menus too.
BUG: 415142

And change menu icon into a hamburger menu.

Test Plan:
Before:
Open the menu, and then open an other applet. Menu will stay open on
the left. Even if change windows and open the system tray, will still
be there.
After:
Use X11. By clicking, for example volume applet, menu will minimize.
Will need to click again to open the applet.
2020-02-21 11:42:31 +02:00
Aleix Pol
36bcade705 plasmoid: move plasmoid buttons into a menu
This way we get to see what they do together with the icon.
Removes some visual clutter as well.

Reviewed by Albert Vaca
2019-10-12 20:06:12 +02:00
Aleix Pol
8a919a8015 Remove clipboard integration on the plasmoid
It's redundant to the clipboard plugin and remote keyboard.
It clutters the UI.

Reviewed by Albert Vaca
2019-10-12 19:44:09 +02:00
Albert Vaca Cintora
db626fa4bd Hide the install buttons if you have already paired some device 2019-09-20 12:24:30 +02:00
Albert Vaca Cintora
7eeed141f0 Add a button to open the settings
If you already have kdeconnect on your phone, the call to action to install
it doesn't make sense.

This doesn't add a string because it reuses one that was already in the
plasmoid
2019-09-19 18:48:58 +02:00
Albert Vaca Cintora
9f8928513d Don't hide the plasmoid, otherwise the install buttons don't make sense 2019-09-19 18:39:45 +02:00
Nicolas Fella
6c40364490 Fix SMS icon in plasmoid 2019-08-15 19:22:29 +02:00
Aleix Pol
20f4ed3e9b sms: forgot to add the file 2019-07-21 20:13:10 +02:00
Aleix Pol
c17972881b sms: Added opening kdeconnect-sms on the plasmoid and indicator
Add a menu entry on the kdeconnect-indicator within devices' submenu that opens kdeconnect-sms.
Also add it in the plasmoid in the device delegate.
2019-07-21 18:30:41 +02:00
Matthijs Tijink
24eb0799f1 Show install instructions for Android when there are no devices 2019-07-20 18:21:41 +02:00
Matthijs Tijink
f8184f84bc Make more of the device name/battery show up in the plasmoid 2019-06-25 16:40:10 +00:00
Nicolas Fella
f9dee192b5 Completely hide plamoid in system tray if no device is connected 2019-06-20 16:22:43 +00:00
Nicolas Fella
1999a0b49e Allow to share multiple files from plasmoid 2019-06-15 18:10:53 +02:00
Nicolas Fella
5a0b82dd6d Fix whitespace 2019-06-15 17:44:27 +02:00
Nicolas Fella
a6dd1f5465 Translate file dialog title 2019-06-15 17:44:11 +02:00
Nicolas Fella
3b8fedea1a Replace http GNU urls with https 2019-03-23 17:29:26 +01:00
Aleix Pol
d650faaa4f Fix warning 2019-01-31 16:38:21 +01:00
Simon Redman
bafbc344b9 Implement Drag n Drop file transfer on the plasmoid
Summary:
A DropArea has been added to the ListItem representing a device in the
plasmoid based on what is done in the Quickshare plasmoid. This enables
Drag n Drop of files onto a section of the plasmoid to transfer files.

The DropArea has the size of the ListItem which is a rectangle
including the title of the device down to the button to share text.
Additionally, there is a tooltip area covering the DropArea to indicate
that it is possible to drop a file to send it to the device. Multiple files
can be sent at once.

Ref T8288

Test Plan: Drag and Drop one or many files onto the widget, the files should be transferred to the connected device.

Reviewers: #kde_connect, sredman

Reviewed By: #kde_connect, sredman

Subscribers: sredman, kdeconnect

Tags: #kde_connect

Maniphest Tasks: T8288

Differential Revision: https://phabricator.kde.org/D17611
2019-01-02 10:26:40 -07:00
Nicolas Fella
99e335ede2 Revert "Fix devicename in Plasmoid"
This reverts commit e029c61935.
2018-11-26 15:50:44 +01:00
Nicolas Fella
e029c61935 Fix devicename in Plasmoid 2018-11-22 16:45:53 +01:00
Billy Laws
9fc55c72f0 Use plasma TextArea for sharing text in plasmoid
Summary:
This also fixes a bug where the box could be too small.
BUG: 400862

Reviewers: #kde_connect, nicolasfella

Reviewed By: #kde_connect, nicolasfella

Subscribers: nicolasfella, sredman, apol, kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D16782
2018-11-09 19:07:10 +00:00
Nicolas Fella
952da637b0 Share RemoteKeyboard implementation
Summary: Extract RemoteKeyboard.qml into declarativeplugin to be able to use it from both plasmoid and app without code duplication

Reviewers: #kde_connect, hkaelberer, apol

Reviewed By: #kde_connect, apol

Subscribers: kdeconnect, mtijink, apol, #kde_connect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D12670
2018-11-08 02:20:34 +01:00
Billy Laws
061d415f83 Add support for sharing text from plasmoid
Summary:
This allows you to enter text in the plasmoid and it be sent
to the connected device.

Reviewers: #vdg, #kde_connect, albertvaka

Reviewed By: #kde_connect, albertvaka

Subscribers: apol, kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D16696
2018-11-07 21:52:53 +00:00
Nicolas Fella
fb2817e3f3 Add missing file 2018-11-02 22:13:36 +01:00
Nicolas Fella
56ebd3f889 Fix with calculation 2018-11-02 15:47:52 +01:00
Nicolas Fella
cefc743307 Only show addCommand button when adding commands is supported 2018-11-02 15:44:14 +01:00
Nicolas Fella
e0766ba532 Align buttons 2018-11-02 15:38:52 +01:00
Billy Laws
eaea0ab667 Add support for sharing files from plasmoid
Summary: This allows you to share a local file on your device to a remote device running kde connect.

Reviewers: nicolasfella

Reviewed By: nicolasfella

Subscribers: kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D16552
2018-11-01 18:33:39 +01:00
Billy Laws
c546c77040 Add a button to open command setup menu in plasmoid
Summary: This patch adds a button to the plasmoid to allow adding commands by opening the menu on the remote device.

Reviewers: #vdg, nicolasfella, albertvaka

Reviewed By: nicolasfella

Subscribers: kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D16535
2018-10-30 23:42:56 +01:00
Yuri Chornoivan
5fe74ce041 Fix minor EBN issues and typos 2018-10-07 21:23:20 +03:00
Nicolas Fella
d7d2917e00 Use new icon for clearing notifications
Summary: {F6303876}

Reviewers: #kde_connect, ngraham, thsurrel

Reviewed By: ngraham

Subscribers: kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D15948
2018-10-04 19:25:53 +02:00
Nicolas Fella
fc4d4e7685 Enable running commands from the plasmoid
Summary: {F5926667}

Reviewers: #kde_connect, apol

Reviewed By: #kde_connect, apol

Subscribers: apol, kdeconnect, #kde_connect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D13660
2018-07-27 08:11:03 +02:00
Thomas Surrel
5ab26096a2 Change the 'Dismiss all notifications' icon
Summary: This matches the KDE notification icon.

Reviewers: apol

Reviewed By: apol

Subscribers: abetts, Fuchs, apol, kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D13534
2018-06-14 17:45:01 +02:00
Nicolas Fella
4d8863bacc Start cleaning up RemoteKeyboard QML code
Summary:
The code for visually disabling the input field is not needed any more since the field is completely hidden when not needed. Also drop the custom available property from the text field.

More cleanups will follow. My long-term goal is to reuse the code in the app to enable remote keyboard input on non-Plasma DEs

Test Plan: (De)activate the remote keyboard on Android.

Reviewers: #kde_connect, apol

Reviewed By: #kde_connect, apol

Subscribers: apol, #kde_connect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D12669
2018-05-09 01:15:14 +02:00
Matthijs Tijink
749351ec64 Add tooltips to the "dismiss" buttons for notifications
Summary: It also makes it clearer that the top line is not a notification, but a header.

Reviewers: #kde_connect, apol

Reviewed By: #kde_connect, apol

Subscribers: #kde_connect

Differential Revision: https://phabricator.kde.org/D11646
2018-03-24 16:40:09 +01:00
Matthijs Tijink
dbce29a66f Revert "Add a refresh button to the plasmoid"
This reverts commit ca4b4227ce.
2018-03-21 17:36:24 +01:00
Matthijs Tijink
ca4b4227ce Add a refresh button to the plasmoid
Summary: This adds a refresh button to the plasmoid if no device is connected.

Test Plan: Refreshing works, and the button only appears when no devices are connected.

Reviewers: #kde_connect, nicolasfella

Reviewed By: #kde_connect, nicolasfella

Subscribers: apol, nicolasfella, #kde_connect

Differential Revision: https://phabricator.kde.org/D11501
2018-03-20 19:53:13 +01:00
Kai Uwe Broulik
0b7578a5e2 Remove custom compact representation
It doesn't do anything special

Differential Revision: https://phabricator.kde.org/D11259
2018-03-12 12:29:15 +01:00
Kai Uwe Broulik
1c3a45fd95 Get rid of ProcessRunner
Use KCMShell singleton in kquickcontrolsaddons
Also while at it honor kiosk restriction

Differential Revision: https://phabricator.kde.org/D9759
2018-01-09 16:33:02 +01:00
Nicolas Fella
5d4821fb9c Only show notification title in plasmoid if different to app name
Summary: Like we do for the notifications in the plugin already. Passing the title and text separately to the plasmoid could be useful later as well.

Reviewers: #kde_connect, albertvaka

Reviewed By: #kde_connect, albertvaka

Subscribers: #kde_connect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D7457
2017-08-22 18:18:34 +02:00
Nicolas Fella
66e5eefe20 Gave plasmoid some love
Summary:
Make the plasmoid content transparent
Include battery info in header
Only show remotekeyboard input if available
Make header always centered

before:
{F3820966}

after:
{F3820965}

Reviewers: #kde_connect, apol

Reviewed By: #kde_connect, apol

Subscribers: jeanv, albertvaka, apol, #kde_connect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D6943
2017-07-27 17:15:07 +02:00
Fabian Vogt
0667c6556f Merge branch '1.x' 2017-07-12 11:37:41 +02:00
Fabian Vogt
5641d818dc Treat device names as plaintext, not rich text
Summary:
Notifications, QML Text and QLabel accept a HTML subset,
which does not make sense for device names.

BUG: 382243

Test Plan:
Sent a pair request and accepted it, device name
now shown as plain text everywhere.

Reviewers: #kde_connect, albertvaka

Reviewed By: #kde_connect, albertvaka

Subscribers: #kde_connect

Differential Revision: https://phabricator.kde.org/D6640
2017-07-12 11:33:21 +02:00