Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Sandro,
Zadanka wyglądają dobrze! 👍
Głównie problem widzę w semantyce, nad którą powinnaś jeszcze popracować (szczegóły w komentarzach).
| <body> | ||
| <header> | ||
| <div>Logo</div> | ||
| <div>Menu |
There was a problem hiding this comment.
Skoro menu to raczej jest to nawigacja czyli znacznik<nav>
| </header> | ||
| <main> | ||
| <article> | ||
| <div> |
There was a problem hiding this comment.
Można użyć znacznika <header> - nie tylko oznacza on nagłówek strony, ale może również sekcji lub artykułu.
01/index.html
Outdated
| </div> | ||
| <p>Treść</p> | ||
| <img /> | ||
| <table></table> |
There was a problem hiding this comment.
Przydałaby się zawartość tabeli
01/index.html
Outdated
| <table></table> | ||
| </article> | ||
| <form> | ||
| <input name="fname"/> |
There was a problem hiding this comment.
Pola formularza warto dodać w element blokowy np. <div><input></div>
01/index.html
Outdated
| <div> | ||
| <div> |
There was a problem hiding this comment.
Tutaj mamy 2 opcje:
- robimy
ul > li, bo to lista komentarzy - robimy
article, bo każdy komentarz to spójna wypowiedź
|
|
||
| .parent--float .child { | ||
| float: left; | ||
| } No newline at end of file |
| <nav> | ||
| <ul class="menu-list"> | ||
| <li class="menu-li">HOME</li> | ||
| <li class="menu-li">ITEMS<span class="rotate"> > </span> |
There was a problem hiding this comment.
Lepiej zrobić "strzałkę" przy pomocy ":after" - jest bardziej PRO :)
| top: 0; | ||
| left: 100%; | ||
| } | ||
|
|
05/index.html
Outdated
| <div class="container"> | ||
| <header class="header"> | ||
| <img class="logo" src="./assets/logo.png" alt="Logo"> | ||
| <div class="menu"> |
There was a problem hiding this comment.
Jak wcześniej pisałem, lepiej użyć nav
| .menu { | ||
| display: flex; | ||
| gap: 1rem; | ||
| } No newline at end of file |
2 participants
No description provided.