Jump to content

Покритикуйте верстку, пожалуйста.


eckimoc
 Share

Recommended Posts

Здравствуйте. Пожалуйста, покритикуйте верстку сайта http://www.training.od.ua. Очень интересует семантика кода. Можете написать, сколько, приблизитнльно, времени должно уходить на верстку такого сайта?

Link to comment
Share on other sites

  On 9/2/2014 at 10:28 AM, Galaxy said:
айпад, хром так же

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

чтобы не было вот такого PXo5APg.jpg

Link to comment
Share on other sites

Спасибо, за ваши ответы.

 

 

  On 9/2/2014 at 10:28 AM, 'Galaxy said:

 

 

Сегодня целый день пробовал понять, почему это происходит, причем, только в Mac OS. Блок не выходит за пределы родительского елемента. Был бы благодарен, если бы помогли с этой проблемой или направили в какую сторону гуглить. Вот сss этого блока 

nav {	margin-top: 10.1rem;	ul {		border-top: .5rem solid #777;		list-style-type: none;		padding: 0;	}	li {		position: relative;		display: inline-block;		font-size: 1.6rem;		font-weight: bold;		margin-left: 3.6rem;		margin-right: 1.3rem;		padding-top: 2.7rem;		a {			padding-top: 2.7rem;			text-decoration: none;			color: #555;		}	}	li:nth-of-type(8) a:after {		content: '';		display: block;		position: absolute;		background: url(/images/sprites.png) no-repeat -11.6rem -3.6rem;		background-color: #fbfbfb;		width: 3.8rem;		height: 2.9rem;		top: -1.9rem;		left: -.3rem;	}}
  On 9/2/2014 at 10:57 AM, 'ALEEX SWN said:

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

 

 

Исправил этот косяк с помощью text-overflow. Высота установлена у слайдера. Пробовал ее убирать, но из-за того, что каждый слайд имеет абсолютное позиционирование, слайдер схлопывается. Убрать абсолютное позиционирование со слайдов не могу, так как нужен эффект перелистывания слайдов. Не знаю, как еще можно решать такие моменты? Подскажите, стоит-ли заморачиваться со слайдерами на css или лучше делать на jquey.  

 

 

  On 9/2/2014 at 1:39 PM, 'Get to heaven said:

13 варна.

 

 

Все предупреждения связаны с тем, что не указаны заголовки у article. Как думаете, рационально-ли было использовать article и section в данном контексте?

 

 

 

  On 9/2/2014 at 1:50 PM, 'Galaxy said:

Чем обусловлено использование rem? <div class="bottom-line"></div> === <hr>

 

 

Действительно ступил, нужно было использовать <hr>. Rem использую, когда не нужна поддержка IE8. Как по мне, так проще чем em, всегда знаешь, что 1rem = 10 px.

 

Был бы благодарен если бы подсказали почему, когда я начинаю играться с масштабом, местами исчезают границы и в webkit браузерах при очень маленьком масштабе начинают скакать картинки?

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