Оцените верстку kinoruletka

[Max22]

Макет: http://alpatriott.ru/works/ruletka.zip

То, что получилось:http://kinorulet.meximas.com/kinorulet/

js не знаю, поэтому понятия не имею как делать слайдер.

[npofopr]

Для начала:

1. В 1200 не влазит.

2. Текст нельзя выделить, т.е. его перекрывает слой.

 

А вообще вам бы почитать предыдущие темы, где этот же шаблон обсуждался. Более чем уверен, что идентичные ошибки.

[Max22]

В задании было написано, что все фиксировано.Ширина макета была 1260px, я и сделал 1260px. Текст выделяет везде. 
Поищу предыдущие темы, спасибо.

p.s. Картинку нельзя выделить ))

Edited November 11, 2013 by Max22

[advokatua]

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

Прохалявил с ушками на слайдере и т.д. (еще пара визуальных расхождений)

В коде основное то, что сразу увидел и не понравилось:

.light {background: url(img/grad.png) no-repeat;opacity: 0.2;height: 600px;position: relative;z-index: 1;margin-top: -8px;;}.block_center {  margin-top: -600px;}

Уберем блик света посередине и все улетит. Так не пойдет. 

Edited November 11, 2013 by advokatua

[Zverushka]

Почему текст не выделяется?

[npofopr]

p.s. Картинку нельзя выделить

при чем тут картинка?

Текст на уровне Спойлера, вообще не выделяется.  

[Max22]

 

p.s. Картинку нельзя выделить

при чем тут картинка?

Текст на уровне Спойлера, вообще не выделяется.  

 

Может я вас неправильно понимаю...

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

Прохалявил с ушками на слайдере и т.д. (еще пара визуальных расхождений)

В коде основное то, что сразу увидел и не понравилось:

.light {background: url(img/grad.png) no-repeat;opacity: 0.2;height: 600px;position: relative;z-index: 1;margin-top: -8px;;}.block_center {  margin-top: -600px;}

Уберем блик света посередине и все улетит. Так не пойдет. 

Буду переделывать, спс

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

[ryter]

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

[Max22]

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

 Спрайты- это маленькие картинки, которые нарезаем, а потом используем в background`е, да?  Что значит разделять блоки на логические части(поясните , пожалуйста, я новичек еще в плане верстки)?

[ryter]

 

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

 Спрайты- это маленькие картинки, которые нарезаем, а потом используем в background`е, да?  Что значит разделять блоки на логические части(поясните , пожалуйста, я новичек еще в плане верстки)?

 

Спрайты это все маленькие картинки объединенные в одну. У вас все блоки идут одним потоком попробуйте перекинуть любой блок в другое место и все развалится, разделяйте блоки отдельными враперами.

[Vin]

В задании было написано, что все фиксировано.Ширина макета была 1260px, я и сделал 1260px.

Имелось в виду без фона страницы, который тянется на всю страницу, конечно. А так ширина чуть больше 1050px.

[Max22]

 

В задании было написано, что все фиксировано.Ширина макета была 1260px, я и сделал 1260px.

Имелось в виду без фона страницы, который тянется на всю страницу, конечно. А так ширина чуть больше 1050px.

Хм, а как вы определили, что 1050px. Я в фотошопе линейкой меряю от края до края и делаю такую ширину. Или это для того, чтобы все пользователи страницу  без горизонтального скролла видели? Тогда лучше 1024px ширину задавать всегда?

[Vin]

Хм, а как вы определили, что 1050px. Я в фотошопе линейкой меряю от края до края и делаю такую ширину. Или это для того, чтобы все пользователи страницу без горизонтального скролла видели? Тогда лучше 1024px ширину задавать всегда?

 

По краям от контента поставил две направлящих и померил. Желательно, чтобы влезало в 1024 без скролла, но здесь дизайнер немного ошибся и сделал контент слишком широким. Если быть точным, то 1068px, но и не 1260 — будет только небольшой скролл на маленьких мониторах.