FINÁLNÍ VÝPRODEJ KREDITŮ na ITnetwork je tady. Využij poslední šanci a získej až 80 % extra kreditů ZDARMA na náš interaktivní e-learning. Zjisti více:
NOVINKA: Kurz kybernetické bezpečnosti s akreditací MŠMT, nyní již od 0 Kč. Staň se žádaným profesionálem. Zjisti více:
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

 
Odpovědět
24.6.2015 19:55
Avatar

Člen
Avatar
Odpovídá 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* :)

 
Nahoru Odpovědět
24.6.2015 19:59
Avatar
Matúš Petrofčík
Tvůrce
Avatar
Odpovídá 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

Nahoru Odpovědět
24.6.2015 20:39
obsah kocky = r^2 ... a preto vlak drnká
Avatar
David Novák
Tvůrce
Avatar
Odpovídá 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čí)

Nahoru Odpovědět
24.6.2015 21:07
Chyba je mezi klávesnicí a židlí.
Avatar
Matúš Petrofčík
Tvůrce
Avatar
Odpovídá 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#…

Nahoru Odpovědět
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*

Editováno
Nahoru Odpovědět
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 {
 
Nahoru Odpovědět
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í

 
Nahoru Odpovědět
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.

 
Nahoru Odpovědět
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

 
Nahoru Odpovědět
26.6.2015 14:27
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 10 zpráv z 10.