Jump to content

Первая вёрстка - как первый секс :)


iamchelovek
 Share

Recommended Posts

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

Фон хоть и быстро грузится, но можно было его еще оптимизировать, вырезать паттерн, он будет меньше размером чем сейчас фон

Комментирование кода конечно хорошо, но комменты типа "<!-- end heading -->", как по мне, весьма бесполезны

Используете id не по назначению. Для оформительских целей стоит использовать class, это гораздо удобнее. id лучше для скриптов использовать.

Сторонний шрифт можно было и прямо в css подключать.

reset.css - плохая практика, очень плохая практика

Link to comment
Share on other sites

А зачем сначала поломать то что нам дали добрые дяди\тети разработчики браузеров, а потом все равно это восстанавливать? Настройте что вам нужно и как вам нужно. Зачем это бесполезное сбрасывание?

Вот спрашивается зачем взяли и сломали списки? Ну вот что они вам плохого сделали? Печеньку украли?


ol, ul {
list-style: none;
}

Зачем поиздевались над таблицами?


table {
border-collapse: collapse;
border-spacing: 0;
}

Это вообще несуразица какая-то O_o


blockquote:before, blockquote:after,
q:before, q:after {
content: '';
content: none;
}

  • Like 1
Link to comment
Share on other sites

Ясно. Спасибо.

Если честно, то я вообще не смотрел в этот reset.css. Просто скопировал с сайта Мейера и вставил себе, так как прочитал на каком-то сайте рекомендацию вставлять такую штуку в начало таблицы стилей. Больше практиковать не буду.

Наверно лучше свой собственный reset сделать и вставлять по мере необходимости.

Link to comment
Share on other sites

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

Вот пример из твоей же работы:

В reset.css у тебя есть запись:


html, body, div, span, applet, object, iframe,
h1, h2, h3, h4, h5, h6, p, blockquote, pre,
a, abbr, acronym, address, big, cite, code,
del, dfn, em, img, ins, kbd, q, s, samp,
small, strike, strong, sub, sup, tt, var,
b, u, i, center,
dl, dt, dd, ol, ul, li,
fieldset, form, label, legend,
table, caption, tbody, tfoot, thead, tr, th, td,
article, aside, canvas, details, embed,
figure, figcaption, footer, header, hgroup,
menu, nav, output, ruby, section, summary,
time, mark, audio, video {
margin: 0;
padding: 0;
border: 0;
font-size: 100%;
font: inherit;
vertical-align: baseline;
}

Сейчас обратим внимание на заголовки h1-6, для них сброшен размер шрифта, отступы и т.д. А потом уже в основном файле стилей идет определение заголовков


h1 {
text-transform: uppercase;
font: 28pt Val;
color: #a1764b;
float: left;
width: 271px;
margin-top: 32px;
}

h3 {
font: 13pt Arial, sans-serif;
color: #777777;
margin: 8px 0;
}

Ну и тому подобное. Но ведь можно было это все сделать сразу и не заниматься ерундой :)

Установленный font-size: 100%; для всего подряд тоже нелепое решение, во-первых это свойство наследуется, достаточно его прописать к селектору html и оно будет применено ко всем вложенным элементам, кроме тех у кого по-умолчанию есть свои значения, например у заголовков, но ведь их мы отдельно настроим, так как нам нужно ;)

А вот еще:

h_1376487952_7399103_f76f86cb0c.png

Если добавить больше текста, или бОльшие картинки, или еще 100500 причин, получится бяка, т.к. блоки фиксированной высоты. В общем совсем не гибко. Это лучше было построить на какой-нибудь сетке, например, основанной на inline-block

Блок справа "фолоу за свиттером" можно было сделать на css, а не картинкой, и тоже гибким

Link to comment
Share on other sites

q:before, q:after {

content: '';

content: none;

}

Кстати, кто-нибудь может объяснить мне, хотя бы примерно, «что хотел сказать автор» этими таинственными строчками? Спасибо, уже сам нашел :)

/*

CSS 2; used to remove quotes in case "none" fails below.

*/

content: "";

/*

CSS 2.1; will remove quotes if supported, and override the above.

User-agents that don't understand "none" should ignore it, and

keep the above value. This is here for future compatibility,

though I'm not 100% convinced that it's a good idea...

*/

content: none;

ну и некрофилия, конечно... :)

  • Like 1
