Jump to content

Оцените верстку 2ух макетов.


advokatua
 Share

Recommended Posts

Я, можно сказать, начинающий верстальщик. Верстать начал только совсем недавно, но с CSS и HTML работаю довольно давно. За последние 2 месяца прочитал порядка 8 книг по дизайну и верстке. Понял, что это все ерунда и книги нужно читать только когда есть какой-то опыт. Тупое чтение книг, даже с выполнением всех заданий дает очень мало в отличии от реальной практики на собственных ошибках. Как говориться - "Если хочешь научиться плавать, надо плавать".

Так вот, за последние пару дней сделал 2 макета. Посмотрите (кому не лень) и тыкните носом в какие-нибудь нубские и очевидные ошибки кода, если таковые имеются. Макеты в основном лепились только на визуальное совпадение с html5 и css3, т.е. всякие мобильные разрешения не учитывались. Кроссбраузерность также только под все самое последнее, без мазохистких вставок всяких хаков под 'IE 1.0000005 PreBeta not stable edition Only for developers' и прочей ахинеи 8)

http://webdevelop.zz.mu/index.html - макет 1

http://webdevelop.zz.mu/master.css - главный css

http://webdevelop.zz.mu/paypaul.png - картинка макета

webdevelop.zz.mu/illustrate/index.html - макет 2

webdevelop.zz.mu/illustrate/master.css - главный css

http://webdevelop.zz.../illustrate.png - картинка макета

psd не заливаю, т.к. все-равно никто смотреть не станет.

Edited by advokatua
Link to comment
Share on other sites

1.

- а чего выпадающее меню то не сделали?

- логотип обычно тоже ссылкой делают, вроде)

2.

- при вращении круга нижний слой дергается.

- тест в блоках срезается, это так и надо? может точечки (...) делать? хотя это можно и в процессе программирования сделать.

круто бы еще если бы слайдеры привернули

Link to comment
Share on other sites

Привет.

До конца не получится оценить. Нет ТЗ. Что как должно быть. Требования к макету и браузерам. Под какие устройства смотреть? Андроиды, Ipad'ы, iphon'ы и т.д. Что там и как...

В общем, сейчас это, простите, сыроватая версия под desctop'ы. :)

Напишу только по разметке. Дизайн, всякие transition и прочие рюшечки по ховеру и т.д. пропустим.

Вы пишите, что делали всё под самое последнее, а почему тогда не используете html5 теги?

А-ля:

<header class="header">

</header>

<main class="main_content">

</main>

<footer class="footer">

</footer>

Отсутствует сетка. Поверьте, верстать по сетке гораздо удобнее. Плюс, я не говорю уже о тех, кто будет заниматься сайтом после вас. Поищите статьи по верстке по сетке, их полно в сети. А уж фреймворков с сетками вообще тысячи.

Только не нужно копировать! Сетку сделайте сами. Типа:

main-frame {

width:1057px;

margin: 0 auto;

display: block;

}

.col1 {

width: 296px;

display: inline;

float: left;

}

.col2, .col3, .col4 {

...

}

Сразу сможете откалибровать макет под разные разрешения.

И ещё подумайте над именованием классов. Это, конечно, кому как удобнее. Но если вы будете верстать что-то большое, то у вас всяких feature и feature_item будет миллион. :) Посмотрите в сторону "БЭМ", это если уж совсем мощно.

links:

http://html5doctor.com/

http://html5boilerplate.com/

http://ru.bem.info/method/

Link to comment
Share on other sites

Я, можно сказать, начинающий верстальщик. Верстать начал только совсем недавно, но с CSS и HTML работаю довольно давно. За последние 2 месяца прочитал порядка 8 книг по дизайну и верстке. Понял, что это все ерунда и книги нужно читать только когда есть какой-то опыт. Тупое чтение книг, даже с выполнением всех заданий дает очень мало в отличии от реальной практики на собственных ошибках. Как говориться - "Если хочешь научиться плавать, надо плавать".

Так вот, за последние пару дней сделал 2 макета. Посмотрите (кому не лень) и тыкните носом в какие-нибудь нубские и очевидные ошибки кода, если таковые имеются. Макеты в основном лепились только на визуальное совпадение с html5 и css3, т.е. всякие мобильные разрешения не учитывались. Кроссбраузерность также только под все самое последнее, без мазохистких вставок всяких хаков под 'IE 1.0000005 PreBeta not stable edition Only for developers' и прочей ахинеи 8)

http://webdevelop.zz.mu/index.html - макет 1

http://webdevelop.zz.mu/master.css - главный css

