Общий суд - покритикуйте работу, буду благодарен

[Nibaluiso]

Всем привет. Покритикуйте (конструктивно) вот эту мою работу плиз, хочется учесть и исправить как можно больше замечаний. http://work-master.000webhostapp.com/works/metal/index.html

[Nibaluiso]

Ну тогда хоть похвалите чтоли))

[Crossfield]

Качество написания кода и правильность использования элементов не смотрел, но сразу могу сказать, что мобильная версия выглядит не очень.

Edited December 13, 2016 by Crossfield

[Nibaluiso]

[BrusSENS]

В целом не плохо, но, лично я, удалил бы эти треугольники в меню. Ни к чему они. А так, в целом даже очень не плохо)

Edited December 29, 2016 by BrusSENS

[Nibaluiso]

10 часов назад, BrusSENS сказал:

В целом не плохо, но, лично я, удалил бы эти треугольники в меню. Ни к чему они. А так, в целом даже очень не плохо)

Дизайнер макет такой нарисовал, с треугольниками) Спасибо!

[Юрий Куренков]

Могу по дизайну раскритиковать.

Мобильная:

- мелкий текст

- не кликабельный номер телефона

- часть пунктов в хиро не читается из-за убого подобранного фона

-  центрирование пунктов в "основные направления компании" не читабельно

- копи-текст мелковат и межстрочный единица это грех

 

Таблет:

- то же, что и мобайл

- кое-где выравнивание поехало

 

Десктоп:

- то же, что и таблет

- стрелки под меню. Не понятно назначение. Надо говорить с дизайнером, что он/она имел в виду.

- кое-где не влазит текст

- общая сетка не прослеживается, отступы не соблюдены. 

- карта зумится на скролл

 

Прошу прощения за стилистику и пунктуацию, но бесплатные ревью в 2 часа ночи они такие =) Надеюсь помог.

[Nibaluiso]

В 10.01.2017 в 01:36, Юрий Куренков сказал:

Могу по дизайну раскритиковать.

Мобильная:

- мелкий текст

- не кликабельный номер телефона

- часть пунктов в хиро не читается из-за убого подобранного фона

-  центрирование пунктов в "основные направления компании" не читабельно

- копи-текст мелковат и межстрочный единица это грех

 

Таблет:

- то же, что и мобайл

- кое-где выравнивание поехало

 

Десктоп:

- то же, что и таблет

- стрелки под меню. Не понятно назначение. Надо говорить с дизайнером, что он/она имел в виду.

- кое-где не влазит текст

- общая сетка не прослеживается, отступы не соблюдены. 

- карта зумится на скролл

 

Прошу прощения за стилистику и пунктуацию, но бесплатные ревью в 2 часа ночи они такие =) Надеюсь помог.

Во! Спасибо за такую развернутую критику, учту в дальнейшем) Сам не додумался бы

[by chris]

В атрибуте lang код языка указывается только в ловеркейсе, т.е код региона убираем. В форме тоже проблемы с атрибутами, они не должны быть пустыми. Нужно понимать для чего каждое свойство предназначено, т.к у формы атрибут value не поддерживается. Кнопка подробнее - это не кнопка, а ссылка, поэтому изменяем элемент ей соответствующий. А вот в форме кнопка прикрепить файл имеет не правильный тип. Попробуй нажать на нее и браузер попытается отравить данные на сервер. По умолчанию любая кнопка имеет тип сабмит. Значит любой кнопке нужно указывать правильный тип самому, в нашем случае type="button". Я писал недавно в похожей теме про телефоны, им хватает интерактивности.

[BrusSENS]

15 часов назад, by chris сказал:

В атрибуте lang код языка указывается только в ловеркейсе, т.е код региона убираем.

Во первых: с чего вдруг?

Во вторых: Это не код региона, а расширение языка (диалект).

Edited January 17, 2017 by BrusSENS

[by chris]

12 часа назад, BrusSENS сказал:

Во первых: с чего вдруг?

Такие стандарты
https://tools.ietf.org/html/rfc5646#section-2.1.1

Цитата

[ISO639-1] recommends that language codes be written in lowercase ('mn' Mongolian).

Цитата

[ISO3166-1] recommends that country codes be capitalized ('MN' Mongolia).

 

12 часа назад, BrusSENS сказал:

Во вторых: Это не код региона, а расширение языка (диалект).

Все-таки это код региона. Расширение языка включает более подробную составную информацию

https://tools.ietf.org/html/rfc5646#section-2.2.4

https://tools.ietf.org/html/rfc5646#section-2.2.6

1.ru-2.RU
1. Код языка
2. Код региона

Т.е не lang="ru-RU", а lang="ru"

 

[BrusSENS]

2 часа назад, by chris сказал:

Такие стандарты

Ну стандарты - это понятно. Но как быть, если я использую GB диалект? Тут получается стандарт нас ужимает.

2 часа назад, by chris сказал:

Все-таки это код региона. Расширение языка включает более подробную составную информацию

Возможно неверно выразился, но регион, в своём понятии не совсем подходит. Да и фактически - это реально диалект) У Англии же нет региона United States) А вот американский диалект есть. Отсюда и en-US.

2 часа назад, by chris сказал:

Т.е не lang="ru-RU", а lang="ru"

ru-RU в принципе не уместно использовать, диалектов то больше, насколько я понимаю, нет. Для en и нескольких других - актуально, а ru - вообще не понятно зачем в стандарт внесли.

P.S.: лучше бы стандарты принимали там, где они реально нужны, ибо в данном случае можно и свободу девелоперу предоставить.

[web-tech]

Чем обосновано расположение в шапке логотипа, меню и телефона, используя абсолютное позиционирование, а не float? Тем более, что для сетки далее по коду уже используется бутстрап? Я так понимаю, чтобы только изменить порядок элементов. Но в чем смысл было в разметке меню ставить последним, чтобы потом абсолютным позиционированием его менять местами с телефоном? 

Edited January 18, 2017 by web-tech

[by chris]

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

У Англии же нет региона United States) А вот американский диалект есть. Отсюда и en-US.

https://tools.ietf.org/html/rfc5646#appendix-A

Цитата

sl-rozaj (Resian dialect of Slovenian)
sl-nedis (Nadiza dialect of Slovenian)

 

Цитата

Language-Region:
   de-DE (German for Germany)
   en-US (English as used in the United States)

Edited January 18, 2017 by by chris

[BrusSENS]

24 минуты назад, by chris сказал:

https://tools.ietf.org/html/rfc5646#appendix-A

У меня уже голова кругом) Как всё тяжело они придумали)