Commit Graph

21 Commits

Author SHA1 Message Date
091ec35580 Stop built-in Syncthing instance synchronously
So `Launcher::terminate()` exits when really everything has terminated and
not just sub processes. It may nevertheless be worthwhile to implement a
timeout.

This may help with
https://github.com/Martchus/syncthingtray/issues/180#issuecomment-1582465960.
2023-06-09 13:13:55 +02:00
dd936b4a81 Workaround lupdate not understanding "enum class" in some cases
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.
2023-06-07 23:14:48 +02:00
925ce383f0 Fix condition and code in supposedAutostartPath()
The `1 || ` was just a debugging leftover.
2023-06-07 22:08:53 +02:00
cca3a8b364 Show "Points to …" in setup detection only if autostart is enabled
Otherwise the path will just be empty.
2023-05-18 23:52:23 +02:00
bc736214dd Allow keeping existing autostart entry in wizard 2023-05-18 23:50:27 +02:00
d3a559446f Apply clang-format 2023-05-11 17:57:22 +02:00
0240f2a9a3 Improve recent autostart-related changes further
* Improve function documentation
* Improve regex to better handle quoting
* Avoid using function only available as of Qt 6.3
2023-05-11 17:50:16 +02:00
eb1b81a8d4 Enable auto-connect via the wizard
This setting should normally not hurt and is likely a better default.
2023-05-11 01:13:05 +02:00
603ba1206e Avoid warnings about unused variables when compiling without systemd support 2023-05-11 00:55:12 +02:00
29ada0f040 Override existing startup entry in wizard; show remark about it in summary 2023-05-11 00:54:42 +02:00
878e973c9d Improve auto start setting
* 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
2023-05-11 00:11:34 +02:00
d2bbbab4f2 Remove remark about default browser when built-in web view disabled
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.
2023-04-15 20:35:24 +02:00
fc8ab2e845 Improve coding style
Improve coding style, mainly in SynctingConnection, for better
consistency with more recently added code.
2023-04-15 20:28:51 +02:00
a5f1ffd439 Never check option for built-in web view when disabled
Even if configured in the settings, check the option to
use a browser tab as this will be used as fallback.
2023-04-06 01:03:13 +02:00
c08d6e54be Save general web view settings persistently also when built-in web view disabled 2023-04-06 00:58:58 +02:00
4b77c313eb Fix compilation with Qt < 6.1.0 on Windows 2023-04-04 22:59:40 +02:00
3fd45accde Allow configuring custom command for launching Chromium-based browser 2023-03-31 22:46:25 +02:00
2767c9484a Fix size of wizard's window icon when using Aero-style
Apparently `wizardStyle()` does not return `QWizard::AeroStyle` at this
point of construction, even if that style will be used.
2023-03-28 23:56:23 +02:00
58b8a4e540 Allow showing Syncthing UI in Chromium-based browser in "app mode"
See https://github.com/Martchus/syncthingtray/issues/178
2023-03-27 20:28:52 +02:00
e71a928ae2 Avoid unusable wizard when dark mode is enabled on Windows 2023-03-23 22:59:25 +01:00
96e92f13a3 Avoid relying on symlink for include directories
Symlinks aren't working very well with Git under Windows so better avoid
them.
2023-02-27 18:27:33 +01:00