Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loss of user data when editing a comment #1261

Open
kukutz opened this issue Jan 24, 2021 · 5 comments
Open

Loss of user data when editing a comment #1261

kukutz opened this issue Jan 24, 2021 · 5 comments

Comments

@kukutz
Copy link
Contributor

kukutz commented Jan 24, 2021

  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
@davidmz
Copy link
Member

davidmz commented Jan 24, 2021

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

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

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

@indeyets
Copy link
Contributor

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

@kukutz
Copy link
Contributor Author

kukutz commented Jan 24, 2021

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

@davidmz
Copy link
Member

davidmz commented Jan 24, 2021

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

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

@kukutz
Copy link
Contributor Author

kukutz commented Jan 24, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants