NOVINKA: Získej 40 hodin praktických dovedností s AI – ZDARMA ke každému akreditovanému kurzu!

Kritika webu www.tesarske-konstrukce.cz

Jak se ti líbí web?
2 hlasů
Přejít na web:
http://www.tesarske-konstrukce.cz

V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity
Avatar
k.ats
Člen
Avatar
k.ats:9.7.2013 10:10

Ahoj, tohle je můj první web. Dělal jsem ho dle místního návodu na html a css + nějaká ta má práce do toho byla vložena. Je to jednoduchý web na prezentaci malé firmičky.

 
Odpovědět
9.7.2013 10:10
Avatar
tastyfish
Tvůrce
Avatar
tastyfish:15.7.2013 18:15

Na první web hodně dobré! Stránky ti ale neprojdou html validátorem a ohledně menu bych vytknul to, že musíš najet na text položky, aby šlo na odkaz kliknout, přitom to vypadá, že je klikatelné celé tlačítko - to jsou ale jenom takové drobnosti.

Nahoru Odpovědět
15.7.2013 18:15
škoda mluvit
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Neaktivní uživatel:15.7.2013 18:35

S kódem je to maličko horší.. ale po vzhledové stránce musím pochválit ;)

Nahoru Odpovědět
15.7.2013 18:35
Neaktivní uživatelský účet
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Michal Žůrek - misaz:15.7.2013 18:43

mě zaráží proč když chcete vycentrovat obsah (a to není jen tvoje chyba, to tu totiž nějací chytrolíni cpou do hlav všem) tak ji celou obalíte do divu. Proč nenastavit toto body:

body {
   max-width:900px;
   margin:0 auto;
}

pokud je to kvůli obarveni pozadí, tak pod body je ještě :root (teď ale nevím s kolika dvojtečkami):

:root {
   background-color:green;
}
 
Nahoru Odpovědět
15.7.2013 18:43
Avatar
Paradise
Tvůrce
Avatar
Paradise:15.7.2013 18:59

Podle mě to je povedené vzhledem k tématu. Akorát tlačítka bych udělal klikací po celém obsahu, ne jen text. Možná ještě odstraň počítadlo, dokud se nebude čím chlubit :)

Btw když web vypadá dobře a funguje, proč řešit validátor? Do webů nedělám, tak mě to zajímá.

 
Nahoru Odpovědět
15.7.2013 18:59
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na Paradise
Neaktivní uživatel:15.7.2013 19:03

Myslím že by to mohlo ovlivňovat pozici ve vyhledávačích, a taky by to nemuselo ve starších browserech fungovat (záleží na typu chyby).

Nahoru Odpovědět
15.7.2013 19:03
Neaktivní uživatelský účet
Avatar
k.ats
Člen
Avatar
k.ats:16.7.2013 12:30

Děkuji za kritiku :-) Bohužel jsem to tím validátrem taky projížděl a své chyby v kódu jsem opravil, ale zůstávají tam chyby v aplikaci lightbox, které neumím opravit :-( S tím teda nevím co dělat.

 
Nahoru Odpovědět
16.7.2013 12:30
Avatar
k.ats
Člen
Avatar
k.ats:16.7.2013 12:35

Jo a to počítadlo jsem tam dávat nechtěl. Chtěl to zadavatel té stránky aby viděl kolik mu tam chodí lidí. Taky jsem ho od toho odrazoval, že to kazí vzhled :-)

 
Nahoru Odpovědět
16.7.2013 12:35
Avatar
Kit
Tvůrce
Avatar
Odpovídá na k.ats
Kit:16.7.2013 12:44

Počitadlo se dělá neviditelné, aby návštěvníka nerušilo. Samotné počitadlo bez grafů je také k ničemu.

Nahoru Odpovědět
16.7.2013 12:44
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na k.ats
David Hartinger:16.7.2013 13:14

Jednoduché, ale vypadá to docela dobře.

Nahoru Odpovědět
16.7.2013 13:14
New kid back on the block with a R.I.P
Avatar
Snorlax
Tvůrce
Avatar
Odpovídá na Kit
Snorlax:16.7.2013 13:21

tak mu doporuč Google Analytics. pokud se nepletu tak je gratis ne? a má mnohem víc funkcí než blbej toplist...
http://www.google.com/analytics/

Nahoru Odpovědět
16.7.2013 13:21
Kdo chce pochopit, pochopí. Kdo dělá že chce pochopit, může pouze dělat, že pochopil...
Avatar
tastyfish
Tvůrce
Avatar
tastyfish:16.7.2013 13:59

Ano, google analytics je skvělý.

Editováno 16.7.2013 14:01
Nahoru Odpovědět
16.7.2013 13:59
škoda mluvit
Avatar
k.ats
Člen
Avatar
k.ats:16.7.2013 14:26