Link to comment
Share on other sites

А зачем сначала поломать то что нам дали добрые дяди\тети разработчики браузеров, а потом все равно это восстанавливать? Настройте что вам нужно и как вам нужно. Зачем это бесполезное сбрасывание?

Вот спрашивается зачем взяли и сломали списки? Ну вот что они вам плохого сделали? Печеньку украли?


ol, ul {
list-style: none;
}

Зачем поиздевались над таблицами?


table {
border-collapse: collapse;
border-spacing: 0;
}

Это вообще несуразица какая-то O_o


blockquote:before, blockquote:after,
q:before, q:after {
content: '';
content: none;
}

Не надо так, человек настроил если так можно сказать под себя reset, у меня точно также стоит, ну просто нелепо прописывать каждый раз list-style: none; потому что все дизайны давно не содержат списков по умолчанию, а когда верстаешь не факт что попадётся нужый стаил для списков, всё равно переделываешь)

Edited by Struggle
Link to comment
Share on other sites

Не знаю, нумерованные списки в контенте, по-моему, очень часто используются как есть. Плюс у меня устойчивое мнение, что если верстальщик случайно забыл застилизовать какой-то момент, то дефолтный браузерный стиль — меньшее зло, чем слипшаяся нечитаемая каша после reset-а. Хотя «на вкус на цвет», да и случаи разные бывают-с... :)

  • Like 1
Link to comment
Share on other sites

Не надо так, человек настроил если так можно сказать под себя reset, у меня точно также стоит, ну просто нелепо прописывать каждый раз list-style: none; потому что все дизайны давно не содержат списков по умолчанию, а когда верстаешь не факт что попадётся нужый стаил для списков, всё равно переделываешь)

Ну так настрой так как надо по дизайну, либо не трогай. Зачем ломать? Если это список, то он должен оставаться списком

Link to comment
Share on other sites

Не надо так, человек настроил если так можно сказать под себя reset, у меня точно также стоит, ну просто нелепо прописывать каждый раз list-style: none; потому что все дизайны давно не содержат списков по умолчанию, а когда верстаешь не факт что попадётся нужый стаил для списков, всё равно переделываешь)

Ну так настрой так как надо по дизайну, либо не трогай. Зачем ломать? Если это список, то он должен оставаться списком

Т.е ресет это плохо? А что насчет normalize.css ?

Link to comment
Share on other sites

Не надо так, человек настроил если так можно сказать под себя reset, у меня точно также стоит, ну просто нелепо прописывать каждый раз list-style: none; потому что все дизайны давно не содержат списков по умолчанию, а когда верстаешь не факт что попадётся нужый стаил для списков, всё равно переделываешь)

Ну так настрой так как надо по дизайну, либо не трогай. Зачем ломать? Если это список, то он должен оставаться списком

Т.е ресет это плохо? А что насчет normalize.css ?

Конечно плохо. А normalize - это как раз приблизительно то о чем я и говорю. Но опять же это попытка сделать какой-то универсальный дырко-затыкатель, который подключишь и все сразу станет так как нужно, но так не бывает.

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

Я лично ни reset'ом, ни normalize'ом не пользуюсь. Хотя когда был начинающим активно использовал reset, но потом понял, что это борода приносящая больше проблем чем пользы.

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

Link to comment
Share on other sites

Не надо так, человек настроил если так можно сказать под себя reset, у меня точно также стоит, ну просто нелепо прописывать каждый раз list-style: none; потому что все дизайны давно не содержат списков по умолчанию, а когда верстаешь не факт что попадётся нужый стаил для списков, всё равно переделываешь)

Ну так настрой так как надо по дизайну, либо не трогай. Зачем ломать? Если это список, то он должен оставаться списком

Т.е ресет это плохо? А что насчет normalize.css ?

Конечно плохо. А normalize - это как раз приблизительно то о чем я и говорю. Но опять же это попытка сделать какой-то универсальный дырко-затыкатель, который подключишь и все сразу станет так как нужно, но так не бывает.

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

Я лично ни reset'ом, ни normalize'ом не пользуюсь. Хотя когда был начинающим активно использовал reset, но потом понял, что это борода приносящая больше проблем чем пользы.

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

Идиотизм, просто понты корявые, разберем простой пример


h1 {
font-size: 2em;
margin: 0.67em 0;
}

собственно зачем здесь отступы ?

