Оцените верстку :)

[Ghost86421]

Доброго дня.

Прошу оценить верстку и указать на ошибки.

http://mihayliack98.url.ph/naturetour/index.html

[WEB-best]

Неплохо, но вот так вот лучше не делать. Задайте это в стилях.

 

<div class="main_content" style="height: 947px;">

 

<aside class="right_sidebar" style="height: 947px;">

 

[Ghost86421]

 

Неплохо, но вот так вот лучше не делать. Задайте это в стилях.

 

<div class="main_content" style="height: 947px;">

 

<aside class="right_sidebar" style="height: 947px;">

 

Это от скрипта - equalHeights

[MichaelD]

Не знаю как у вас, а у меня на большом мониторе выглядит все так.

http://radikal.ru/fp/95e9b0da545448fa847deba23beacc72

 

если еще и уменьшить масштаб, то и right_sidebar ломается.

Edited April 4, 2014 by MichaelD

[Ghost86421]

Не знаю как у вас, а у меня на большом мониторе выглядит все так.

http://radikal.ru/fp/95e9b0da545448fa847deba23beacc72

 

если еще и уменьшить масштаб, то и right_sidebar ломается.

А почему блоки выскакивают и как это предотвратить? Помогите, пожалуйста

[MichaelD]

Мне не очень нравится такая многоуровневая вложенность элементов, но с точки зрения семантики, все верно)

 

Вы располагаете элементы в places_results с помощью inline-block, если располагать с помощью float ничего не съезжает. Не использовал inline-block'и так что в чем ошибка могу только догадываться).

[Ghost86421]

Мне не очень нравится такая многоуровневая вложенность элементов, но с точки зрения семантики, все верно)

 

Вы располагаете элементы в places_results с помощью inline-block, если располагать с помощью float ничего не съезжает. Не использовал inline-block'и так что в чем ошибка могу только догадываться).

А почему вам не нравится вложення блоков?

 

Я уже решил, что можно было бы сделать, добавить справа 4px и отнять 4px слева,

[abcde]

Как не профессионал говорю. Сайт норм

[akorshun]

http://akorshun.e3w.ru/Prechu/ вот и ссылка

не туда написал, извиняюсь

Edited April 21, 2014 by akorshun

[Kirill M]

Для красоты еще уберите стандартную стрелочку из селектов и сделайте вкладки рабочими.

[Ghost86421]

Для красоты еще уберите стандартную стрелочку из селектов и сделайте вкладки рабочими.

Что-то я не понимаю, какую именно стрелку?

[Kirill M]

 

Для красоты еще уберите стандартную стрелочку из селектов и сделайте вкладки рабочими.

Что-то я не понимаю, какую именно стрелку?

 

 

Кнопку которая раскрывает селект

[Aleksandr.L]

Вот так лучше не делать (верхний регистр):

 

 

Для этого есть text-transform: uppercase.

[WEB-best]

Вот так лучше не делать (верхний регистр):

 

 

Для этого есть text-transform: uppercase.

Какая разница, через какой регистр писать в HTML li a ?

Можно написать маленькими и прописать text-transform: uppercase

или написать большими.

[Kirill M]

Вот так лучше не делать (верхний регистр):

 

 

Для этого есть text-transform: uppercase.

 

А почему, очень интересно? СЕО?

[xzarxzes]

 

Вот так лучше не делать (верхний регистр):

 

 

Для этого есть text-transform: uppercase.

 

А почему, очень интересно? СЕО?

 

Здесь верхний регистр относиться к стилистическому оформлению, следовательно лучше указывать это в стилях, нужно отделять представление страницы от содержания. Кроме того если вам понадобится эта строка в скриптах, то вам придется приводить ее к нижнему регистру или писать ее с капсом, что не совсем удобно. Так что это очень - очень плохо и так делать не надо. И еще к примеру если у вас ссылка написана капсом и она указывает на какую то страницу, то логично предположить что имя файла с этой страницей также должно быть написано капсом, а так выходит что у вас ABOUT US указывает на about_us.html, да и вообще навигация генерируется обычно на сервере и никто ее не будет писать в верхнем регистре чтобы она подобающим образом смотрелась в документе, я так думаю. А так конечно если имеешь дело со статичной страницей то может показаться, что это совсем и не важно, да и даже проще, просто взять и прописать все заглавными буквами.

Edited April 22, 2014 by xzarxzes

[Ghost86421]

Вот так лучше не делать (верхний регистр):

 

 

Для этого есть text-transform: uppercase.

Упс, это понимаю, просто скопировал текст в psd, там текст в верхнем регистре.