http://webdevelop.zz.mu/paypaul.png - картинка макета

webdevelop.zz.mu/illustrate/index.html - макет 2

webdevelop.zz.mu/illustrate/master.css - главный css

http://webdevelop.zz.../illustrate.png - картинка макета

psd не заливаю, т.к. все-равно никто смотреть не станет.

Любой сайт хорош в работе. Картинка на любителя. На первом сайте я бы остановился, второй пробросил бы.
Link to comment
Share on other sites

1.

- а чего выпадающее меню то не сделали?

- логотип обычно тоже ссылкой делают, вроде)

2.

- при вращении круга нижний слой дергается.

- тест в блоках срезается, это так и надо? может точечки (...) делать? хотя это можно и в процессе программирования сделать.

круто бы еще если бы слайдеры привернули

1.

- такой задачи не было

- забыл )

2.

- проверил в хром, фаер и ИЕ10, у меня все ок. Без дерганий. В ИЕ10 вообще круг плавнее всего крутится.

- это норма, по задаче.

(Это буквально стартовые пробы по верстке. Упор идет на CSS3 и HTML5, JS пока не подключаю)

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

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

Егкиф в первом посте было описание по задачам.

- По html5 тегам верно подмечено. Работаем

- Сетка. Во втором макете 24-колоночная сетка. Я ее слепил вручную и думал, что половину размеров и отступов действительно просто задам по размеченным колонкам и отступам. На деле же оказалось, что многие элементы не совпадают с размеченной сеткой. Ну например, есть блок шириной 270px, а в сетке колонка есть либо на 260px либо на 300px и вот что тут делать я пока не разобрался )

Пункты в меню, кстати, как в первом так и втором макете вообще сущие непонятки преподнес. Там выравнивание и не посередине и не по какому-либо краю. В итоге неясно, то ли дизайнер думает, что верстальщик сам догадается все выровнять, толи там есть какой-то хитрожопый алгоритм выравнивания )

Вот менюшка второго макета. С какой стороны его выровнять я так и не понял )

Untitled.jpg

Справа-слева отступы разнятся, единой ширины также не заметил, если разделить от края до края выходит ~78px. Пункт Сontacts влезет в такой столбик вообще без отступов по краям. Ну да ладно, это вообще не главная забота была .

ЗЫ. Всем спасибо за комменты.

Edited by advokatua
Link to comment
Share on other sites

Кстати, по поводу непоняток.

Общайтесь с дизайнером. Если какие-то блоки выезжают за границы или имеют отличный от сетки размер, то в большинстве случаев - это косяк дизайнера.

Если же это всё-таки не косяк, то такой блок можно вписать в ближайший подходящий по размерам со смещением.

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

Link to comment
Share on other sites

На картинке еще и видно, что красный блок (в шапке перед лого-текстом) не дотягивает буквально 1 пиксель до прилипания к сетке. И вот когда смотришь на это дело и таких мест минимум по 5 штук на макет, то невольно начинаешь задумываться:"Какого хрена диз рисовал сетку и делает эти 1-2пиксельные недоходы?!" )

Edited by advokatua
Link to comment
Share on other sites

для второй верстке

убери слеши в меню и сделай вот так


.main-nav li {
float: left;
font-size: 0.825em;
letter-spacing: 0.4px;
line-height: 62px;
text-align: center;
text-transform: uppercase;
}
li:after {
content: "/";
padding: 0 22px 0 5px;(+ - сам подгонишь)
}