Google analytics tam mám, ale jde o to, že web je dělaný pro člověka co na počítači moc nedělá a on to chce vidět přímo na stránce. Je to jeho volba. Já jsem to tam nechtěl. Ale děkuji za tipy. Teď se ještě koukám na ten svůj kód a zkouším ho vyladit aby prošel validatorem. Děkuji za rady :-)

 
Nahoru Odpovědět
16.7.2013 14:26
Avatar
Ex_0
Člen
Avatar
Ex_0:16.7.2013 15:06

neuvazoval si nad novsim vzhladom? :)

nieco som spravil, ale nieje to este hotove

 
Nahoru Odpovědět
16.7.2013 15:06
Avatar
k.ats
Člen
Avatar
k.ats:16.7.2013 16:47

Jak udelam cele tlacitko jako odkaz. Tato syntaxe neni dle validatoru dobre a jiny zpusob me nenapada?

<a href="fotogalerie.html" title="Fotogalerie"><li class="neaktivni" >Fotogalerie </li></a>

Udajne nemuze byt prvek "a" pred prvkem "li", jako jediny zpusob me teda jeste napada, dat ten text do divu, nastavit mu sirku a vysku stejnou jako je "tlacitko" a ten dat do odkazu. Pak ale budu muset zrusit padding u prvku "li" a napozicovat tam text nejak jinak.

 
Nahoru Odpovědět
16.7.2013 16:47
Avatar
Ex_0
Člen
Avatar
Ex_0:16.7.2013 18:01

v <li> si zrus pading-left:15px; a do <a> si dopln

width: 210px; height: 50px; display: block; padding: 0 0 0 15px;

malo by to ist, aspon mne to ide, ked som si to tak upravil

a este si tu daj medzi <li> daj <a> ...takto

<li id="prvni" class="aktivni"><a href="index.html" title="O nas">O nás</a></li>

cize to bude vyzerat

<ul>
                            <li id="prvni" class="aktivni"><a href="index.html" title="O nas">O nás</a></li>
                            <li><a href="fotogalerie.html" title="Fotogalerie">Fotogalerie</a> </li>
                            <li><a href="kde_nas.html" title="Kde nás najdete?">Kde nás najdete?</a> </li>
                            <li id="posledni"><a href="kontakt.html" title="Kontaktujte nás">Kontaktujte nás</a> </li>
                    </ul>

a kod

article nav ul li {
color: #FFFECF;
background: url("obr/footer.png") #182706;
font-size: 15px;
text-transform: uppercase;
font-weight: bold;
height: 50px;
line-height: 3em;
margin-top: 5px;
margin-left: 5px;
margin-bottom: 5px;
margin-right: 0px;
letter-spacing: 2px;
border: 2px solid #FFFECF;}

article nav a {
color: #FFFECF;
text-decoration: none;
width: 210px;
height: 50px;
display: block;
padding: 0 0 0 15px;
}
 
Nahoru Odpovědět
16.7.2013 18:01
Avatar
k.ats
Člen
Avatar
Odpovídá na Ex_0
k.ats:16.7.2013 19:35

Děkuji :-) já to mezi tím udělal do toho divu, vůbec mě nenapadlo nastylovat přímo odkaz. Pak to zkusím předělat do toho prvku "a", jak píšeš, aspoň bude o div méně :-)

Editováno 16.7.2013 19:38
 
Nahoru Odpovědět
16.7.2013 19:35
Avatar
k.ats
Člen
Avatar
k.ats:16.7.2013 20:06

tady to mám udělaný zatím s těmi divy http://tesarskeprace.tode.cz/

 
Nahoru Odpovědět
16.7.2013 20:06
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Neaktivní uživatel:16.7.2013 20:28

Není to špatné. Chválím používání HTML5. Jen při nastavování kódování bych místo

<meta http-equiv="Content-Type" content="text/html; charset=utf-8">

napsal

<meta charset="utf-8" />

. Je to kratší, a uspůsobený k HTML5.

Nahoru Odpovědět
16.7.2013 20:28
Neaktivní uživatelský účet
Avatar
Snorlax
Tvůrce
Avatar
Snorlax:16.7.2013 20:29

nevim jestli se to stalo jen mě ale tu mapu máš tady posunutou http://tesarskeprace.tode.cz/kde_nas.html (sídlo neni vidět ;) )

Nahoru Odpovědět
16.7.2013 20:29
Kdo chce pochopit, pochopí. Kdo dělá že chce pochopit, může pouze dělat, že pochopil...
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Neaktivní uživatel:16.7.2013 20:32

A ještě taková drobnost: když se zvětší fotografie, píše to Fotogragie, místo Fotografie. :)

Nahoru Odpovědět
16.7.2013 20:32
Neaktivní uživatelský účet
Avatar
k.ats
Člen
Avatar
k.ats:16.7.2013 21:09

