WIP: Rearrange Pushbuttons, Deletion, Add system entries to favourites, Save #15

Suljettu
crtxcr haluaa yhdistää 0 committia lähteestä wip/rearrange kohteeseen master
Omistaja

Missing

  • QIcon information loss (cannot get the icon path from it in all cases)
  • Missing assignment in Entryconfig update
  • Find next free slot in grid (next row/col) to implement "add to favoruites"
  • Deletion
Missing * [x] QIcon information loss (cannot get the icon path from it in all cases) * [x] Missing assignment in Entryconfig update * [x] Find next free slot in grid (next row/col) to implement "add to favoruites" * [x] Deletion
crtxcr added a new dependency 2020-09-13 21:29:18 +02:00
crtxcr added a new dependency 2020-09-13 21:29:40 +02:00
crtxcr reviewed 2020-09-27 22:38:45 +02:00
crtxcr jätti kommentin
Tekijä
Omistaja

Problems

Problems
window.cpp Outdated
@ -126,0 +144,4 @@
QMessageBox::critical(this, "Failed to save item to favourites", e.what());
return;
}
userEntryButtons.append(createEntryButton(userConfig));
Tekijä
Omistaja

This is not a complete button since inheritance is not actually applied here, causing e. g. the icon not to show.

This is not a complete button since inheritance is not actually applied here, causing e. g. the icon not to show.
crtxcr marked this conversation as resolved
crtxcr reviewed 2020-09-27 22:44:05 +02:00
window.cpp Outdated
@ -126,0 +134,4 @@
userConfig.inherit = button.getEntryConfig().entryPath;
QFileInfo fi{button.getEntryConfig().entryPath};
QString entryName = fi.completeBaseName() + ".qsrun";
userConfig.entryPath = this->settingsProvider->userEntriesPaths()[0] + "/" + entryName;
Tekijä
Omistaja

Dirty

Dirty
Tekijä
Omistaja

Problem

  1. Add to favourites
  2. Rearrange
  3. Entry from 1 gets saved completely. Has inheritances property, but it's useless since everything will be stored.

Maybe:

Remove boolean
{ SYSTEM_ENTRY, USER_ENTRY, INHERITED_ENTRY }

INHERITED_ENTRY does not save anything beyond "inherit", "rows, "cols".

**Problem** 1. Add to favourites 2. Rearrange 3. Entry from 1 gets saved completely. Has inheritances property, but it's useless since everything will be stored. Maybe: Remove boolean { SYSTEM_ENTRY, USER_ENTRY, INHERITED_ENTRY } INHERITED_ENTRY does not save anything beyond "inherit", "rows, "cols".
crtxcr reviewed 2020-09-28 19:30:12 +02:00
@ -249,0 +291,4 @@
if(ret != 0)
{
qDebug() << strerror(errno);
throw std::runtime_error("Failed to save entry file( error during rename)");
Tekijä
Omistaja

Space

Space
crtxcr marked this conversation as resolved
crtxcr closed this pull request 2020-10-04 21:55:36 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
Ei merkkipaalua
Ei käsittelijää
1 osallistujaa
Ilmoitukset
Määräpäivä
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

Määräpäivää ei asetettu.

Reference: crtxcr/qsrun#15
No description provided.