dom1n1k

Всё просто: если этот отступ не сделать, пространство между элементами будет выглядеть непропорционально большим в сравнении с краями - там ведь соседствуют два паддинга, пространство визуально удваивается.
Другое решение - разделить элементы списка линиями. Но сейчас это немодно (к сожалению), всем подавай чисто и пусто воздушно.

Не так, логичнее вешать модификатор на элемент item, а не link - потому что линк у своего родителя один и ничего не знает о том, что происходит снаружи.

Хотя мне кажется, что это тот редкий случай, когда даже с бэмом вполне уместно использовать селекторы first-child/last-child. Ну потому что не может такой модификатор объявиться где-то в средине и более одного раза.

Только что попробовал в консоли

> $('body').css('background-color')
> "rgba(255, 255, 0, 0.25)"

Ответ "нет" в обоих случаях.
Единицы вьюпорта бывают очень полезны в отдельных случаях, но переводить на них всё целиком и особенно шрифты бессмысленно.
Вариант с font-size: calc(vw + px) [гуглить по словам "css шлюзы", если не знаете что это такое] - более-менее приемлем, но тоже не универсален на 100%.

Весь Бутстрап вы не то что за 2 часа, а даже за 2 недели не напишете. Точнее, напишете черновое нечто, а потом еще неизвестно сколько времени баги ловить будете. И где-то через месяцок-другой дебага поймете, что пора делать глобальный рефакторинг :)

Хотя упоминание гридов заставляет заподозрить, что вам нужна не вся библиотека, а только сетка? Если так, то да, можно и самому написать все эти col-1 и т.д.

По поводу кастомизации - Бутстрап ей поддается хорошо, только если ваши изменения укладываются в рамки переопределения переменных: https://github.com/twbs/bootstrap/blob/master/scss...
И очень плохо, если что-то сложнее/глубже (в силу архитектуры и исторических причин).

В то же самое меню?
home
...
accessories
----
other menu items

Но вообще это не должно быть проблемой верстальщика. Дизайнер, менеджер, заказчик - аж вот сколько народу есть, кому скорее это стоит поручить.

В целом хорошо. Конечно, есть шероховатости, но ничего критичного.

1. Ширина блоков с мороженым ограничивается через max-width, отчего при некоторых размерах окна они слипаются. Нужно сделать нормальные отступы между ними.

2. В блоке .features я не понимаю, зачем нужно иконку выносить в before и вешать на модификатор? Ну точнее, понимаю - лаконичность кода, бла-бла - но считаю это плохой практикой. Иконка должна быть отдельным элементом, это очевиднее и гибче.

3. Название мороженного это ссылка (логично), но при ховере на блок подчеркивание пропадает почему-то.

4. Не нужно использовать слишком общие классы типа .header или .modal. В почти любом сайте у вас будет несколько хедеров и так далее. Нужно конретизировать, даже если прямо сейчас это вроде бы пока не нужно: page-header, modal-header, section-header, catalog-header и т.п.

5. Телефон некликабельный. И даже если бы он был кликабельный, все равно не получилось бы нажать, потому что слайдер его перекрывает.

6. В слайдере перелистывание только по "биатлону"? Для мобилы это мелко, нужен свайп или боковые кнопки.

7. Я сделал бы кнопки тегом [button], а не [a]

8. Ну и для кнопок имеет смысл отключать возможность выделения текста.

Это то что я нашел за несколько минут.

Найти работу однозначно можно, потому что качество результата уже сейчас выше, чем у большинства людей, которые называют себя верстальщиками. Самое главное - видно, что автор обладает аккуратностью и вниманием к деталям. Это просто бич огромной части разрабов! Всякие scss и gulp-ы они значит изучили, а разницу между 10 и 15 пикселями не видят...

Я не знаю, куда вы смотрели в интернете, потому что это гуглится в одно касание.
Хотя бы википедия: https://en.wikipedia.org/wiki/Methods_of_computing...