Zverushka Posted July 24, 2013 Report Share Posted July 24, 2013 vtest.uphero.com/wp/psd шаблон http://vtest.uphero....psdtemplate.psdЕго скриншот http://vtest.uphero....psdtemplate.jpg Quote Link to comment Share on other sites More sharing options...
saiidr Posted July 24, 2013 Report Share Posted July 24, 2013 (edited) Вроде вёрстка хорошая =) У меня вроде багов не было...Заметил только,что на кнопке (Submit) в низу при наведении,предоставляет вставить курсор. Можно было бы сделать cursor: pointer;,что было понятно,что ты нажимаешь на данную кнопку =) Edited July 24, 2013 by saiidr Quote Link to comment Share on other sites More sharing options...
T001 Posted July 26, 2013 Report Share Posted July 26, 2013 vtest.uphero.com/wp/psd шаблон http://vtest.uphero....psdtemplate.psdЕго скриншот http://vtest.uphero....psdtemplate.jpgможно поинтересоваться, сколько у Вас уходит времени на верстку данного шаблона ? Quote Link to comment Share on other sites More sharing options...
Zverushka Posted July 26, 2013 Author Report Share Posted July 26, 2013 У меня ушло полтора полных дня (день 8+ часов), особенно много времени потеряла на поиске слайдера и подделки под него. Quote Link to comment Share on other sites More sharing options...
SelenIT Posted July 26, 2013 Report Share Posted July 26, 2013 Можно было бы сделать cursor: pointer;,что было понятно,что ты нажимаешь на данную кнопку =) Лучше всё-таки реализовать для нее полноценный :hover. Показывать кликабельность одной сменой курсора... ну как-то несолидно, имхо. Всё-таки изначально и по стандарту pointer для ссылок и только для них. Да и юзабилити-эксперты призывают не полагаться на курсор мыши как «индикатор кликабельности»:Users should not be required to use the mouse pointer to discover clickable areas of images.Пользователи не должны нуждаться в курсоре мыши, чтобы обнаружить кликабельные областиThere is a visible change when the mouse points at something clickable (excluding cursor changes).Присутствует видимое изменение, когда мышь указывает на что-то кликабельное (помимо изменения курсора). Quote Link to comment Share on other sites More sharing options...
Vulner Posted July 26, 2013 Report Share Posted July 26, 2013 Думаю можно было бы объединить<link href="css/reset.css" rel="stylesheet"><link href="css/style.css" rel="stylesheet">в один файл. Одним запросом меньше Создавать для тени отдельные элементы<div class="footer-shadow-top"> </div><div class="footer-shadow-bottom"> </div>не есть хорошо. Лучше их сделать через footer:after, footer:before. Кстати сам футер неплохо было бы прижать к низу И по поводу кнопки DOWNLOAD NOW. Наверное лучше чтобы это была не картинка. А блочок с заливкой и текстом. Quote Link to comment Share on other sites More sharing options...
Zverushka Posted July 26, 2013 Author Report Share Posted July 26, 2013 @Vulner,А для чего прижимать футер к низу страницы? Какой в этом смысл, ведь он все равно всегда будет внизу же.?А по поводу бефо и афте в футере. Дело в том, что бэкграунд по задумке белый и при вставке теней через псевдо-элементы полупрозрачные тени оказались бы на белом. Или пришлось бы внутрь футера вставлять еще один див с белым фоном... Quote Link to comment Share on other sites More sharing options...
SelenIT Posted July 26, 2013 Report Share Posted July 26, 2013 и при вставке теней через псевдо-элементы полупрозрачные тени оказались бы на белом Но ведь можно задать для них и background-color, и background-image? Quote Link to comment Share on other sites More sharing options...
Zverushka Posted July 26, 2013 Author Report Share Posted July 26, 2013 ну, background color должен быть прозрачным, а под прозрачным окажется белый цвет футера, а background-image и есть сама тень. Quote Link to comment Share on other sites More sharing options...
SelenIT Posted July 26, 2013 Report Share Posted July 26, 2013 background color должен быть прозрачным Кому должен? Кто запретит ему слиться со светло-серым основным фоном, например? Эх, жалко, что :after от .wrap уже занят, на мой взгляд, эта тень в него напрашивается... Quote Link to comment Share on other sites More sharing options...
Zverushka Posted July 27, 2013 Author Report Share Posted July 27, 2013 На случай, если захочется другой background у страницы сделать, чтобы не пришлось править еще другие элементы, чтобы сайт был "масштабируемым" изначально. Quote Link to comment Share on other sites More sharing options...
SelenIT Posted July 27, 2013 Report Share Posted July 27, 2013 В принципе, логично. Тем более аргумент за то, чтобы эта тень не была привязана к белому футеру. Может, всё-таки ее на .wrap:after повесить, а боковые тенюшки к .top-у прицепить, с которым они и визуально связаны? Quote Link to comment Share on other sites More sharing options...
Vulner Posted July 27, 2013 Report Share Posted July 27, 2013 А для чего прижимать футер к низу страницы? Какой в этом смысл, ведь он все равно всегда будет внизу же.? В том виде, в каком оно все сейчас, так оно и выходит. Но исходя из того, чтобы сайт был "масштабируемым" изначально. то нужно его прижать. Ведь если убрать контент, то где окажется футер? Верно, не в низу страницы, где ему и место, а где-то по серединке. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
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.