Оцените мой не первый сайт

[WebNewbe]

Здравствуйте. Оцените пожалуйста данную верстку. Судить строго! 
https://alex2033.github.io/
Использовал: сетку с поддержкой флексов (Flexbox Grid)
На данный момент пока самый сложный макет, который верстал.
Реализованы:
- слайдеры

- плавные переходы по ссылкам на якори

- адаптивное меню Toggle(специально верхнюю часть хедера делал через флоаты, т.к. с флексами Toggle несколько конфликтует)
- табы

- scroll up

- всплывающие окна

и прочее
 

[andrey7287]

Transition надо на исходное состояние вешать а не на ховер, чтоб вот такой фигни как у тебя сейчас не было.

[WebNewbe]

19 минут назад, andrey7287 сказал:

Transition надо на исходное состояние вешать а не на ховер, чтоб вот такой фигни как у тебя сейчас не было.

Уточните, о какой секции идет речь?

Edited September 19, 2017 by WebNewbe

[DevChristmas]

7 часов назад, WebNewbe сказал:

Уточните, о какой секции идет речь?

Вот наглядный пример : 

 

При переносе свойства "transition" с состояния "hover" в дефолтный стиль ссылки  , плавность не будет пропадать при отведении мышки с элемента меню.

Edited September 20, 2017 by DevChristmas

[WebNewbe]

21 час назад, andrey7287 сказал:

Transition надо на исходное состояние вешать а не на ховер, чтоб вот такой фигни как у тебя сейчас не было.

 

13 часа назад, DevChristmas сказал:

Вот наглядный пример : 

 

При переносе свойства "transition" с состояния "hover" в дефолтный стиль ссылки  , плавность не будет пропадать при отведении мышки с элемента меню.

свойство transition был перемещен с ховера на сами элементы для всех элементов. Какие еще есть замечания?

[DevChristmas]

Да это было не замечание , я просто поддержал слова Андрея.

В 19.09.2017 в 22:45, andrey7287 сказал:

Transition надо на исходное состояние вешать а не на ховер, чтоб вот такой фигни как у тебя сейчас не было.

Transition и на hover иногда ставится.

Edited September 20, 2017 by DevChristmas

[Tilonorrinco]

Почему-то вспомнилась серия роликов "Джедай верстки" с канала webdesignmaster. Стиль верстки чем-то похож. Вот что привлекло внимание: 

1) Путаница с методологией именования классов. Где-то БЭМ (ссылки в меню), где-то без БЭМа. Нужно выбрать какую-то одну методологию и придерживаться только ее

2) Смешивается оформление и содержание. html код можно сократить в несколько раз:

- Для декоративной линии под заголовком не нужно делать отдельный div в разметке.

- Пять вложенных элементов в разметке для стилизации декоративной иконки с якорем -- это перебор.

- Декоративные линии с красной точкой посередине под заголовком, которые верстаются как div с тремя span'ами, один из которых содержит инлайн-svg (красную точку) -- это слишком сложно. 

- Декоративные треугольники через svg -- это слишком сложно. Размещение неконтентного декоративного элемента (красный треугольник в блоге) по вертикальному центру через align-self + position:absolute -- это весьма необычное сочетание, которое, вдобавок, не работает в IE11. Достаточно абсолютного позиционирования и одного псевдоэлемента (треугольники делаются через border).  

3) В IE11 появляется горизонтальный скролл из-за svg-элемента в разделе team, который растягивает контейнер в ширину. Нужно что-нибудь типа oveflow:hidden на родительском элементе.

4) На ширине 992 пикселя контейнер выходит за пределы блока-родителя (section) в результате чего образуется уродливая белая полоса справа.

Edited September 22, 2017 by Tilonorrinco

[Hasiev]

Как я понял,  якорь в хедере, это якорь.