Valentínska akcia je tu! Získaj až 80 % extra kreditov ZADARMO na náš interaktívny e-learning. ZISTIŤ VIAC:
NOVINKA: Najžiadanejšie rekvalifikačné kurzy teraz s 50% zľavou + kurz AI ZADARMO. Nečakaj, táto ponuka dlho nevydrží! Zisti viac:
Avatar
Bohouš Franc
Člen
Avatar
Bohouš Franc:24.6.2015 19:55

Zdravím pánové (případně náhodně zabloudivší dámy),

prosím o kritiku webu. Je to v podstatě můj první "opravdový" počin. Snažil jsem se využít všechny své nevelké a omezené znalosti, a teď je vše uděláno tak, jak si myslím, že by to asi správně mělo vypadat. (Je vítáno, pokud mě vyvedete z omylu.)

O čem vím:

  • doména není úplně nejšťastnější, ale měl jsem na to pramalý vliv
  • web není responzivní, na tom se snažím pracovat

Rád bych znal názor na technickou i "baj voko" část stránek.

Nakonec bych chtěl podotknout, že jsem neměl tu drzost (nebo chcete-li odvahu) si za to říct peníze a s největší pravděpodobností stránky budou jen dočasným řešením. Pro mě to nicméně význam má :) Díky

 
Odpovedať
24.6.2015 19:55
Avatar

Člen
Avatar
Odpovedá na Bohouš Franc
:24.6.2015 19:59

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* :)

 
Hore Odpovedať
24.6.2015 19:59
Avatar
Matúš Petrofčík
Tvůrce
Avatar
Odpovedá na Bohouš Franc
Matúš Petrofčík:24.6.2015 20:39 
<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ý :D

Hore Odpovedať
24.6.2015 20:39
obsah kocky = r^2 ... a preto vlak drnká
Avatar
David Novák
Tvůrce
Avatar
Odpovedá na Matúš Petrofčík
David Novák:24.6.2015 21:07

HTML tagy se nemusí psát malým :D :P

(ale je to samozřejmě hezčí)

Hore Odpovedať
24.6.2015 21:07
Chyba je mezi klávesnicí a židlí.
Avatar
Matúš Petrofčík
Tvůrce
Avatar
Odpovedá na David Novák
Matúš Petrofčík:24.6.2015 21:30

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#…

Hore Odpovedať
24.6.2015 21:30
obsah kocky = r^2 ... a preto vlak drnká
Avatar
Michal Šmahel
Člen
Avatar
Michal Šmahel:24.6.2015 21:59

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*

Editované
Hore Odpovedať
24.6.2015 21:59
Nejdůležitější je motivace, ovšem musí být doprovázena činy.
Avatar
Tonda Kozák
Člen
Avatar
Tonda Kozák:25.6.2015 8:06

Ř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 {
 
Hore Odpovedať
25.6.2015 8:06
Avatar
smiesek
Člen
Avatar
smiesek:26.6.2015 6:38

mě se líbí přehlednost a jednoduchost stránek, pokud to má hlavně sloužit jako internetová prezentace ubytování a žádný přehnaný požadavek na častou změnu obsahu stránek, forma statických stránek je postačující

 
Hore Odpovedať
26.6.2015 6:38
Avatar
vojtanosek
Člen
Avatar
vojtanosek:26.6.2015 7:11

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.

 
Hore Odpovedať
26.6.2015 7:11
Avatar
Bohouš Franc
Člen
Avatar
Bohouš Franc:26.6.2015 14:27

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 :D

 
Hore Odpovedať
26.6.2015 14:27
Robíme čo je v našich silách, aby bola tunajšia diskusia čo najkvalitnejšia. Preto do nej tiež môžu prispievať len registrovaní členovia. Pre zapojenie sa do diskusie sa zaloguj. Ak ešte nemáš účet, zaregistruj sa, je to zadarmo.

Zobrazené 10 správy z 10.