Jump to content

BusinessName.com — опять


Gaspode
 Share

Recommended Posts

у вас зазор между меню и облаками получился и при разных разрешениях дыры появляются

поиск с контентом вылазит за пределы body, при появлении горизонтальной прокрутки

думаю облака надо было фоном для .wrapper ставить

и, разве название сайта не должно быть кликабельным?

Edited by ceil100
Link to comment
Share on other sites

у вас зазор между меню и облаками получился и при разных расширениях дыры появляются

Может, не при разных разрешениях, а при масштабировании? Это да. Я как-то не знаю, как это пофиксить нормально.

думаю облака надо было фоном для .wrapper ставить

Вероятно, да. Но тогда ещё одну обёртку надо. Потому что фон с облаками шире, чем .wrapper.

и, разве название сайта не должно быть кликабельным?

Зачем? Это же главная страница. По-хорошему, так и ссылка Home в навигации кликаться не должна.

Link to comment
Share on other sites

Может, не при разных разрешениях, а при масштабировании?

ну да при масштабировании

.wrapper .site-header {

overflow: hidden;

padding: 48px 0 18px;

}

можно дырку над меню отрегулировать, а то не красиво как то

Зачем? Это же главная страница. По-хорошему, так и ссылка Home в навигации кликаться не должна

но с этой страницы будут делаться и остальные, поэтому вопрос спорный

по хорошему ссылки Home и, вовсе, не нужно, лого должно вести на главную

а так все ок, только с шириной разобраться

Link to comment
Share on other sites

1) Я бы не стал отводить для среднего блока вверху целый section.

2) Поиск можно было засунуть в aside

3) Лениво разбираться в стилях, но по-моему есть избыточность оберток. Например, вот здесь:


<div class="news-box">
<aside class="news">

4) Для изображений слева и справа не пойдет использование figure?

5) Ну и в html5 можно каждый самостоятельный блок может начинаться с заголовка h1

Link to comment
Share on other sites

но с этой страницы будут делаться и остальные, поэтому вопрос спорный

На других страницах вместо h1 будет ссылка с тем же классом. Всегда так делаю.

можно дырку над меню отрегулировать

Не понимаю. Каким образом?

1) Я бы не стал отводить для среднего блока вверху целый section.
2) Поиск можно было засунуть в aside

Да, согласен.

3) Лениво разбираться в стилях, но по-моему есть избыточность оберток. Например, вот здесь:

Тут я в курсе, но это дело такое: сегодня избыточно, завтра нет. Внешняя обёртка отвечает за положение на странице, внутренняя — за внешний вид. Может, чуток громоздко, зато модифицировать легче гораздо.

Хотя конкретный случай, указанный тобой, можно и упростить, пожалуй.

4) Для изображений слева и справа не пойдет использование figure?

Сомнительно как-то, если честно. Не похоже, чтобы текст служил пояснением к картинкам. Больше похоже на то, что картинки вставлены просто так любые подходящие, потому что сайт "про бизнес".

5) Ну и в html5 можно каждый самостоятельный блок может начинаться с заголовка h1

И тут в курсе, но у меня это диссонанс вызывает. Заголовки "по старинке" как-то привычнее.

Edited by Gaspode
Link to comment
Share on other sites

Тут я в курсе, но это дело такое: сегодня избыточно, завтра нет. Внешняя обёртка отвечает за положение на странице, внутренняя — за внешний вид. Может, чуток громоздко, зато модифицировать легче гораздо.

Хотя конкретный случай, указанный тобой, можно и упростить, пожалуй.

Тогда, наверное надо поменять местами aside и див. Т.е. сначала обозначить, что это сайдбар, а потом рулить дивами внутри.

Сомнительно как-то, если честно. Не похоже, чтобы текст служил пояснением к картинкам. Больше похоже на то, что картинки вставлены просто так любые подходящие, потому что сайт "про бизнес".

Да я тоже только предположил)

И тут в курсе, но у меня это диссонанс вызывает.

Тоже было, но справился :)

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

×
×
  • 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