search

ya-pomogau / frontend

7 stars 8 forks source link

Frontend Blog-page; #87 #129

Closed margo-yunanova closed 6 months ago

margo-yunanova commented 7 months ago

Список вопросов, которые не предусмотрены дизайном и предлагаю решить после MVP:

  1. Отмена редактирования поста;
  2. Пагинация списка постов: будет ли кнопка "подгрузить ещё", кнопки со списком страниц, либо бесконечный скролл;
  3. Будет ли попап с увеличенной картинкой при клике?
  4. Проблема со скроллом, он везде отключён и его нужно решать одновременно для всех компонентов.
kspshnik commented 7 months ago

Список вопросов, которые не предусмотрены дизайном и предлагаю решить после MVP:

  1. Отмена редактирования поста;
  2. Пагинация списка постов: будет ли кнопка "подгрузить ещё", кнопки со списком страниц, либо бесконечный скролл;
  3. Будет ли попап с увеличенной картинкой при клике?
  4. Проблема со скроллом, он везде отключён и его нужно решать одновременно для всех компонентов.

@margo-yunanova создай, пожалуйста, новый issue, чтобы не потерялось.

kspshnik commented 7 months ago

И добавь коллег для peer-review, пожалуйста

margo-yunanova commented 7 months ago

Я сделала пока драфт, вы пока не ревьювьте, т.к. доделываю ещё страницу, как будет готов, поменяю статус.

kspshnik commented 7 months ago

Я сделала пока драфт, вы пока не ревьювьте, т.к. доделываю ещё страницу, как будет готов, поменяю статус.

ОК. Я тогда исправил и статус в проекте на более корректный :)

margo-yunanova commented 7 months ago

Всё готово, можно проверять PR.

При нажатии на кнопку Опубликовать всё падает, потому что json server не может сохранить вложения, проверить работоспособность без вложений картинок можно в коммите https://github.com/ya-pomogau/frontend/pull/129/commits/37046b4b5e99e03317f058f61e6f09876cf993c6

На текущем этапе не реализовано редактирование картинок в уже опубликованном посте. Для MVP текущего функционала должно быть достаточно и я предлагаю эту часть дописать отдельным issue, после того как сделаю бэк и будет понятно, как сообщить серверу какие картинки удалились, остались, новые.

Чтобы проверить скорлл смените на overflow-y: scroll здесь page-layout/styles.module.css. Вопрос со скроллом глобальный, его предлагаю решать отдельным issue и делать сразу на всех страницах. В случае блога скролировать только внутри секции с постом не удобно, мышь нужно прям на неё наводить, и это не стандартное поведение.

Бэкграунд я тоже предлагаю заменить отдельным issue, т.к. он устанавливается на всю страницу и может затронуть поведение многих компонентов.

kspshnik commented 7 months ago

На текущем этапе не реализовано редактирование картинок в уже опубликованном посте. Для MVP текущего функционала должно быть достаточно и я предлагаю эту часть дописать отдельным issue, после того как сделаю бэк и будет понятно, как сообщить серверу какие картинки удалились, остались, новые.

Смотри, с картинками всё чуть сложнее: они будут загружаться на отдельную ручку на бэкенде, которая будет возвращать их временный id. И загружаться они будут сразу, не дожидаясь сабмита. Более того - до загрузки всех картинок и наличия всех временных id отправка формы должна быть заблокирована. Эти id мы будем отправлять в соответствующих текстовых полях формы (как делали в Месте с аватарой), а уже бэкенд по ним разберётся, что делать с соответствующим файлом :)