Jump to content

Great Rash

Expert
  • Posts

    7,974
  • Joined

  • Last visited

  • Days Won

    144

Everything posted by Great Rash

  1. Я же не послал человека, извините, куда подальше. Просто указал, что в данном виде этот блог выглядит плохо. А ведь он задумывался автором для кого-то. И автор лукавит когда пишет, что делала это только для себя, ибо зачем тогда выкладывать это в сеть? Да не просто выкладывать, а затем регистрироваться на форуме и начинать обсуждение того, что сделано. Я считаю, что если ты морально не готов к критике, то и незачем показывать свою работу. Сколько тут было таких странных посетителей, которые просили обсудить их вёрстку, дизайн, задумку, а когда им говорили, что они так себе или вообще шлак, то они обижались как дети малые. Вспомнить того же Вердера, который в ответ на любую критику чуть ли не матюками крыть начинал...
  2. Вам нужно сделать как минимум 2 улучшения. 1) Отвяжитесь от ID и привяжитесь к классам или data-атрибутам. Это позволит вам не писать кучу бессмысленных идентификаторов типа car2, car3 и т.п. 2) В качестве аргумента функции вам нужно использовать массив. А если уж вы хотите использовать текст, то вам нужно превратить его в массив (распарсить). Пример: var elementsToHide = 'car'; // будем искать элементы с классом .carfunction hide(clName) { var nodes = document.getElementsByTagName('img'), // находим все картинки length = nodes.length, // узнаём сколько всего у нас картинок на странице node = null; for (var i = 0; i < length; i++) { node = nodes[i]; // i-тая нода if (node.className.indexOf('clName') > -1) { // если в имени класса присутствует 'car', значит этот элемент нужно спрятать node.style.display = 'none'; } }}Этот пример подходит для старых браузеров. Если же вас интересуют только новые браузеры (на самом деле основная проблема только в ИЕ7 и ниже, даже в ИЕ8 приведённый ниже код будет работать с некоторыми оговорками), то можно смело использовать более просто и правильный подход: var elementsToHide = '.car'; // ищем элементы с CSS-селектором '.car' (обратить внимание на точку впереди)function hide(elements) { var nodes = document.querySelectorAll('img' + elements), // получаем все картинки на странице с классом car length = nodes.length; for (var i = 0; i < length; i++) { nodes[i].style.display = 'none'; // нет нужды отсеивать что-то (за нас это сделал метод .querySelectorAll() }}
  3. Мне казалось она сама просила критики когда написала "жду вопросы и предложения". А вам, Лика я хочу ответить, что зря вы на меня обиделись, те кто меня знает тут чуть подольше/получше не дадут соврать, что я иногда может и бываю резковат, но никогда не делаю это со зла.
  4. Причём тут это? Вы ваш блог для кого ведёте, для себя или для того чтоб людям нравилось?
  5. Видео ужасно. Если это первый дубль, то его стоит удалить прямо сейчас и записать по-новой. Если это не первый дубль, то вам не стоит этим заниматься вообще. Видно, что вы очень волнуетесь, проблемы с дикцией (видимо от того же волнения). В руках у вас там шпаргалка? Если да, то вам стоит её хотя бы систематизировать, ну а ещё лучше, заучить.
  6. Нет. Если только скриптом.
  7. У вас по сути табличные данные, почему вам не нравится таблица в данном случае - непонятно.
  8. 1) Нужно явно задать высоту родительскому блоку. 2) Внутреннему блоку указать высоту 100%.
  9. Тут нечего реализовывать, потому что тут нет сетки. То что дизайнер положил поверх этого дерьма колонки ещё не означает, что тут образовалась сетка.
  10. Это очень плохо на самом деле...
  11. А при чём тут PHP?
  12. http://codepen.io/GreatRash/pen/yYVLaw
  13. Тупо добавить бекграунд и отступ инпуту.
  14. #0lga, плюс за щелкунчика, а ещё вроде как единственная работа с ховером. Sensei, самый лучший по дизайну, а ещё самый чистый код (уж не знаю руками или TidyUp ), что тоже очень важно! Jack_V, не смог Йоду не плюсануть просто.
  15. Можно закинуть картинку на сторонний хостинг картинок и забрать оттуда ссылку. Только реальную ссылку из инспектора, а не ту, что выдаёт хостинг.
  16. Ну по картинке трудно судить в чём именно у вас проблемы. На первый взгляд всё правильно. Попробуйте сверстать меню отдельно на том же сервисе, что и я (codepen или jsfiddle)
  17. 1) У вас нарушена вложенность тегов. для начала исправьте этот момент. 2) Установите вертикальный отступ (margin) для <p>. 3) Для <div id="social"> нет смысла указывать автоматическую высоту и ширину, так же как и отступ снизу.
  18. Вообще я просто сперва идею цветового кодирования не понял Теперь когда я её понял, то могу сказать. что пофиг каким цветом там будет. Ну т.е. не пофиг конечно, но желтый не нужен в общем
  19. Вообще, судя по картинке, тут резина не предусмотрена, так что фикс. В таком случае, я бы ленточке задал фиксированную высоту, а внутрь накидал бы ссылки вместе с логотипом. Лого и ссылки я бы сделал инлайн-блоками, и до кучи логотипу задал бы отрицательный отступ (position: relative; top: -Npx;). В таком случае логотип вылез бы за пределы ленты, но при этом раздвигал бы ссылки, которые его окружают. Ну а инлайн-блоки - потому что их по центру выравнивать проще простого. Любой слайдер выводит контент, а не картинку. Контентом может быть всё, что угодно.
  20. 1. Сайт адаптивный, резиновый или фиксированный? 2. Возможно ли, что в меню будет больше 4-х ссылок? А слайдер совершенно обыкновенный, по-моему вообще любой плагин можно использовать.
×
×
  • 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