Jump to content

Оцените мою первую верстку на заказа


Liadonne
 Share

Recommended Posts

Всем добрый день!

 

Получила первый заказ на коммерческую верстку и вот что у меня получилось http://www.kenguru-taxi.com.ua/

Заказчик верстку принял, но хочу услышать конструктивную критику.

Все страницы сайта верстала сама, js и jQuery для слайдера, заказа звонка и страницы вакансии делала сама.

Edited by Liadonne
Link to comment
Share on other sites

Использовать h6 для линии (section.Tarif h6) - не уверен, что так правильно)

 

Не забывайте про sup и sub) не всё же span'у отдавать.

 

Кенгуру с благотворительностью проще было просто картинкой вставить, и администрировать так потом проще.

 

Ну и скролл снизу, плохо. И когда снизу появляется скролл, то фоновая картинка сеточкой срезается.

Link to comment
Share on other sites

я бы в формах outline убрал 

копирайт делается спец. символами 

страницу  "услуги" можно было сделать списками dt и регистр слов,  надо трансформировать через text-transform:uppercase

  Edited by DivMan
Link to comment
Share on other sites

Использовать h6 для линии (section.Tarif h6) - не уверен, что так правильно)

 

Не забывайте про sup и sub) не всё же span'у отдавать.

 

Кенгуру с благотворительностью проще было просто картинкой вставить, и администрировать так потом проще.

 

Ну и скролл снизу, плохо. И когда снизу появляется скролл, то фоновая картинка сеточкой срезается.

1. Почему нельзя использовать h6, это связано с SEO? Лучше на span заменить?

2. Sup и sub ни разу не использовала, теперь буду знать )))

3. Понятно

4. На счет скролла я эту проблему решить так адекватно и не смогла, min-width не помогает. Может что-то посоветуешь?

Link to comment
Share on other sites

 

 

а сколько стоит такая вёрстка?

Я за неё получила мало всего 500грн. А что?

 

а сколько часов ушло на вёрстку?

 

Не часов, 5 дней. Это мой первый заказ поэтому долго, думаю потом быстрее пойдет.

Link to comment
Share on other sites

500грн за первую верстку мало? Не слабые аппетиты у нынешнего поколения.

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

 

Макет вообще неплохо бы тоже выкладывать, на проверку соответствия.

 

upd.

Проглядел сайт - вертска жестяк, все держится мягко говоря на добром слове, работа с padding-ом и центровкой также жесть. Деньги за работу буквально подарили.

 

Как добрый совет: заканчивайте с видео уроками и переходите на нормальные книги. Я такую верстку в портфолио себе не добавил бы даже под пытками. Например: просто допустите мысль, что клиент захочет удалить блок с языками (справа вверху), удалите его через инспектор и посмотрите, что будет с верхней частью.

Edited by advokatua
Link to comment
Share on other sites

500грн за первую верстку мало? Не слабые аппетиты у нынешнего поколения.

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

 

Макет вообще неплохо бы тоже выкладывать, на проверку соответствия.

 

upd.

Проглядел сайт - вертска жестяк, все держится мягко говоря на добром слове, работа с padding-ом и центровкой также жесть. Деньги за работу буквально подарили.

 

Как добрый совет: заканчивайте с видео уроками и переходите на нормальные книги. Я такую верстку в портфолио себе не добавил бы даже под пытками. Например: просто допустите мысль, что клиент захочет удалить блок с языками (справа вверху), удалите его через инспектор и посмотрите, что будет с верхней частью.

согласен, а ты попробуй потестить мою вёрстку http://allnet.16mb.com/ переставь правый сайдбар в лево и в оморок упадёшь)))))

Edited by DivMan
Link to comment
Share on other sites

DivMan, если бы тебе за такое 500 грн заплатили, то упал бы )  

Как для новичка, который изучает html,css не более 3-4 месяцев и лепит свой первый тестовый макет, вполне еще хорошо. 

