Fix bug of updating the start page #271
Conversation
|
Слушай, а может нам тут пойти другим путём - не обновлять инфу о проекте на стартовой странице до тех пор, пока он не был сохранён с изменениями? |
|
хмм... принял, переделаю) |
|
я скорее хотел спросить - что если такой подход рассмотреть? будет нам от этого лучше вообще? не смутит ли пользователей, что у них тексты/картинка не сразу обновляется, а только после сохранения? |
|
ну кстати да, немного неожиданное поведение получится, как мне кажется. |
|
вопросы - это только в самый критический момент стоит использовать, так по плёвой ситуации, лучше ими не сыпать) всё таки у нас данные сохраняются автоматом довольно часто, если конечно не отключено автоматическое сохранение... но а если отключено, то тогда вроде как логично, что инфа не обновилась, т.к. ты изменения не сохранил. Давай реально сделаем обновление инфы со списком проекта в момент сохранения модели проекта, если там были какие-то изменения |
iljukhaput
force-pushed
the
fix/SAD-91-updating-project-info-on-start-page
branch
from
03f1cb3 to
d248efd
Compare
|
переделал, перезалил |
|
Как по ощущениям, стало лучше?) |
|
сложно сказать какой из вариантов лучше с точки зрения пользователей) |
|
кстати, тут понял, что осталась проблема с обложкой:
сейчас это поправлю, перезалью |
iljukhaput
force-pushed
the
fix/SAD-91-updating-project-info-on-start-page
branch
from
d248efd to
5885f7b
Compare
|
перезалил плюс добавил обработку такого случая:
в итоге в БД записывается только последняя обложка, а не обе |
iljukhaput
force-pushed
the
fix/SAD-91-updating-project-info-on-start-page
branch
2 times, most recently
from
d975b09 to
72998a8
Compare
| // | ||
| // Очистим буфер обложки проекта | ||
| // | ||
| projectManager->clearCoverBuffer(); |
There was a problem hiding this comment.
почему мы делаем это тут, а не внутри метода сохранения измененеий менеджера проекта?
| @@ -2755,16 +2761,8 @@ void ApplicationManager::initConnections() | |||
| [this] { d->markChangesSaved(false); }); | |||
| connect(d->projectManager.data(), &ProjectManager::projectUuidChanged, | |||
| d->projectsManager.data(), &ProjectsManager::setCurrentProjectUuid); | |||
| connect(d->projectManager.data(), &ProjectManager::projectNameChanged, this, | |||
There was a problem hiding this comment.
вот тут мы убираем обновление инфы о проекте - как она теперь обновляется при открытии проекта?
| struct { | ||
| Domain::DocumentImage cover = {}; | ||
| bool isChanged = false; | ||
| } coverBuffer; |
There was a problem hiding this comment.
так, а зачем нам тут буфер нужен? почему не получается использовать просто одно изображение?
|
Смотрю я короче на него и он мне не нравится) как будто должно быть всё как-то проще, типа если изменения ввели и пользователь сохраняет проект - они обновились на странице проектов, если не сохранил, то список проектов остался в том виде, в котором был... сейчас получается как-то наворочено, буфер, методы очистки буфера торчат во все стороны))) хочется максимально тупого решения что ли |
|
так, вернусь тогда к этой задаче после ремарок, ок? |
|
Да, конечно. |
No description provided.