search

Pet-projects-CodePET / Frontend

5 stars 1 forks source link

Feature/specialist detailed page #219

Closed Primis1 closed 1 week ago

Primis1 commented 3 weeks ago
Julia-Papina commented 3 weeks ago

Олег, посмотрела ПР, что в первую очередь увидела: карточка специалиста( карточка-превью из списка всех специалистов) должна быть ссылкой, при клике на которую открывается детальная страница. При клике по кнопке "Пригласить в проект" будет открываться модальное окно с сопроводительным письмом. В требованиях это описано, прикреплю скрин. Подобный пример можно посмотреть с проектами и детальной страницей проектов (в ветке develop). Поэтому это нужно будет поправить. В app/specialists/details-about-specialist сейчас висит ошибка с типом, из-за этого build не собирается (проверяем всегда перед мержем npm run build). И еще момент с версткой планшета и мобилки, нужно доделать, хотя бы приближенно к макету или пока резиновой с перестройкой блоков, чтобы на планшете и мобилке тоже можно было посмотреть страницу, сейчас этих разрешений совсем нет, уходит в горизонтальный скролл. 2024-06-10_13-26-39 2024-06-10_14-04-29

Primis1 commented 3 weeks ago

Оке все будет

On Mon, Jun 10, 2024 at 7:31 AM Julia Papina @.***> wrote:

Олег, посмотрела ПР, что в первую очередь увидела: карточка специалиста( карточка-превью из списка всех специалистов) должна быть ссылкой, при клике на которую открывается детальная страница. При клике по кнопке "Пригласить в проект" будет открываться модальное окно с сопроводительным письмом. В требованиях это описано, прикреплю скрин. Подобный пример можно посмотреть с проектами и детальной страницей проектов (в ветке develop). Поэтому это нужно будет поправить. В app/specialists/details-about-specialist сейчас висит ошибка с типом, из-за этого build не собирается (проверяем всегда перед мержем npm run build). И еще момент с версткой планшета и мобилки, нужно доделать, хотя бы приближенно к макету или пока резиновой с перестройкой блоков, чтобы на планшете и мобилке тоже можно было посмотреть страницу, сейчас этих разрешений совсем нет, уходит в горизонтальный скролл. 2024-06-10_13-26-39.png (view on web) https://github.com/Pet-projects-CodePET/Frontend/assets/118973229/b849c0a5-5362-4a6a-b819-8da82769dbb4 2024-06-10_14-04-29.png (view on web) https://github.com/Pet-projects-CodePET/Frontend/assets/118973229/dd6f25e5-f9d8-43d9-a19c-39ca8ac1cbb1

— Reply to this email directly, view it on GitHub https://github.com/Pet-projects-CodePET/Frontend/pull/219#issuecomment-2158092223, or unsubscribe https://github.com/notifications/unsubscribe-auth/BDWPMD745A7D25NZMDJVWCDZGWFBXAVCNFSM6AAAAABJBO44PKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYGA4TEMRSGM . You are receiving this because you authored the thread.Message ID: @.***>

Julia-Papina commented 2 weeks ago

в detailed-specialist-card.module.scss .specialist { } По макету на разрешении для планшете ширина 780px. Сейчас ширина меньше, отличается сильно от макета

Julia-Papina commented 2 weeks ago

В компоненте InviteSpecialist переиспользуем shared/text-editor, библиотека react-quill. Он уже есть в редактировании профиля.