Jump to content

Верстка, Мой первый code review


Ilya Kashitsyn
 Share

Recommended Posts

Здравствуйте! Это мой первый код ревью. Знаю он далек от идеала, но Ваша помощь очень будет нужна.

Сайт https://ilyakash.github.io/Blitz/index.html

Шаблон https://github.com/ilyakash/ilyakash.github.io/tree/master/Blitz/Template%20Blitz

1. Проблема в html верстке. В блок <header> проваливается нижний блок, поэтому постарил <br>.

ezgif.com-resize.gif.7da95201eb6cc16c28f412fa61d23b9b.gif

2. Как сделать на странице Project иконку search при наведении по центру?  И правильно ли я вообще это сделал, может есть другие варианты? Может можно было как то отобразить иконку search не через html, а через оформление в css?

search.jpg.545e894947f04f1941f92ef33b1673d5.jpg  637.png.b3eab13295cb90178d710afca0bc2045.png

3. Еще практически на всех страницах footer проваливается в верхний блог. сделал я это через кОстыли. Знаю, что так делать не зя. Похоже я в Html разметке накосячил?

footer-crutches.png.27f846dccbab03a4c0c12239a03a38e6.png

4. Еще у меня образовалась разница/попрыгушка в заголовке на странице About.

about2.gif.e979f2f80ff8a0eb18d2949fa8ca0661.gif

———————————————

Выслушаю любую полезную критику по улучшению написания кода. Спасибо.

 

Link to comment
Share on other sites

Сразу видно в чем проблема, это я про header. Вы используете float'ы, но при этом родителям не прописываете clearfix.
Зайдите с браузера в режим разработчика и посмотрите, что у вашего header высота 0.

<br> выкидывайте из кода вообще, я так понимаю вы начинающий верстальщик, и пока что легче отказаться насовсем от br, потом как разберетесь с ним, можете юзать. Он на самом деле довольно редко, когда нужен.

Как легко центрировать элемент по горизонтали и вертикали в 2018? 
Смотрите, юзайте flexbox.
 

родитель {
  display: flex;
  align-items: center;
  justify-content: center;
}

По поводу иконки, саму иконку search можно смело брать с fontawesome. Можно и запариться и нарисовать самому, но зачем?
Окружающий его квадрат с закругленными бордерами пишем сами на css. 
Далее, футер. Никогда не называйте классы так, как называете их вы в этой работе. Почитайте на тему наименование элементов страницы, это несложно. Footer с названием margin-top это жесть:D
Не знаю в чем у вас тут проблема, я поставил пару clearfix на header .container и navbar, послего чего перешел к подвалу страницы, убрал оттуда этот ужасный margin и все работало как надо.

Edited by WebNewbe
Link to comment
Share on other sites

Если это первая верстка, то хорошо. У меня было хуже.

Замечания

1) Задавайте размеры, а от этого проблемы:

https://prnt.sc/iy5meu

 https://prnt.sc/iy5nht

2) Че-то ругается  https://validator.w3.org/nu/?doc=https%3A%2F%2Filyakash.github.io%2FBlitz%2Findex.html

3) слайдер в хедер, ну не знаю, я бы не стал.

 

Рекомендации, кратко:

Прочитайте про БЭМ.

Учитесь делать адаптив.

Переходите на препроцессор html и css (pug и scss, например)

Используйте сборщики.

Edited by Schamil74
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

  • Similar Content

    • By Andy_Code
      Здравствуйте! Проблема заключается в следующем: есть стартовый шаблон от "Фрилансера по жизни" версии 2021 года, открываю в VSCode, ввожу в терминале npm i затем gulp, все срабатывает как и должно, создается папка с результатом, начинаю прописывать стили для header, задаю ограничивающий контейнер и при проверке работоспособности данного контейнера выясняется, что стили, прописанные мной перекрывают стили медиа-запросов, прописанных в шаблоне и по итогу адаптив не работает, получается что итоговый css собирается в не правильном порядке! Подскажите, пожалуйста, как подобную проблему можно исправить?)
    • By vladdvin
      Верстаю макет с Фигмы - https://www.figma.com/file/2rdy1qjBTl7D6BAC8S4qlf/Webovio?node-id=0%3A1. Наверстал
      CSS
      В результате отображается следующая картинка

      в режиме инструментов разработчика. А вне режима вообще отображает

      Подскажите как добиться отображения текста как на макете.
    • By RaiderCoder
      Здравствуйте , сделал макет сайта для интернет магазина . Но столкнулся с проблемой при верстке кнопок на главной странице и не нашел информации как можно уместить текст + несколько картинок в одну кнопку. Знает кто решение и реалезуемо оно вообще ?

    • By dissamen2
      Здравствуйте, столкнулся с такой проблемой, что в десктопной версии сайта все работает нормально, однако при переходе на мобильную, блок становится пустым и занимает гораздо меньше места, в чем проблема?
    • By Hinn
      Всем приветик, начинающий фрилансер. Ищу первые заказы в данной сфере. Занимаюсь:
      • На малом уровне версткой;
      • Хорошо разбираюсь в таких CMS: как Wordpress и Tilta - легко и быстро разработаю одно страничные, многостраничные сайты, интернет-магазины (могу как дорабатывать, так и создания с нуля, так же занимаюсь переносом одного сервера на другой);
      • Увлекаюсь разработкой логотипов (логотипы с нуля, доработка логотипа, рисование логотипа по эскизу).
      Готов заниматься работой за любую плату. Так же присутствует портфолио, но за частую людям интересно когда о тебе есть отзывы. Поэтому пришел сюда за практическим опытом, и поиском постоянной работы.
      Использую:
      - Верстка: (HTML5/CSS3), верстаю из Figma, Photoshop, illustrator.
      - Разработка сайта: (CMS WordPress, Tilta).
      - Разработка логотипа: (Photoshop, illustrator)
      Если не знаю чего-то необходимого для Вашего проекта\работы -- обязательно разберусь (все-таки начинающий фрилансер).

      Если вы разработчик и вам некому спихнуть рутинную работу - я с радостью возьму её на себя;

      телеграмм: HinnWork;
      почта: balistic.baty@gmail.com

      Открыт для любых предложений. 
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy