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

the react project #25

Open
DinamickaDevelopment opened this issue Jul 25, 2017 · 6 comments
Open

the react project #25

DinamickaDevelopment opened this issue Jul 25, 2017 · 6 comments

Comments

@DinamickaDevelopment
Copy link

  1. При нажатии на изображение выполняется переход по пути '/id', где id - id новости. На открывшейся странице отображается дата и текст новости?
  2. При нажатии на кнопку load more на странице должны отображаются следующие 8 новостей из файла data.json.
@Serginio06
Copy link
Owner

Пунтк 1. В ТЗ на мой взгляд есть двузначность:

  • При нажатии на изображение выполняется переход по пути '/id', где id - id новости. На открывшейся странице отображается дата и текст новости.
  • Маршрутизацию нужно реализовать без перезагрузки страницы.
    Маршрутизация идет по новости с параметром /id без перезагрузки страницы - т.е. новость выводится в блоке ниже - вся страница не перерендеривается. В моем понимании я выполнил задачу в соответсвии с пунктом 2. Если реализовывать согласно пункта 1 т.е. "На открывшейся странице отображается дата и текст новости" то это будет новая страница где весь контент сайта нужно будет убрать - то есть перерендерить все страницу. Или я что-то не так понял?

@Serginio06
Copy link
Owner

Пункт 2. Так и есть - следующие 8 новостей выводятся с файла json с помощью Redux. Я просто картинки новостей не меняю так как их не было в json файле поэтому они статичны. Изменение можно проследить по датам новостей - они соответствуют датам новостей из файла.

@DinamickaDevelopment
Copy link
Author

  1. Мне так и не удалось открыть новость целиком при нажатии на нее, хотя нажатие срабатывает и путь меняется.

  2. Обычно, когда нажимаешь на кнопку "loadmore" догружается несколько новых блоков на страницу ниже тех, что уже показаны.

@Serginio06
Copy link
Owner

  1. http://prntscr.com/g06hnh - блок с полной новостью отображается под кнопкой Load More. Я просто ничего более оригинального не придумал как его отобразить. Если возможно дать более развернутое описание как оно должно отобразиться - я переделаю.

  2. Понял. Спасибо

@DinamickaDevelopment
Copy link
Author

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

@Serginio06
Copy link
Owner

Исправил пункт 2 с Load More.

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

No branches or pull requests

2 participants