Edited by advokatua
Link to comment
Share on other sites

500грн за первую верстку мало? Не слабые аппетиты у нынешнего поколения.

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

 

Макет вообще неплохо бы тоже выкладывать, на проверку соответствия.

 

upd.

Проглядел сайт - вертска жестяк, все держится мягко говоря на добром слове, работа с padding-ом и центровкой также жесть. Деньги за работу буквально подарили.

 

Как добрый совет: заканчивайте с видео уроками и переходите на нормальные книги. Я такую верстку в портфолио себе не добавил бы даже под пытками. Например: просто допустите мысль, что клиент захочет удалить блок с языками (справа вверху), удалите его через инспектор и посмотрите, что будет с верхней частью.

На счет цены я не в курсе сколько берет нормальный фрилансер за работу и посчитала что больше и как видно ошиблась. А работать готова и бесплатно только бы набить руку и начать версталь по-человечески.

 

Видеоуроками никогда особо не увлекалась читаю Мержевича  "Верстка веб-страницы" и Дэвид Макфарланд "Большая книга CSS".

 

Отцентрировать по-другому у меня не получилось к сожалению.

 

Спасибо за критику.

Link to comment
Share on other sites

согласен, а ты попробуй потестить мою вёрстку http://allnet.16mb.com/ переставь правый сайдбар в лево и в оморок упадёшь)))))

 

Оформить тегом img хэдер - дикость помоему)) Да и фон делать картинкой, где сплошной цвет без всяких сложных градиентов и фигур - тоже плохо.

 

 

 

Видеоуроками никогда особо не увлекалась читаю

Если любите читать, читайте Эрика Майера - Каскадные таблицы стилей. Сильная книга. Видеоуроки от Soraxa тоже отличные, спасибо этому форуму что подсказали их. Там воды 0, для полного новичка будет трудно понять, а Вы как видно уже не полный новичок )

Edited by CroaToa
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 Mondeus
      Доброго всем времени суток. Прошу помощи. Научите принципу изменения футера. Движок Xenforo. Версия 2.2.10. Стиль дефолтный. Что именно нужно в итоге на фото примере. Мой шаблон app.footer less имеет следующее значение. 
      .p-footer { .xf-publicFooter(); a { .xf-publicFooterLink(); } } .p-footer-inner { .m-pageWidth(); .m-pageInset(); padding-top: @xf-paddingMedium; padding-bottom: @xf-paddingLarge; } .p-footer-row { .m-clearFix(); margin-bottom: -@xf-paddingLarge; } .p-footer-row-main { float: left; margin-bottom: @xf-paddingLarge; } .p-footer-row-opposite { float: right; margin-bottom: @xf-paddingLarge; } .p-footer-linkList { .m-listPlain(); .m-clearFix(); > li { float: left; margin-right: .5em; &:last-child { margin-right: 0; } a { padding: 2px 4px; border-radius: @xf-borderRadiusSmall; &:hover { text-decoration: none; background-color: fade(@xf-publicFooterLink--color, 10%); } } } } .p-footer-rssLink { > span { position: relative; top: -1px; display: inline-block; width: 1.44em; height: 1.44em; line-height: 1.44em; text-align: center; font-size: .8em; background-color: #4682B4; border-radius: 2px; } .fa-rss { color: white; } } .p-footer-copyright { margin-top: @xf-elementSpacer; text-align: center; font-size: @xf-fontSizeSmallest; } .p-footer-debug { margin-top: @xf-paddingLarge; text-align: right; font-size: @xf-fontSizeSmallest; .pairs > dt { color: inherit; } } @media (max-width: @xf-responsiveMedium) { .p-footer-row-main, .p-footer-row-opposite { float: none; } .p-footer-copyright { text-align: left; padding: 0 4px; // aligns with other links } }  

    • 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'); }, });  
×
×
  • 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