Оцените вёрстку?

Question

Леонид Роженцев @RLeo777

Code review

Оцените вёрстку?

https://calcfert.000webhostapp.com/
Вот мой учебный проект. Можете пожалуйста оценить вёрстку и сайт в общем? Буду очень благодарен, готов к конструктивной критике.

Вопрос задан более трёх лет назад
129 просмотров

12 комментариев

Подписаться 1 Простой 12 комментариев

Владислав Лысков @Vlatqa

Там нечего оценивать, а из того, что есть
Это не нормально

Написано более трёх лет назад
Denis @sidni

Поражение.
Такой дизайн был хорош в начале 2000, еще сайт на вордпрессе там есть же куча нормальных тем

Написано более трёх лет назад
Леонид Роженцев @RLeo777 Автор вопроса

Владислав Лысков, а как лучше спозиционировать элементы калькулятора?

Написано более трёх лет назад
Ankhena @Ankhena

0. Было бы не плохо, если бы вы сами проверили её хотя бы валидатором. И разметку и стили. А то там всякие float: center.
1. Нет отступов у инпутов
2. Очень странные толстые зеленые рамки, привет 90ые. Но, возможно, это не к вам, а к "дизайнеру".
3. Некачественные, плохо обрезанные лого, да ещё и на белом фоне.
4. Кривой адаптив. Часть центрирована, часть нет. Контент прилипает к краю страницы. На 320 совсем беда.
5. При расчете пишется "Ошибка", а как юзеру понять чего ошибка не известно.
6. Неверно подключены шрифты.

Хватит пока

Написано более трёх лет назад
Владислав Лысков @Vlatqa

Леонид Роженцев, отступы сделать, и сделать так, что бы текст в инпуты влазил

Написано более трёх лет назад
Леонид Роженцев @RLeo777 Автор вопроса

Ankhena, спасибо. Дизайн я сам разрабатывал, учитель информатики немного помогал. Но исправлю

Написано более трёх лет назад
Леонид Роженцев @RLeo777 Автор вопроса

Ankhena, а почему неверно подключены шрифты? Вроде работает

Написано более трёх лет назад
Леонид Роженцев @RLeo777 Автор вопроса

Спасибо, буду исправлять

Написано более трёх лет назад

Ankhena @Ankhena

Леонид Роженцев, потому что один шрифт подключается с одним названием вне зависимости от жирности.
Так:

@font-face {
        font-family: "OpenSans";
        src: url("fonts/opensans.woff2") format("woff2"),
        url("fonts/opensans.woff") format("woff");
        font-weight: normal;
        font-style: normal;
        font-display: swap;
        }

        @font-face {
        font-family: "OpenSans";
        src: url("fonts/opensansbold.woff2") format("woff2"),
        url("fonts/opensansbold.woff") format("woff");
        font-weight: 700;
        font-style: normal;
        font-display: swap;
        }

div {
      font-family: OpenSans, Arial, sans-serif;
      font-weight: bold;
    }

Представьте, сколько всего нужно исправить, чтобы сменить шрифт в вашем варианте.

Написано более трёх лет назад

Леонид Роженцев @RLeo777 Автор вопроса

Спасибо

Написано более трёх лет назад
Леонид Роженцев @RLeo777 Автор вопроса

Ankhena, а есть альтернатива свойству float?

Написано более трёх лет назад
Ankhena @Ankhena

Леонид Роженцев, ага, google -> "CSS центрирование"

Написано более трёх лет назад

Решения вопроса 2

Комментировать

3 комментария

Леонид Роженцев @RLeo777 Автор вопроса

Спасибо, буду исправлять

Написано более трёх лет назад
Леонид Роженцев @RLeo777 Автор вопроса

Только не знаю, как в тамплейтк обратной связи подключить внешние стили

Написано более трёх лет назад

Steppp @Steppp

index.html

<!DOCTYPE html>
<head>
    <?php wp_head(); ?>
</head>
<main></main>
<footer></footer>

  <?php wp_footer(); ?>
</body>

</html>

functions.php

add_action( 'wp_enqueue_scripts', 'theme_style' );
  function theme_style() {
    wp_enqueue_style( 'style', get_stylesheet_uri());
    wp_enqueue_style( 'default', get_template_directory_uri() . '/assets/css/default.css');
    wp_enqueue_style( 'layout', get_template_directory_uri() . '/assets/css/layout.css');
  };

  add_action( 'wp_enqueue_scripts', 'theme_scripts' );
  function theme_scripts() {
    wp_deregister_script( 'jquery' );
    wp_register_script('jquery', '//ajax.googleapis.com/ajax/libs/jquery/3.5.1/jquery.min.js');
    wp_enqueue_script('jquery');


    wp_enqueue_script('flexslider', get_template_directory_uri() . '/assets/js/jquery.flexslider.js', ['jquery'], null, true );
    wp_enqueue_script('doubletaptogo', get_template_directory_uri() . '/assets/js/doubletaptogo.js', ['jquery'], null, true );
    wp_enqueue_script('init', get_template_directory_uri() . '/assets/js/init.js', ['jquery'], null, true );
    wp_enqueue_script('modernizr', get_template_directory_uri() . '/assets/js/modernizr.js', ['jquery'], null, false, null);

  }

