Compare commits
15 Commits
backports/
...
ubuntu/plu
Author | SHA1 | Date |
---|---|---|
Simon Quigley | ad0110a3e8 | 1 month ago |
Simon Quigley | 930b286e48 | 2 months ago |
Simon Quigley | 1c535d4861 | 2 months ago |
Simon Quigley | 6b63130197 | 2 months ago |
Simon Quigley | 697b925f5f | 2 months ago |
Aaron Rainbolt | 68238dec46 | 4 months ago |
Aaron Rainbolt | 9babf34766 | 6 months ago |
Aaron Rainbolt | 421e3610b7 | 6 months ago |
Aaron Rainbolt | 1810f6227c | 6 months ago |
Aaron Rainbolt | 1ef85ddeac | 6 months ago |
Aaron Rainbolt | b1b4bd3f7a | 6 months ago |
Aaron Rainbolt | d8f8822d70 | 6 months ago |
Aaron Rainbolt | 5ec887ef0d | 6 months ago |
Simon Quigley | 5fd7bd3860 | 12 months ago |
Simon Quigley | 61dedca925 | 1 year ago |
@ -1,2 +0,0 @@
|
||||
[preferred]
|
||||
default=lxqt;kde;gtk;
|
@ -0,0 +1,4 @@
|
||||
# This is expected
|
||||
lxqt-session: desktop-entry-lacks-keywords-entry [usr/share/*/*.desktop]
|
||||
lxqt-session: desktop-entry-lacks-icon-entry [usr/share/*/*.desktop]
|
||||
lxqt-session: desktop-entry-invalid-category LXQt [usr/share/*/*.desktop]
|
@ -1,55 +0,0 @@
|
||||
Description: lxqt-config-session/autostart: More strict user inputs
|
||||
Makes name and command mandatory when adding or editing a autostart an
|
||||
application. Check for already existing files when editing.
|
||||
.
|
||||
TODO: Add and Edit functions are almost indentical. Candidates to some
|
||||
refactoring.
|
||||
Author: Luís Pereira <luis.artur.pereira@gmail.com>
|
||||
Origin: upstream
|
||||
Bug: https://github.com/lxqt/lxqt-session/issues/486
|
||||
Applied-Upstream: 35acc5e7fe64ab78616968a5b73b150fdf2f002a
|
||||
Last-Update: 2023-12-22
|
||||
---
|
||||
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
|
||||
--- a/lxqt-config-session/autostartpage.cpp
|
||||
+++ b/lxqt-config-session/autostartpage.cpp
|
||||
@@ -133,6 +133,11 @@ void AutoStartPage::addButton_clicked()
|
||||
while (!success && edit.exec() == QDialog::Accepted)
|
||||
{
|
||||
QModelIndex index = ui->autoStartView->selectionModel()->currentIndex();
|
||||
+ if (edit.name().isEmpty() || edit.command().isEmpty() )
|
||||
+ {
|
||||
+ QMessageBox::critical(this, tr("Error"), tr("Please provide Name and Command"));
|
||||
+ continue;
|
||||
+ }
|
||||
XdgDesktopFile file(XdgDesktopFile::ApplicationType, edit.name(), edit.command());
|
||||
if (edit.needTray())
|
||||
file.setValue(QL1S("X-LXQt-Need-Tray"), true);
|
||||
@@ -148,8 +153,14 @@ void AutoStartPage::editButton_clicked()
|
||||
QModelIndex index = ui->autoStartView->selectionModel()->currentIndex();
|
||||
XdgDesktopFile file = mXdgAutoStartModel->desktopFile(index);
|
||||
AutoStartEdit edit(file.name(), file.value(QL1S("Exec")).toString(), file.contains(QL1S("X-LXQt-Need-Tray")));
|
||||
- if (edit.exec() == QDialog::Accepted)
|
||||
+ bool success = false;
|
||||
+ while (!success && edit.exec() == QDialog::Accepted)
|
||||
{
|
||||
+ if (edit.name().isEmpty() || edit.command().isEmpty() )
|
||||
+ {
|
||||
+ QMessageBox::critical(this, tr("Error"), tr("Please provide Name and Command"));
|
||||
+ continue;
|
||||
+ }
|
||||
file.setLocalizedValue(QL1S("Name"), edit.name());
|
||||
file.setValue(QL1S("Exec"), edit.command());
|
||||
if (edit.needTray())
|
||||
@@ -157,7 +168,10 @@ void AutoStartPage::editButton_clicked()
|
||||
else
|
||||
file.removeEntry(QL1S("X-LXQt-Need-Tray"));
|
||||
|
||||
- mXdgAutoStartModel->setEntry(index, file, true);
|
||||
+ if (mXdgAutoStartModel->setEntry(index, file, true))
|
||||
+ success = true;
|
||||
+ else
|
||||
+ QMessageBox::critical(this, tr("Error"), tr("File '%1' already exists!").arg(file.fileName()));
|
||||
}
|
||||
}
|
||||
|
@ -1,3 +1,2 @@
|
||||
ensure-necessary-paths-are-pulled-from.patch
|
||||
autostart-ux-polish.patch
|
||||
gtk-3-configuration-copy.patch
|
||||
|
@ -0,0 +1,2 @@
|
||||
# We will not be forwarding this patch
|
||||
lxqt-session source: patch-not-forwarded-upstream [debian/patches/gtk-3-configuration-copy.patch]
|
Loading…
Reference in new issue