Open TomasProkes opened 3 years ago
Ahoj Kristiano,
vizitku máš moc hezky zpracovanou, díky! Jen pár drobností:
1) chybí ti změna podbarvení tlačítek při najetí myší. Až budeš mít chvilku, zkus si to prosím přidat, děkuji.
2) myšlenka vyčlenění společných vlastností pro obě tlačítka do samostatného selektoru je dobrá. Dělá se to ale nikoli přes selektor elementu, ale přes samostatnou třídu. V html kódu se pak pro dané elementy nastaví obě třídy najednou -- ta obecná i ta konkrétní (např. class="tlacitko odkazt" pro první, a class="tlacitko odkazf" pro druhý z nich). Důvod je ten, že časem budeš chtít přidat třeba další element s obyčejným odkazem, a nebudeš chtít, aby se zobrazoval jako tlačítko.
https://github.com/Crystalette/PROJEKT-Vizitka/blob/43f5371233889aa501884c27f7d5ff8ce0f055ec/style.css#L38-L47
Jinak prima, dobrá práce. Gratuluju!
Ahoj,
děkuji za kontrolu, opravila jsem. Teď by už to mělo být v pořádku. Kdyžtak prosím o kontrolu.
Kris
Ahoj Kristiano,
vizitku máš moc hezky zpracovanou, díky! Jen pár drobností:
1) chybí ti změna podbarvení tlačítek při najetí myší. Až budeš mít chvilku, zkus si to prosím přidat, děkuji.
2) myšlenka vyčlenění společných vlastností pro obě tlačítka do samostatného selektoru je dobrá. Dělá se to ale nikoli přes selektor elementu, ale přes samostatnou třídu. V html kódu se pak pro dané elementy nastaví obě třídy najednou -- ta obecná i ta konkrétní (např. class="tlacitko odkazt" pro první, a class="tlacitko odkazf" pro druhý z nich). Důvod je ten, že časem budeš chtít přidat třeba další element s obyčejným odkazem, a nebudeš chtít, aby se zobrazoval jako tlačítko.
https://github.com/Crystalette/PROJEKT-Vizitka/blob/43f5371233889aa501884c27f7d5ff8ce0f055ec/style.css#L38-L47
Jinak prima, dobrá práce. Gratuluju!