Задать вопрос

frozifonte

Лайки

  • Покритикуете верстку?

    @artuh_a
    - Прелоадер или совсем уберите или сделайте его исчезновение более плавным, у вас статичная страничка и это мерцание как-то не очень смотрится;
    - названия картинкам лучше давать латиницей:
    5efb8a1033c3e982374636.png
    - там где белый фон у картинок лучше сделать его прозрачным, там где нет прозрачности лучше вместо png использовать jpg. На таких маленьких страницах не важно, а в будущем может пригодиться для оптимизации;
    - добавить ховер эффекты для пунктов меню;
    - убрать ссылку с логотипа, тут итак одностраничник.
    Ответ написан
    Комментировать
    Комментировать

  • Покритикуете верстку?

    @Qualiant
    Не буду ничего говорить по реализации, так как не шарю во фронте. Скажу как пользователь.

    "Закажите сейчас по акции" - в этом блоке отступы между абзацами бы побольше.

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

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

    Есть понятие о максимально комфортной ширине контента. Чтобы его можно было читать, не мотая головой. Я бы ограничил толщину контента второго, третьего и четвёртого блоков до 1200 пикселей.
    Ответ написан
    7 комментариев
    7 комментариев

  • Покритикуете верстку?

    @Allan11
    Фронтенд
    в хедере 1 и 3 ссылки никуда не ведут.
    сайт заканчивается странно, какого-нибудь футера не хватает или отступа хотя бы.
    попап с ума сходит если много раз нажать на кнопку при пустых инпутах
    Ответ написан
    Комментировать
    Комментировать

  • Покритикуете верстку?

    @UPSA
    anykey. Я не программист, я просто ленивый.
    1. В других местах тоже есть наезды текста на картинку, наезд кнопки на картинку ....
    Смотри ширину окна
    2. Сделай отступ снизу, а то как отрезали что то снизу )))

    Если показывать на своем мониторе - то хорошо )))
    Ответ написан
    Комментировать
    Комментировать

  • Покритикуете верстку?

    wapster92
    @wapster92 Куратор тега Вёрстка
    * {transition: .5s ease-in-out;} плохая практика. БЭМ ты вообще не понял. И почему-то мне кажется верстка не закончена.
    Ответ написан
    3 комментария
    3 комментария

  • Покритикуете верстку?

    @karminski
    Senior React.JS Developer
    Opera 68.0.3618.173
    5efb4c6e189ea788199436.png
    Ответ написан
    4 комментария
    4 комментария

  • Покритикуете верстку?

    @domres
    Как человек с 12 летним опытом производства сайтов и эксперт по UI и лендингам могу сказать одно: забейте на трансформации и прочие ненужные эффекты...

    1. Примитивизируйте максимально
    2. Облегчайте
    3. Информация: четко, просто, как для 5-летнего

    И запомните - Ваш клиент в 80% заходов - это человек у которого медленный интернет в смартфоне, у него неуклюжие большие пальцы, маленький экран и смартфон старый и лагает... Если под такого посетителя сделаете чтобы "летало" и было разжевано - продаваться будет даже с хренового дизайна.
    Ответ написан
    Комментировать
    Комментировать

  • Можете покритиковать верстку начинающего?

    sfi0zy
    @sfi0zy Куратор тега CSS
    Creative frontend developer
    С первого взгляда видно:

    1. Font-size в 1px для HTML, а потом все в REM? - интересное начало...
    2. CSS-методология? - отсутствует.
    3. Препроцессор для облегчения жизни? - отсутствует.
    4. Все в одном файле? - да, так и есть.
    5. Магические числа в CSS? - много.
    6. Префиксы прямо в коде? - ага.
    7. 21 брейкпоинт для такой простой страницы? - ежик...
    8. Шрифты? - Fira отвалилась. Letter-spacing скачет.
    9. Critical CSS? - отсутствует.
    10. Бесполезные условные комментарии для IE6, гриды и флексы в одном флаконе? - о дааа.
    11. HTML lang='ru' и контент на английском? - почему бы и нет.
    12. Грузить весь FontAwesome ради пары иконок? - классика.
    13. Асинхронная загрузка картинок? - отсутствует.

    Продолжать можно долго. Рекомендации можно дать следующие:

    1. Освойтесь с инструментами верстальщика. Откройте для себя препроцессоры и автопрефиксер. Почитайте про БЭМ. Используйте.
    2. Добейтесь такого, чтобы вы понимали абсолютно все в своей верстке. Что? Зачем? Почему именно так? Избегайте копипасты. Не используйте какие-то "фишки" только потому, что кто-то их где-то поиспользовал - вполне вероятно, что там был другой контекст, а в вашей ситуации получится что-то странное. Разбирайтесь со всем.
    3. Изучайте хорошие практики.
    4. Ну и JS, без него никак.
    Ответ написан
    2 комментария
    2 комментария

  • Можете покритиковать верстку начинающего?

    notiv-nt
    @notiv-nt
    Как ваше ничего? Да, моё тоже
    Я не вижу верстку, я вижу какую-то кашу, это дизайн такой?

    Что это? https://vimeo.com/349206769

    Хммм, из-за чего же это может быть?
    5d33316763f21644316115.png5d33318999ef4210351417.png

    Это самое худшее использование ремов в истории, в них здесь толку ноль, а fz 1px на html это выстрел мне в ногу, как человеку у которого хреновое зрение, и который ставит ограничение на минимальный шрифт в браузере

    5d33324269da9554829896.png
    Ответ написан
    11 комментариев
    11 комментариев

  • Как сделать так, чтобы, наводя курсор на картинку, появлялся внизу блок с измененным цветом в тексте и с новой информацией (как в примере)?

    twobomb
    @twobomb
    Типа такого можно
    Ответ написан
    Комментировать
    Комментировать