Skip to content

Loss of user data when editing a comment #1261

Open

Description

@kukutz
  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

Activity

  1. davidmz commented on Jan 24, 2021

    @davidmz
    Member

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

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

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

  2. indeyets commented on Jan 24, 2021

    @indeyets
    Contributor

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

  3. kukutz commented on Jan 24, 2021

    @kukutz
    ContributorAuthor

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

  4. davidmz commented on Jan 24, 2021

    @davidmz
    Member

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

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

  5. kukutz commented on Jan 24, 2021

    @kukutz
    ContributorAuthor

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

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions