Что можно сделать, чтобы улучшить этот заголовок?

Я бы хотел знать:

  • эта шапка профессионального качества?
  • если нет, что я могу добавить или изменить?

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

У меня также возникли проблемы с установкой правильных идентификаторов и классов для моих элементов, знаете ли вы, как сделать этот код более СУХИМ?

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

Буду очень признателен, если вы дадите мне несколько советов, спасибо!

HTML:

<header id = "master-head">
    <div id = "master-head-content">

        <span id = "master-head-title">Brodie Been</span>

        <span id = "master-head-primary">
            <a href = "#">My Work</a>
            <a href = "#">About Me</a>
        </span>

        <span id = "master-head-secondary" class = "menu-right">
            <button id = "master-head-contact">CONTACT ME</button>
        </span>
    </div>
</header>

SCSS:

$shadowColor:rgba(0,0,0,0.16);
$softBlack:#404040;
$primaryColor:#348DC8;

@mixin vertCenter{
    position:absolute;
    top:50%;
    transform:translate(0, -50%);
}


body{
    background:#404040;
    font-family:Open Sans;

    button{
        display:inline-block;
        position:relative;
        border:none;
        color:white;
        text-decoration:none;
        font-size:14px;
        background:$primaryColor;

        &:hover{
            cursor:pointer;
            color:white;
            background:$softBlack;
        }
    }
}








#master-head{
    position:relative;
    height:100px;
    font-size:14px;
    background:white;

    #master-head-content{
        width:100%;
        height:100%;

        #master-head-title{
            @include vertCenter;
            padding-left:50px;
            font-size:24px;
        }

        #master-head-primary{
            @include vertCenter;
            padding-left:300px;

            a{
                text-decoration:none;
                color:inherit;
                margin-right:30px;

                &:hover{
                    margin-bottom:-1px;
                    border-bottom:1px solid black;
                }
            }
        }

        #master-head-secondary{
            @include vertCenter;
            right:50px;

            #master-head-contact{
                width:130px;
                height:50px;
            }
        }
    }
}

Лучше подходит для просмотра кода, чем SO

CertainPerformance 03.08.2018 10:23

Я думаю, что это довольно хорошо, я бы сделал его немного более дружественным к SEO, используя соответствующие текстовые теги, например: <title> для заголовка.

Ricardo Costa 03.08.2018 10:44
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
0
2
29
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Переполнение стека, вероятно, не место для этого для справок в будущем, но не переживайте! Вот мой вывод:

  1. Никогда не используйте идентификаторы с CSS. (Вот объяснение, почему)
  2. Вам следует избегать вложения селекторов элементов в CSS. Вместо этого я рекомендую следовать БЭМ.
  3. При именовании переменных я люблю форматировать их по типу. Так что назовите свои цвета $ color-black, а не просто $ black и т. д.
  4. Я бы не стал использовать именованные цвета css. Либо используйте переменные, либо запишите их. Даже с «безопасными» цветами, такими как черный и белый, он выглядит неаккуратно.
  5. Это восходит к пункту 2, но избегайте использования типов элементов в качестве селекторов. Дайте вашей кнопке и т. д. Классы и используйте их вместо этого. Если, конечно, этот стиль кнопок не будет универсальным для вашего сайта.

Надеюсь, этот отзыв поможет вам! Удачи с заголовком.

Другие вопросы по теме