Děkuji za rady a přípomínky. Kratší kód se znakovou sadou vyměním :-) Fotogragie je úspěšně spravena, zatím ale pouze na lokálu. A ta mapa je bohužel naprd. První dva dny to fungovalo, ale gůglovský mapy tu adresu už nějak neznají ani v plných mapách, tak odkazují na střed obce... Taky to nechápu. Předtím to ulice a čísla znalo, teď už ne. Jedině tam dát mapu z mapy.cz jinak nevím co s tím google má.

 
Nahoru Odpovědět
16.7.2013 21:09
Avatar
Snorlax
Tvůrce
Avatar
Odpovídá na k.ats
Snorlax:16.7.2013 21:14

tak zkus udělat znova ne? otevřít G maps, najít si tu adresu,...

Nahoru Odpovědět
16.7.2013 21:14
Kdo chce pochopit, pochopí. Kdo dělá že chce pochopit, může pouze dělat, že pochopil...
Avatar
k.ats
Člen
Avatar
Odpovídá na Snorlax
k.ats:16.7.2013 21:30

To jsem zkousel, ale kdyz jdu na gmaps, tak to tu adresu uz nezna a odkazuje to na stred obce, pritom kdyz jsem to delal, tak to jeste znalo. Ted uz ne.

 
Nahoru Odpovědět
16.7.2013 21:30
Avatar
Vojtěch Pospíchal
Tvůrce
Avatar
Odpovídá na k.ats
Vojtěch Pospíchal:16.7.2013 22:17

Tak si to oprav ne? viz: http://dotekomanie.cz/…r-pro-cesko/

 
Nahoru Odpovědět
16.7.2013 22:17
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Neaktivní uživatel:16.7.2013 23:57

Proč mám ten pocit, že tohle IE 8 & lower neveme?

Nahoru Odpovědět
16.7.2013 23:57
Neaktivní uživatelský účet
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na k.ats
Neaktivní uživatel:16.7.2013 23:59

Máte koupenou doménu a nekoupíte si hosting? Zvláštní ...

Nahoru Odpovědět
16.7.2013 23:59
Neaktivní uživatelský účet
Avatar
k.ats
Člen
Avatar
Odpovídá na Neaktivní uživatel
k.ats:17.7.2013 7:37

Pro účely této stránky není potřeba mít zaplacený hosting. Stačí free, tak nevím proč do toho vrážet ještě víc peněz, když není potřeba.

 
Nahoru Odpovědět
17.7.2013 7:37
Avatar
k.ats
Člen
Avatar
Odpovídá na Vojtěch Pospíchal
k.ats:17.7.2013 8:04

Děkuji za radu na úpravu gmaps :-) Něco jsem tam podělal, ale nevím jestli jsem to nepodělal :D Nicméně, zatím je to čekající a žádná změna se zatím nestala, tak uvidíme časem.

 
Nahoru Odpovědět
17.7.2013 8:04
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Michal Žůrek - misaz:17.7.2013 11:17

veme, jen místo :root musíš dát html nebo to rovnou spojit (tím si ale nejsem jist že to funguje, mělo by):

:root, html {
   ...
}
 
Nahoru Odpovědět
17.7.2013 11:17
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Kit:17.7.2013 11:33

Podle mne má selektor :root smysl hlavně u XML, protože tam CSS nemusí tušit, jak se jmenuje kořenový element. U HTML je to vždy html a funguje obojí, jen starší MSIE ignoruje :root.

Nahoru Odpovědět
17.7.2013 11:33
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
k.ats
Člen
Avatar
Odpovídá na Neaktivní uživatel
k.ats:19.7.2013 19:58

Tak uz by to melo jet i na IE8, sice ne vsechny efekty css3 to pobere, ale zaklad ano a neni to tak rozhasene. Myslim, ze uz od IE6 by to melo byt funkcni.

 
Nahoru Odpovědět
19.7.2013 19:58
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na k.ats
Neaktivní uživatel:19.7.2013 23:51

vypadá to fajn, jen to počítadlo bych ještě hodil do patičky :) jestli je to tvůj první web, tak smekám, asi si znova sepíšu svůj první web, mám ho bohužel v živě paměti a chodí mě strašit po nocích :D

Nahoru Odpovědět
19.7.2013 23:51
Neaktivní uživatelský účet
Avatar
k.ats
Člen
Avatar
Odpovídá na Neaktivní uživatel
k.ats:20.7.2013 8:15

Děkuji, no ono to úplně první stránka není. První stránku jsem dělal kdysi dávno na základce před cca 14 lety ve frontpage express, pak taky jednu na vysoké před 5 lety, ale nikdy jsem to neměl na webu a nikdy jsem se tím moc nezabýval. Bylo to vlastně vždycky dělané z povinnosti kvůli zápočtu :-) Teď za mnou přišel kamoš, že by chtěl udělat webovky, tak jsem po 5ti letech oprášil svoje dovednosti a za pomocí tohoto webu jsem udělal první stránky, které mají prezentovat malou firmičku :-) Je to jen zákládní html a css, nic extra. Pokud se to ale bude lidem líbit a budou chtít po mě dělat weby, tak se určitě pustím i do jiných věcí a třeba se i jednou zdokonalím :-)

 
Nahoru Odpovědět
20.7.2013 8:15
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 34 zpráv z 34.