story-apps / starc

Reinventing the screenwriting software.
https://starc.app
GNU General Public License v3.0
238 stars 27 forks source link

Fix bug of updating the start page #271

Closed iljukhaput closed 1 month ago

dimkanovikov commented 2 months ago

Слушай, а может нам тут пойти другим путём - не обновлять инфу о проекте на стартовой странице до тех пор, пока он не был сохранён с изменениями?

iljukhaput commented 2 months ago

хмм... принял, переделаю)

dimkanovikov commented 2 months ago

я скорее хотел спросить - что если такой подход рассмотреть? будет нам от этого лучше вообще? не смутит ли пользователей, что у них тексты/картинка не сразу обновляется, а только после сохранения?

iljukhaput commented 2 months ago

ну кстати да, немного неожиданное поведение получится, как мне кажется. А что если нам спрашивать пользователей о сохранении при выходе из проекта на главный экран?

dimkanovikov commented 2 months ago

вопросы - это только в самый критический момент стоит использовать, так по плёвой ситуации, лучше ими не сыпать)

всё таки у нас данные сохраняются автоматом довольно часто, если конечно не отключено автоматическое сохранение... но а если отключено, то тогда вроде как логично, что инфа не обновилась, т.к. ты изменения не сохранил. Давай реально сделаем обновление инфы со списком проекта в момент сохранения модели проекта, если там были какие-то изменения

iljukhaput commented 2 months ago

переделал, перезалил

dimkanovikov commented 2 months ago

Как по ощущениям, стало лучше?)

iljukhaput commented 2 months ago

сложно сказать какой из вариантов лучше с точки зрения пользователей) но вроде вполне логичный вариант, если рассматривать список проектов, как список именно сохраненных а в плане кода стало однозначно поаккуратнее

iljukhaput commented 2 months ago

кстати, тут понял, что осталась проблема с обложкой:

сейчас это поправлю, перезалью

iljukhaput commented 2 months ago

перезалил

плюс добавил обработку такого случая:

в итоге в БД записывается только последняя обложка, а не обе

dimkanovikov commented 1 month ago

Смотрю я короче на него и он мне не нравится) как будто должно быть всё как-то проще, типа если изменения ввели и пользователь сохраняет проект - они обновились на странице проектов, если не сохранил, то список проектов остался в том виде, в котором был...

сейчас получается как-то наворочено, буфер, методы очистки буфера торчат во все стороны))) хочется максимально тупого решения что ли

iljukhaput commented 1 month ago

так, вернусь тогда к этой задаче после ремарок, ок? а то чувствую это не будет быстро)

dimkanovikov commented 1 month ago

Да, конечно.