reteza / projekt-vizitka Goto Github PK

Terko dobrá práce, tady je pár poznámek:

• Chybí margin 30px pod H1 a nad tlačítky
• Na boxu i kolečku chybí stín
• Není potřeba použít třídu box a contentbox, můžeš to sloučit do jedné
• Border radius na tlačítkách máš 10px, v zadání je 5px. Ta šipka dole ukazuje na roh kartičky.
• Místo značky img bych stylovala nějakou třídu (např. foto). Kdyby bylo použito více obrázků, ať nejsou všechny kulaté. Ale tady je jen jeden, tak to nevadí.
• V CSS se snaž dodržet nějakou posloupnost nahoře bys měla mít body
• Nu a co s tou navigací 🤔, pokud chceš použít odrážky, tak bych to viděla v CSS takto:

.menu {
text-decoration: none;
list-style-type: none;
padding: 0;
margin: 30px 0 0 0;
text-align: center;
}
.menu a {
display: block;
border-radius: 5px;
text-decoration: none;
text-transform: uppercase;
font-weight: bold;
color: white;
padding: 8px 0;
}
.twitter {
background-color: #1da1f2;
margin-bottom: 5px;
border-radius: 5px;
}
.twitter:hover {
background-color: #0b7bc1;
}
.fb {
background-color: #3b5998;
border-radius: 5px;
}
.fb:hover {
background-color: #2b406e;
}

• V tomto případě, bych ale ul a li vyházela a nechala jen dva odkazy nějak takto:

< a class=“button twitter" href="https://twitter.com/" >Twitter
< a class=“button fb" href="https://www.facebook.com/" >Facebook

Té společné třídě .button pak nastavíš jen
color: white;
  font-weight: bold;
  text-decoration: none;
  text-transform: uppercase;
  display: block;
  padding-top: 10px;
  padding-bottom: 10px;
  text-align: center;
  border-radius: 5px;

A twitter a fb zůstane viz 🔝

Nejsem si jistá, že jsem vše napsala srozumitelně, kdyžtak to v úterý dořešíme :-)