Kritika webu ubytovani-auto-kvasiny.eu
http://ubytovani-auto-kvasiny.eu
V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.
Člen
Zobrazeno 10 zpráv z 10.
//= Settings::TRACKING_CODE_B ?> //= Settings::TRACKING_CODE ?>
V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.
Design na první opravdový počin perfektní! Jednoduché, přehledné.
Kód - snad jen co se týče přehlednosti, odsazuj! Konec kódu je fakt
jednořádková prasárna. 4*
<BR><BR>
keď už sa tie html tagy píšu malým písmom, tak prečo dávať br veľké?
A inak na odsadzovanie blokov slúži css vlastnosť margin aplikovaná na daný element ktorý chceme odsadiť, nie pridávanie dvoch zlomov riadkov. To by potom ešte chcelo niektoré spany zameniť za divy.
Inak je to fakt pekné, ja ti dám 3* lebo som zlý
HTML tagy se nemusí psát malým
(ale je to samozřejmě hezčí)
No ja sa snažím dodržiavať niektové veci z XHTML (napr uzatváranie br tagu...) a jednoducho som na to alergický, i keď to nie je chyba
Napr. hľa na príklady u wikipedii https://en.wikipedia.org/wiki/XHTML#…
Designově pěkné, povedlo se ti to.
Responzivitu by to chtělo dořešit, ovšem psal jsi, že na tom pracuješ,
hodně štěstí.
Kód je trochu horší. Jak psali ostatní
<br />
piš malé jako ostatní a k odsazení používej margin. Občas používáš zastaralé tagy, např.
<b></b>
. Často používáš ID ke stylování, což není moc dobré. ID je
primárně pro JS a na stylování by se mělo používat minimálně, na to je
class. Občas je horší se v kódu vyznat, snaž se ho psát
přehlednější.
Celkově je web pěkný, využívá HTML5, sice není responzivní, ale pracuje
se na tom, kód je poměrně v pořádku. Dávám ti 4*
Řekl bych pěkné.
Ze stránek se člověk nedozví, kdo ubytování provozuje.
BR si můžeš psát jak chceš, i když na to může být někdo alergický,
můžeš psát třeba jen každé druhé písmenko malé, je to fuk (já jsem
také alergický na včelí bodnutí, ale nikomu nezakazuji chovat včely).
Ale používáš ho špatně. Pokud chceš udělat nový odstavec, použij
<p></p>
Mezeru si můžeš nastylovat v CSS.
Není to chyba, ale zbytečně deklaruješ id tam, kde existuje jen jeden
element toho druhu (třeba header, nav...).
V CSS také zbytečně moc zanořuješ:
Nemusíš mít:
#nav ul li a:hover{
stačí
#nav a:hover{
nebo místo:
body#home a#active {
stačí :
#active {
Hezkým design, kód neřeším. 1* Sám bych takový web neudělal. Ještě se učím spoustu příkazů a jdu do javascriptu. Pak do těžšího PHP.
Děkuji za všechny připomínky i hodnocení. Jsem rád, že má práce byla přijata kladně. Mám teď alespoň o něco větší motivaci v tomto pokračovat, a za nějaký čas vám třeba budu schopen ukázat něco "na úrovni."
Mnoho chyb v kódu je dáno tím, že jsem tu část pro jistotu nechal být, jakmile začala nějakým způsobem fungovat. Obecně jsem totiž ten typ člověka, který rozbitou věc opraví stylem "funguje to tak napůl, a tohle mi zbylo."
Zapracuji na tom.
Matúš: 3* jsou dobré, válku se Slovenskem kvůli tomu začínat neplánuji
Zobrazeno 10 zpráv z 10.