可以采取什么措施来改善此标头?

时间:2018-08-03 08:22:37

标签: javascript header

我想知道:

  • 这是专业素质的标头吗?
  • 如果没有,我可以添加或更改什么?

我尝试不完全将元素放置在标题中并将按钮浮动到右侧,但这总是导致按钮过低。我应该保持元素的绝对位置,还是我不知道另一种方法?

我也无法为我的元素设置正确的ID和类,您知道一种使此代码更干燥的方法吗?

我知道这与专业素质相去甚远,而且我是个白痴,但是我在编写干净的代码时遇到了麻烦,而且我不知道哪种方法最好制作这样的标头。我曾尝试使用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;
            }
        }
    }
}

1 个答案:

答案 0 :(得分:0)

堆栈溢出可能不适合作为将来的参考,但请不要多加注意!这是我的看法:

  1. 切勿在CSS中使用ID。 (Here is an explanation why)
  2. 您应该避免在CSS中嵌套元素选择器。相反,我建议遵循BEM
  3. 命名变量时,我喜欢根据类型设置其格式。所以称您的颜色为$ color-black而不是$ black等。
  4. 我不会使用命名的CSS颜色。使用变量或将其写出。即使使用像黑白这样的“安全”颜色,它看起来也很草率。
  5. 这可以回到#2,但要避免将元素类型用作选择器。给您的按钮等类,并使用它们代替。当然,除非该按钮样式在您的网站中通用。

希望此反馈对您有所帮助!祝你头个好运。