Edited by cris7iano
  • Like 1
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

  • Similar Content

    • By zeiger2
      Здравствуйте! У меня стоит задача, что при наведении на блок li строка должна поменять цвет, в том числе и картинка. Я меняю картинку с помощью 
      background-image: none;     background: url(../img/check_icon_red.png) left no-repeat;   Но теперь картинка позицианируется не там где должна, её можно поставить на место только вручную, через -100px. Нужно поставить ровно туда, где она была. Должна быть в одном ряду с другими
    • By Mix9
      есть див с 5 img, при уменьшении экрана див выходит за него. Я добавил overflow: auto для этого div в надежде на то, что я смогу прокручивать фотки с помощью скроллбара, однако даже с ним почему-то я не вижу часть фоток которые вышли за границу. Что с этим можно сделать? класс video повторяется 5 раз, я тут оставил только 1 
      .content{ width: 90%; background-color: #333; } .video{ margin: 0px 4px 0px 4px; width: 310; display: flex; flex-direction:column; } .video_button_text{ margin-top: 10px; display: flex; flex-direction: row; font-size: 20px; color: white; } .video_text_div{ display: inline-block; width: 250px; } .video_text{ text-align: justify-all; margin: 0px; display: -webkit-box; -webkit-line-clamp: 2; -webkit-box-orient: vertical; overflow: hidden; } .slidan_videos{ margin: 0px 10px 0px 20px; overflow: auto; width: auto; margin-bottom: 50px; display: flex; flex-direction: row; justify-content: space-around; } <div class="content"> <div class = slidan_videos> <div class = video> <div> <a href = 'ссылка'><img class="img" src=""картинка"></a> </div> <div class = video_button_text> <div class = avatarka_div> <a href="ссылка" target="_blank"><img class = avatarka src="картинка"></a> </div> <div class = video_text_div> <p class = video_text><a href="ссылка">текст</a></p> </div> </div> </div>
    • By Марко
      Добрый день. Начинающий программист, столкнулся с проблемой. Селектор .class не работает должным образом. Несмотря на правильное, я надеюсь, описание, на web-странице не отображается ни одно изображение. С чем может быть связано? Заранее спасибо за помощь. 



    • By Kaido
      Использую готовый плагин для модальных окон(от MaxGraph). Проблема в том, что когда у меня открыто два модальных окна, для примера Форма + Политика конфендициальности, и мне нужно закрыть политику вместе с ней закрывается и другое модальное окно. В JS я не сильно разбираюсь(собственно из за этого и использую готовый плагин), можете помочь кто работал с этим плагином? Я примерно понимаю как он работает, но реализовать чтобы закрывалось только одно не получается.
       
        <div class="content"> <button class="modal-btn" data-path="first" data-animation="fadeInUp" data-speed="1500">Открыть окно 1</button> </div> <div class="modal"> <div class="modal__wrapp" data-target="first"> <div class="modal__content"> <button class="modal__close">Закрыть</button> модальное окно <button data-path="policy">Политика</button> </div> </div> <div class="modal__wrapp" data-target="policy"> <div class="modal__content"> <button class="modal__close">Закрыть</button> политика </div> </div> </div> .modal { --transition-time: 0.3s; position: fixed; left: 0; top: 0; right: 0; bottom: 0; z-index: 1000; cursor: pointer; overflow-y: auto; overflow-x: hidden; text-align: center; opacity: 0; visibility: hidden; transition: opacity var(--transition-time), visibility var(--transition-time); } .modal__wrapp { display: none; cursor: default; width: fit-content; height: fit-content; } .modal__content{ position: absolute; left: 500px; width: 500px; height: 500px; display: flex; color: white; flex-direction: column; text-align: left; background-color: #000; } .modal__content button{ width: 200px; height: 50px; margin: 50px 0; } .modal.is-open { opacity: 1; visibility: visible; transition: opacity var(--transition-time), visibility var(--transition-time); } .modal__wrapp.modal-open { display: flex; } .disable-scroll { position: relative; overflow: hidden; height: 100vh; position: fixed; left: 0; top: 0; width: 100%; } .fade { opacity: 0; transition: opacity var(--transition-time); } .fade.animate-open { opacity: 1; transition: opacity var(--transition-time); } .fadeInUp { opacity: 0; transform: translateY(vw(-100)); transition: opacity var(--transition-time), transform var(--transition-time); } .fadeInUp.animate-open { opacity: 1; transform: translateY(0); transition: opacity var(--transition-time), transform var(--transition-time); } .modal__wrapp[data-target="policy"] .modal__content{ left: 1050px; background-color: #000; opacity: .5; } class Modal { constructor(options) { let defaultOptions = { isOpen: () => {}, isClose: () => {}, } this.options = Object.assign(defaultOptions, options); this.modal = document.querySelector('.modal'); this.speed = false; this.animation = false; this.isOpen = false; this.modalContainer = false; this.previousActiveElement = false; this.fixBlocks = document.querySelectorAll('.fix-block'); this.focusElements = [ 'a[href]', 'input', 'button', 'select', 'textarea', '[tabindex]' ]; this.events(); } events() { if (this.modal) { document.addEventListener('click', function(e){ const clickedElement = e.target.closest('[data-path]'); if (clickedElement) { let target = clickedElement.dataset.path; let animation = clickedElement.dataset.animation; if (clickedElement.classList.contains('modal-close')) { this.close(); } let speed = clickedElement.dataset.speed; this.animation = animation ? animation : 'fade'; this.speed = speed ? parseInt(speed) : 300; this.modalContainer = document.querySelector(`[data-target="${target}"]`); this.open(); return; } if (e.target.closest('.modal__close')) { this.close(); return; } }.bind(this)); window.addEventListener('keydown', function(e) { if (e.keyCode == 27) { if (this.isOpen) { this.close(); } } if (e.keyCode == 9 && this.isOpen) { this.focusCatch(e); return; } }.bind(this)); this.modal.addEventListener('click', function(e) { if (!e.target.classList.contains('modal__wrapp') && !e.target.closest('.modal__wrapp') && this.isOpen) { this.close(); } }.bind(this)); } } open() { this.previousActiveElement = document.activeElement; this.modal.style.setProperty('--transition-time', `${this.speed / 1000}s`); this.modal.classList.add('is-open'); this.disableScroll(); this.modalContainer.classList.add('modal-open'); this.modalContainer.classList.add(this.animation); setTimeout(() => { this.options.isOpen(this); this.modalContainer.classList.add('animate-open'); this.isOpen = true; this.focusTrap(); }, this.speed); } close() { if (this.modalContainer) { this.modalContainer.classList.remove('animate-open'); this.modalContainer.classList.remove(this.animation); this.modal.classList.remove('is-open'); this.modalContainer.classList.remove('modal-open'); this.enableScroll(); this.options.isClose(this); this.isOpen = false; this.focusTrap(); } } focusCatch(e) { const focusable = this.modalContainer.querySelectorAll(this.focusElements); const focusArray = Array.prototype.slice.call(focusable); const focusedIndex = focusArray.indexOf(document.activeElement); if (e.shiftKey && focusedIndex === 0) { focusArray[focusArray.length - 1].focus(); e.preventDefault(); } if (!e.shiftKey && focusedIndex === focusArray.length - 1) { focusArray[0].focus(); e.preventDefault(); } } focusTrap() { const focusable = this.modalContainer.querySelectorAll(this.focusElements); if (this.isOpen) { focusable[0].focus(); } else { this.previousActiveElement.focus(); } } disableScroll() { let pagePosition = window.scrollY; this.lockPadding(); document.body.classList.add('disable-scroll'); document.body.dataset.position = pagePosition; document.body.style.top = -pagePosition + 'px'; } enableScroll() { let pagePosition = parseInt(document.body.dataset.position, 10); this.unlockPadding(); document.body.style.top = 'auto'; document.body.classList.remove('disable-scroll'); window.scroll({ top: pagePosition, left: 0 }); document.body.removeAttribute('data-position'); } lockPadding() { let paddingOffset = window.innerWidth - document.body.offsetWidth + 'px'; this.fixBlocks.forEach((el) => { el.style.paddingRight = paddingOffset; }); document.body.style.paddingRight = paddingOffset; } unlockPadding() { this.fixBlocks.forEach((el) => { el.style.paddingRight = '0px'; }); document.body.style.paddingRight = '0px'; } } const modal = new Modal({ isOpen: (modal) => { console.log(modal); console.log('opened'); }, isClose: () => { console.log('closed'); }, });  
    • By stasN1
      Мне нужен такой результат :

      Не понимаю почему background не применяется вокруг иконок:

       
      Сам код:
      Html:
      css:

      Html:
      <section class="finish"> <div class="container"> <h2 class="finish_header">По окончании обучения Вы сможете!</h2> <div class="finish_divider"></div> <div class="finish_wrapper"> <finish_item> <div class="finish_round"> <img src="/icons/finish/1 (1).png" alt="" class="finish_icon"> </div> <div class="finish_descr">Создать свой сайт или блог</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/2.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Создать свой сайт или блог</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/3.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Обеспечить ему медленный, но верный рост в ТОП</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/4.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Достигнуть стабильного прироста посетителей</div> </finish_item> <finish_item> <div class="finish_round"> <img src="/icons/finish/5.png" alt="" class="finish_icon"> </div> <div class="finish_descr">Достигнуть стабильного прироста посетителей</div> </finish_item> </div> </div> </section>  
      Css:
      .finish .finish_wrapper { margin-top: 41px; display: flex; justify-content: space-between; } .finish .finish_wrapper .finish_item { width: 204px; } .finish .finish_wrapper .finish_item .finish_round { width: 115px; height: 115px; background-color: #b4e2ff; border-radius: 8px; } .finish_descr { font-family: Roboto; font-size: 17px; line-height: 20px; font-weight: 300; color: #efefef; Спасибо!
       

      Уже решил, спасибо!
      У меня CSS селектор вида:
      .finish .finish_wrapper .finish_item .finish_round

      А должен быть:
      .finish .finish_wrapper finish_item .finish_round
      Т.е. без точки перед finish_item, так как это не класс, а элемент
×
×
  • 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