This is useful when the target platform/packaging is not actually supported, e.g.
currently the code does not work when Syncthing Tray is packaged as Flatpak.
* Use command syntax and double-dashes to be in-line with what Syncthing's
help text suggests
* Drop the `--no-restart` parameter; I probably only added it after
checking what flags the systemd-unit file specifies. However, the
systemd-unit file specifies `--no-restart` only because restarts are
handled by systemd in that case. For the launcher of Syncthing Tray it
makes probably more sense to keep restarts enabled by default.
This is the same as the feature already present in the launcher and
connection settings. Not sure how much sense it makes for systemd services
as they run rather independently from the UI but maybe it is useful.
* Add `--wait` parameter so Syncthing Tray will wait until the tray becomes
available; this might help if Syncthing Tray is started by the desktop
environment before the tray is available (unless it is started too soon
even for that)
* Add `X-LXQt-Need-Tray=true` to ensure the system tray is available on
LXQt because just `--wait` might not be sufficient here (according to
https://github.com/Martchus/syncthingtray/issues/240); this is in-line
with the autostart entry of https://github.com/lxqt/lxqt-powermanagement
Since this feature will likely not be available in mingw-w64-based builds of
the next release (due to missing cppwinrt) it is important to make it clear
when those options don't actually work. With this change the checkboxes are
completely disabled in case the feature is not available (instead of just
providing the information via the tooltip).
* Allow to pause all devices on metered network connections (devices will
be automatically resumed when network is no longer metered)
* Allow to stop Syncthing when it was started via the built-in launcher
on metered network connections (it will be automatically started again
when the network connection is no longer metered)
* See https://github.com/Martchus/syncthingtray/issues/231
This reverts commit 8cac56c246 and extends
the related comment as the new Windows 11 style actually supports custom
palettes and then wizard's aero style breaks in the same way as with
Fusion.
This test would keep the autostart disabled but override an existing
autostart entry (see https://github.com/Martchus/syncthingtray/issues/221).
With this change it will no longer override an existing entry. Reading and
writing the autostart entry is now mocked and properly checked.
This makes the code simpler and avoids inconsistent spacing. Unfortunately
this function has only been introduced in Qt 6.4 so the old code has to
stay for older Qt versions.
* Remember new size after user resized the window
* Remember new position after user repositioned window and using explicit
coordinates has been configured in the appearance settings
Apparently this is still problematic leading to errors like
`Qualifying with unknown namespace/class`. It seems that moving the enum to
a different header file works. The trick to use a macro to define
`enum class` ceased to work on the other hand.
Specifying a function with the namespace like
`void QtGui::FinalWizardPage::showResults()` leads to the same problem and
also needs to be avoided.
* Do not blindly override existing auto start entry
* Warn if an auto start entry exists but points to a different executable
* Allow removing such an entry explicitly
This is now misleading as it depends on the general web settings. The
presence of this new settings tab alone should make it clear what will
happen instead so the sentence can just be dropped.
Martchus
John Veness