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

닫힘
crtxcr "wip/rearrange 에서 master 로 0 commits 를 머지하려 합니다"
소유자

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 검토됨 2020-09-27 22:38:45 +02:00
crtxcr left a comment
Author
소유자

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));
Author
소유자

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 검토됨 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;
Author
소유자

Dirty

Dirty
Author
소유자

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 검토됨 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)");
Author
소유자

Space

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

Pull request closed

"로그인하여 이 대화에 참여"
No reviewers
마일스톤 없음
담당자 없음
참여자 1명
알림
마감일
기한이 올바르지 않거나 범위를 벗어났습니다. 'yyyy-mm-dd'형식을 사용해주십시오.

마감일이 설정되지 않았습니다.

Reference: crtxcr/qsrun#15
No description provided.