نظر درباره کد های همبرگر منوی من

سلام علیکم .
آقا من یه همبرگر منو برای موبایل ساختم(هنوز ریسپانسیو نیست البته.
میخوام بدونم درست و عادی هست یا نه،چون نمیدونم چرا احساس میکنم احتمال داره یکی بیاد مثل خودم فایل های سایتمو دانلود کنه چکشون کنه آبروم بره/:
Html :

 <div class="Cmenu">
            <button type="button" id="cb" onclick="menubutton()">
            <div class="bmenu"></div>
            <div class="bmenu"></div>
            <div class="bmenu"></div>
            </button>
        <ul class="menu" id="menu">
            <li>
                <button id="close" onclick="closebutton()">
                <div class="bmenu" id="one"></div>
                <div class="bmenu" id="two"></div>
            </button>
            </li>
            <li><a href="./index.htm">خانه</a></li>
            <hr>
            <li><a href="">ثبت نام</a></li>
            <hr>
            <li><a href="">سوالات متداول</a></li>
            <hr>
            <li><a href="">بلاگ</a></li>
            <hr>
            <li><a href="">درباره ما</a></li>
        </ul>
        </div> 

Css

  .menu {
   display:none;
    background:white;
    height:100%;
    width:55%;
    z-index:300;
    overflow:hidden;
    top:0;
    position:fixed;   
}
.menu li {
    list-style-type:none;
    margin-bottom:20px;
}
.menu li a {
    text-decoration:none;
    color:#121212;
}
.bmenu {
    width:25px;
    background:black;
    height:4px;
    border-radius:3px;
    margin-bottom:3px;
    position:realtive;
    margin-right:10px;
    transform-origin:50%;
}
#cb {
    background:none;
    border:none;
}
#close {
    background:none;
    border:none;
    float:left;
    margin-left:35px;
    margin-top:10px;
    position:realtive;
    z-index:300;

}
#one {
    transform:rotate(45deg);
    position:absolute;
}
#two {
    transform:rotate(-45deg);
    position:absolute;
}

Js

function menubutton(){
document.getElementById("menu").style.display = "block"
}
function closebutton(){
if(document.getElementById("menu").style.display === "block") {
    document.getElementById('menu').style.display = "none"
}

}

آهان راستی یه کمک هم بکنین لطفا من تو جاوا اسکریپت خیلی نوبم میخوام انیمیشن داشته باشه منو.تنکس(: