Ответы пользователя по тегу Code review
  • Оцените верстку по БЭМ?

    @BATPYIIIKOB
    PHP, JS
    Зацени вёрстку:

    5de013c62cbff038213481.png
    Ответ написан
    Комментировать
  • Зацените верстку?

    @BATPYIIIKOB
    PHP, JS
    Ну такое... Я так понимаю тут не по дизайну а по вёрстке, т.к. дизайн просто шаблон... коих мильон.
    По вёрстке и адаптации + кроссбраузерность:
    5de012134e3d5775949514.png
    5de0121bc4e20723919606.png
    5de01221c3e51293600561.png
    5de0122aec982551708185.png
    5de0123345b2c585855854.png

    Мельком пробежался....
    Выравнивания! Отступы! Композиция!..
    Ответ написан
    Комментировать
  • Не могли бы вы оценить мой верстку?

    @BATPYIIIKOB
    PHP, JS
    Если дизайн был таким же - то вёрстка пойдёт!
    По дизайну:
    1. Типографика просто ниочём. Не делайте чёрную обводку на тёмном фоне. Подберите интересные современные шрифты
    2. Срочно смени фон на главной!!! Я вначале подумал, что у меня очки запачкались.
    3. Фон - тёмный! да ещё и цвета странные. Тёмный сайт тяжелее воспринимается если он не ВАУ!
    4. Огромные тени у блоков - не модно
    5. Зачем вообще блоки и обводки, по заголовкам ясно где начинается раздел...
    5.1. Да и навигация должна поднимать вверх заголовок .
    6. Скруглённые блоки в сочетании с панелями с прямыми углами... 90-еее и опять обводки..
    5de00cb58ce05458023227.png

    ну и т.д....

    7. Чаще заходи на https://www.awwwards.com/ и изучай какие сайты в тренде, какие шрифты и цвета.

    Дополню: Всё таки залез вёрстку:
    1. header__nav nav - НАХЕРА!!!!!! __ - это что??? как же я задолбался разгребать потом шаблоны после __. Кто вас учил делать __ ????? Если делаешь следуй тогда БЭМ! А просто от балды везде пихать....ненадо
    2. Отступы - 4 ПРОБЕЛА.. 4!!!! а не как нравится.
    3. /> - необязателен, уже не нужен. Просто >
    4. Почему <h1>Мое резюме</h1> , а <h2>Мои контакты</h2> ?! Они ж на одном уровне.
    h1,h2,h3 - это не размер шрифта....
    5. Если выкладываешь вёрстку и просишь заценить - то : НЕМИМИМИШЬ ЕЁ!!!! как посмотреть css ?
    .....
    Ответ написан
    2 комментария
  • Этот код сильно ужасен)?

    @BATPYIIIKOB
    PHP, JS
    1. Функция должна занимать не более одной страницы в высоту.
    2. Общие блоки - выдели в отдельные функции
    3. Комментарии кто будет писать?
    4. Зачем выделять переменные и не использовать их?
    $curtime = date(H);
    5. Разбейте код на отдельные функции - наименования по смыслу (не говорю про ООП, нет смысла). А то всё это похоже на if-овый бред. Непонятен алгоритм. Код должен читаться как книжка.
    6. советую почитать https://refactoring.guru/ru/refactoring/smells
    https://refactoring.guru/ru/decompose-conditional - по ифам

    да и....
    ЗАБЕЙ НА БИТРИКС!
    Ответ написан
    Комментировать