freefeed-react-client icon indicating copy to clipboard operation
freefeed-react-client copied to clipboard

Loss of user data when editing a comment

Open kukutz opened this issue 4 years ago • 5 comments

  1. Any OS, browser and affects both desktop and mobile
  2. Start editing last comment in thread which is not expanded (you can see only last comment). Wait until someone add new comment to this thread
  3. Expected result: new comments added beneath your comment
  4. Actually: new comment hides your comment and your edited text is lost

kukutz avatar Jan 24 '21 09:01 kukutz

Тут следующие нюансы. У нас сейчас всегда, при количестве комментов достаточном для фолда, видны только первый и последний. Когда прилетает новый коммент, он автоматически становится последним, а предпоследний уходит под фолд. Очевидно, именно это поведение нужно будет менять.

  1. В Гамме после фолда показываются до двух комментариев. Это слегка смягчает проблему, но не решает её полностью — если мы редактируем предпоследний коммент, то новый всё равно загонит его под фолд. Можно увеличить это количество, скажем, до 3-4, на практике, вероятно, этого хватит всем™.
  2. Можно вообще не класть новые комментарии под фолд. Вроде бы так было в старом фидике — в свежезагруженной ленте был пост со свёрнутыми комментариями, а новые просто добавлялись после фолда. Недостаток — если оставить ленту открытой надолго, под свёрнутыми постами вырастут длинные треды.
  3. Можно не сворачивать новые комментарии, пока один из них редактируется. То есть, если юзер начал редактирование после фолда, все новые комментарии будут подклеиваться снизу. Когда он закончит редактирование, сработает фолд.

Я сам бы, наверное, предпочёл комбинацию из 1 и 3 — показывать до 3 новых комментариев после фолда, но пока коммент редактируется, все новые комментарии просто добавляются внизу.

davidmz avatar Jan 24 '21 11:01 davidmz

А ещё дополнительно можно разделить представление (Fold) и хранение. И не удалять из редукса комментарии ушедшие под фолд 🤔

indeyets avatar Jan 24 '21 12:01 indeyets

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

kukutz avatar Jan 24 '21 12:01 kukutz

@indeyets они у нас и так не очень жёстко связаны, у нас ведь можно раскрыть комментарии, а потом закрыть. Из стора они при этом не удаляются.

@kukutz там у @clbn в посте есть обоснование, почему не всегда два. Это позволяет увидеть, какие посты на странице обновились после загрузки, а какие нет.

davidmz avatar Jan 24 '21 13:01 davidmz

ну ладно, окей, я за 1+3 )

kukutz avatar Jan 24 '21 13:01 kukutz