когда начнешь верстать, на макете другие отступы, другие единицы измерения, + ко всему для каждого h1 придется потом вычитать и прибавлять эти самые 0.67em, велосипед, когда можно обнулить все и знать что нигде ничего лишнего не прибавиться, и таких примеров кучу

Уважаю фреймворки, т.к их используют в больших проектах где дизайн создаётся раз навсегда, большой проект - больше усилий, там уже как-то придеться подгоняться под него, кода значительно больше становиться, но и качество тоже.

Edited by Struggle
Link to comment
Share on other sites

Ничего вычитать не нужно, потому, что есть такое понятие как схлопывание margin.

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

Link to comment
Share on other sites

Ничего вычитать не нужно, потому, что есть такое понятие как схлопывание margin.

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

Омг, про схлопывание знаю, но причем здесь оно выясню, по вашему все люди которые верстают и пользуются сбросами идиоты? что-то около 95%, элементарно даже на вашем примере, всё равно придется переопределять, точно также, в h1 заданы основные стили, для заголовка блока left например будут другие свойства, ну и как ? что?) велосипед...но и возможно я и не прав)

Link to comment
Share on other sites

я не сказал, что идиоты, не стоит перекручивать. Да и вообще про идиотизм, пока что говорите, только вы :) Я говорю о том, что волшебной пилюли нету.

всё равно придется переопределять, точно также, в h1 заданы основные стили, для заголовка блока

Так вот именно, но зачем это делать ДВА раза, если можно сделать ОДИН, вот что я пытаюсь донести.

Link to comment
Share on other sites

я не сказал, что идиоты, не стоит перекручивать. Да и вообще про идиотизм, пока что говорите, только вы :) Я говорю о том, что волшебной пилюли нету.

всё равно придется переопределять, точно также, в h1 заданы основные стили, для заголовка блока

Так вот именно, но зачем это делать ДВА раза, если можно сделать ОДИН, вот что я пытаюсь донести.

Разные браузеры по своему это делают, не так ли? Мне кажется лучше оталкиваться от нулей (пусть хоть и 2 раза), чем выщитывать и подгонять под все браузеры.

Хотя я новичок :) Быть может когда стану таким же как вы, тоже перестану заниматься этой ерундой. ^_^

Edited by ALEEX SWN
  • Like 1
Link to comment
Share on other sites

Может, пора обсуждение «за» и «против» ресета/нормалайза выделить в отдельную тему? Тема-то важная, а здесь затеряется со временем...

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

Edited by wwt
Link to comment
Share on other sites

Может, пора обсуждение «за» и «против» ресета/нормалайза выделить в отдельную тему? Тема-то важная, а здесь затеряется со временем...

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

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

Разные браузеры по своему это делают, не так ли? Мне кажется лучше оталкиваться от нулей (пусть хоть и 2 раза), чем выщитывать и подгонять под все браузеры. Хотя я новичок Быть может когда стану таким же как вы, тоже перестану заниматься этой ерундой.

Когда-то давно я мыслил абсолютно точно так же. Но потом я понял свои ошибки ;)

Link to comment
Share on other sites

Мне кажется лучше оталкиваться от нулей

Лучше отталкиваться от дизайна :). Где дизайнер недодизайнил — там, по идее, всем пофиг. Но если случилось непредвиденное, имхо, пользователю сайта лучше отталкиваться от чего-то осмысленного (даже если в его браузере оно «осмысливается» иначе, чем в соседнем*), нежели от нулей, с которыми текст слипается в кашу.

* кстати, по мере развития браузеров различия дефолтных стилей становятся всё меньше и меньше

Link to comment
Share on other sites

Вот, собственно.

http://freshiamchelovek.p.ht/

Код можно сократить, конечно, но я уже устал. Потом сделаю.

HTML валиден, CSS - нет. Это из-за градиента. Я воспользовался генератором градиентов ColorZilla, скопировал код, а там используется много невалидных префиксов. В остальном всё валидно.

И ещё проблема. Если уменьшать окно браузера, то файл media.css срабатывает, а если уменьшать разрешение, то media.css не работает. На локалке всё нормально, а на хостинге такие проблемы. Не знаю почему.

Ах да, я ещё синюю линию под HOME забыл поставить. Только сейчас вспомнил.

Мелочи доделаю.

Шаблон 9edee05422c1t.jpg

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