Skip to content

Sprint5changed#7

Open
Rakhim92 wants to merge 15 commits into
mainfrom
sprint5changed
Open

Sprint5changed#7
Rakhim92 wants to merge 15 commits into
mainfrom
sprint5changed

Conversation

@Rakhim92
Copy link
Copy Markdown
Owner

@Rakhim92 Rakhim92 commented Apr 23, 2023

  1. Внедрил сборку модулей JS при помощи Webpack, настроил Gulp
  2. Добавил расширенную форму, которая будет открываться по клику в модальном окне. При клике на кнопку в шапке "Записаться онлайн", кнопки записаться в секции "Популярные услуги", по клику по карточке мастера. Сама форма в черновом варианте
  3. Клонировал себе директорию сервера. Отправку данных с формы на сервер не делал
  4. CRM не делал

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2023

Deploy Preview for cheery-salamander-244b2e ready!

Name Link
🔨 Latest commit db6b4df
🔍 Latest deploy log https://app.netlify.com/sites/cheery-salamander-244b2e/deploys/6453cad34e72a50008e35941
😎 Deploy Preview https://deploy-preview-7--cheery-salamander-244b2e.netlify.app/site/src
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2023

Deploy Preview for funny-lily-79a8c1 failed.

Name Link
🔨 Latest commit db6b4df
🔍 Latest deploy log https://app.netlify.com/sites/funny-lily-79a8c1/deploys/6453cad30185a90008554aff

@Rakhim92 Rakhim92 requested a review from euge-huge April 23, 2023 16:29
Copy link
Copy Markdown
Collaborator

@euge-huge euge-huge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Работа и предыдущие засчитана!

Оставил комментарии по тем блокам, которые было бы неплохо поправить!
К остальному у меня вопросов нет!
Работает все что нужно, табы, адаптив есть (нужно поправить только немного), бургер меню, форма открывается, валидируется и отправляет значения.
За равернутую и начало crm тоже лайк!

Молодец, продолжай изучать, развиваться и познавать и welcome в мир IT!

Comment thread site/src/js/main.js

let user = {
name: name.value,
number: number.value,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вместо number должен быть phone, судя по серверу)

Comment thread site/src/js/main.js
const form = document.getElementById('modalWindow');
const statusMessage = document.createElement('div');

fetch('http://localhost:3002/api/staff')
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Форма отправляется на localhost:3001, а данные получаешь с localhost:3002, нужно выбрать тот, на котором открывается сервер)

Comment thread site/src/scss/_modal.scss

/* фон нашего модального окна */
.modalBackground {
display: flex;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь по умолчанию сделал бы
display: none;
Чтобы оно сразу не было открыто, как мы попадаем на страницу

Comment thread site/src/index.html
Comment on lines +216 to +221
<li class="prices__type">Стрижка и укладка</li>
<li class="prices__type">Косметология</li>
<li class="prices__type">Маникюр и педикюр</li>
<li class="prices__type">Макияж</li>
<li class="prices__type">Брови и ресницы</li>
<li class="prices__type">Массаж</li>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Табы ломают адаптив, нужно поправить
Uploading image.png…

Comment thread site/src/index.html
Comment on lines +62 to +66
<li class="navigation__item"><a class="navigation__link" href="#">Услуги и цены</a></li>
<li class="navigation__item"><a class="navigation__link" href="#">Мастера</a></li>
<li class="navigation__item"><a class="navigation__link" href="#">Наши работы</a></li>
<li class="navigation__item"><a class="navigation__link" href="#">О нас</a></li>
<li class="navigation__item"><a class="navigation__link" href="#">Контакты</a></li>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно сделать скролл к нужным элементам по клику на них, для этого в href="#" нужно указать id нужного элемента

Comment thread site/src/index.html
<section class="works">

<div class="container">
<h2 class="works__title">Наши работы</h2>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В данной секции не хватает прикрученного slick или другой карусели, чтобы прокручивать изображения, когда их станет больше:
image

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

Successfully merging this pull request may close these issues.

2 participants