Написано более трёх лет назад

Пригласить эксперта

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Похожие вопросы

Python

+1 ещё

Простой
Как устранить дублирование кода?
- 1 подписчик
- 02 окт.
- 191 просмотр
1

ответ
Python

+1 ещё

Простой
Как улучшить код?
- 1 подписчик
- 08 июл.
- 162 просмотра
2

ответа
Spring

+1 ещё

Простой
Как пометить геттеры, используемые Jackson, чтобы ревьювер меня понимал?
- 1 подписчик
- 02 июл.
- 61 просмотр
1

ответ
C

+1 ещё

Простой
Взгляд опытных программмистов, что улучшить и исправить?
- 1 подписчик
- 22 янв.
- 219 просмотров
2

ответа
PHP

+2 ещё

Простой
Какие архитектурные решения можно применить?
- 6 подписчиков
- более года назад
- 745 просмотров
3

ответа
Python

+1 ещё

Простой
Сортировка вставками?
- 1 подписчик
- более года назад
- 106 просмотров
1

ответ
Code review

Средний
Мой код слишком громоздкий?
- 1 подписчик
- более года назад
- 159 просмотров
2

ответа
C#

+1 ещё

Простой
В чем может быть ошибка при решении задания?
- 1 подписчик
- более года назад
- 138 просмотров
1

ответ
C#

+1 ещё

Средний
Можно ревью супергрязного(наверное) кода на с#?
- 2 подписчика
- более года назад
- 513 просмотров
3

ответа
C#

+1 ещё

Простой
Куда расти дальше?
- 1 подписчик
- более года назад
- 306 просмотров
4

ответа
Показать ещё Загружается…

Senior Golang Developer (офис)

SpectrumData • Екатеринбург

от 300 000 до 400 000 ₽

React-Native Developer (Mobile)

Intelinvest

от 90 000 ₽

Yii2 Бекенд-разработчик backend middle+

MKOMOV Studio

До 190 000 ₽

Выложить 75 курсов на геткурс и настроить базовые функции

26 нояб. 2024, в 02:18

15000 руб./за проект

Редизайн и доработка интернет магазина на wordPress

25 нояб. 2024, в 23:42

50000 руб./за проект

Разработка мобильного приложения с дополненной реальностью (AR)

25 нояб. 2024, в 23:32

200000 руб./за проект

Там нечего оценивать, а из того, что есть
Это не нормально
Поражение.
Такой дизайн был хорош в начале 2000, еще сайт на вордпрессе там есть же куча нормальных тем
Владислав Лысков, а как лучше спозиционировать элементы калькулятора?
0. Было бы не плохо, если бы вы сами проверили её хотя бы валидатором. И разметку и стили. А то там всякие float: center.
1. Нет отступов у инпутов
2. Очень странные толстые зеленые рамки, привет 90ые. Но, возможно, это не к вам, а к "дизайнеру".
3. Некачественные, плохо обрезанные лого, да ещё и на белом фоне.
4. Кривой адаптив. Часть центрирована, часть нет. Контент прилипает к краю страницы. На 320 совсем беда.
5. При расчете пишется "Ошибка", а как юзеру понять чего ошибка не известно.
6. Неверно подключены шрифты.

Хватит пока
Леонид Роженцев, отступы сделать, и сделать так, что бы текст в инпуты влазил
Ankhena, спасибо. Дизайн я сам разрабатывал, учитель информатики немного помогал. Но исправлю
Ankhena, а почему неверно подключены шрифты? Вроде работает
Леонид Роженцев, потому что один шрифт подключается с одним названием вне зависимости от жирности.
Так:

@font-face { font-family: "OpenSans"; src: url("fonts/opensans.woff2") format("woff2"), url("fonts/opensans.woff") format("woff"); font-weight: normal; font-style: normal; font-display: swap; } @font-face { font-family: "OpenSans"; src: url("fonts/opensansbold.woff2") format("woff2"), url("fonts/opensansbold.woff") format("woff"); font-weight: 700; font-style: normal; font-display: swap; }

div { font-family: OpenSans, Arial, sans-serif; font-weight: bold; }

Представьте, сколько всего нужно исправить, чтобы сменить шрифт в вашем варианте.
Ankhena, а есть альтернатива свойству float?
Леонид Роженцев, ага, google -> "CSS центрирование"

Answer 1 · 2021-02-12 15:50:49

Отсутствие отступов в input'ах - автоматически лишает вас рабочего места. Я бы даже после такого на порог вас не пустил.

Answer 2 · 2021-02-12 15:40:00

))) Ну ты попал)))

1)<link rel="favicon.ico" type="image/x-icon"> нету href

<link rel="icon" type="image/png" sizes="16x16" href="./favicon.png">

2) А зачем туда подключать все?

Скриншот

3) на 320px горизонтальный скролл + иконки в блоке about, не по центру! И по идеи это совсем другой блок!

4) Форма обратной связи, уехала в низ на пол экрана!
Меньше 900px горизонтальный скролл
Так же, в textarea полосы скрола!
Зато фавиконка подключени правильно

5) И не совсем хорошо писать css в html style! Засунь все в 1 css фаил, и подключи на всех страницах

Оцените вёрстку?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт