Kritika webu http://www.simek.illusion-pictures.cz

HTML a CSS Kritika webů - ukažte se! http://www.simek.illusion-pictures.cz American English version English version

Jak se ti líbí web?
Celkové hodnocení (3 hlasů):
44444
Přejít na web:
http://www.simek.illusion-pictures.cz
Avatar
Jaroslav Šimek
Neregistrovaný
Avatar
Jaroslav Šimek:

Moderní grafika & design - Jaroslav Šimek
http://www.simek.illusion-pictures.cz

Jsem vděčný za každý názor.

 
Odpovědět 16.4.2013 17:00
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Jaroslav Šimek
Jiří Gracík:

Jednoduchý, příjemný, ideální pro jakékoliv rozlišení. Validátor už takovou euforii nezažívá, 11 erorů je poměrně dost. Hodil by se ještě nějaký stručný text - Jsem ten a ten, dělám tohle s tímhle takovou a takovou dobu, v součastnosti se zaměřuji hlavně na ... víš co myslím. Oprav erory a bude to skvělý ;)

http://validator.w3.org/check?…

Nahoru Odpovědět  +1 16.4.2013 17:07
Creating websites is awesome till you see the result in another browser ...
Avatar
Honza Bittner
Redaktor
Avatar
Honza Bittner:

Možná bys mohl nastavit dole v OBSAHU ZPRÁVY max-width ... pak ti nebude skákat tlačítko

Nahoru Odpovědět 16.4.2013 17:17
Ptejte se mě na cokoli na https://github.com/HoBi/ama a followujte mě na Twitteru https://twitter.com/tenhobi. :-)
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Honza Bittner
Jiří Gracík:

U obsahu zprávy mi nic neskáká (G Chrome)

Nahoru Odpovědět 16.4.2013 17:19
Creating websites is awesome till you see the result in another browser ...
Avatar
Honza Bittner
Redaktor
Avatar
Odpovídá na Jiří Gracík
Honza Bittner:

když šáhneš na pravý dolní roh a začneš tahat doprava tak se šířka zvětšuje a tlačítko se posune pod obsah ..
zvětšování výšky nevadí ..

proto bych nastavil MAX-WIDTH stejný jako MIN-WIDTH

Editováno 16.4.2013 17:21
Nahoru Odpovědět 16.4.2013 17:20
Ptejte se mě na cokoli na https://github.com/HoBi/ama a followujte mě na Twitteru https://twitter.com/tenhobi. :-)
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na Jaroslav Šimek
Petr Nymsa:

Krásný web, ale přeci mám jednu vítku. Ikony sociálních sítí by se při přejetí mohly přebarvit do originální podoby a u kontaktního formuláře bych pozměnil barvy. Celý web je do tmava a náhle tam máme zelenou a modrou :). Jinak super práce ! :)

Nahoru Odpovědět  +1 16.4.2013 17:24
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Honza Bittner
Jiří Gracík:

Jo takhle, toho jsem si ani nevšiml že jde. Přijde mi to zbytečný aby se to dalo zvětšovat, natož takhle. Stejně si toho většina lidí jako já nevšimne ;)

Nahoru Odpovědět 16.4.2013 17:25
Creating websites is awesome till you see the result in another browser ...
Avatar
Honza Bittner
Redaktor
Avatar
Odpovídá na Jiří Gracík
Honza Bittner:

no já bych zakázal ten pohyb do stran .. na výšku to je jedno :)

Nahoru Odpovědět 16.4.2013 17:26
Ptejte se mě na cokoli na https://github.com/HoBi/ama a followujte mě na Twitteru https://twitter.com/tenhobi. :-)
Avatar
jardazero
Člen
Avatar
jardazero:

Děkuji za postřehy, nejsem programátor, tak mám občas s tím kódováním stránky problém, nicméně max-width jsem zvládl nastavit. Děkuji. Ještě bych rád něco udělal s tím light boxem, když kliknu na referenci aby se neroztahovala přez rámec monitoru (ocení uživatelé s menším rozlišením.

 
Nahoru Odpovědět 16.4.2013 17:34
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na Jaroslav Šimek
Petr Nymsa:

S tím lightboxem. Zobrazuješ obrázek v určité velikosti pokud se nepletu. Takže jedno, horší řešení, zmenšit obrázek anebo nejspíš přes JS vypočítat potřebnou velikost a nastavit

Nahoru Odpovědět 16.4.2013 18:08
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
jardazero:

Zmenšovat obrázek striktně se mi nechce (takto si mohou majitelé většího rozlišení vychutnat originální rozměry reference) Potřeboval bych takového aby se maximální šířka lightboxu upravovala dle rozlišení obrazovky. Tak aby nikdy nepřesáhla právě toto rozlišení.

 
Nahoru Odpovědět 16.4.2013 18:26
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

Což je druhé řešení. Bohužel vím jak to udělat "pseudokódem" ale přímo v JS ne, snad se ozvou JavaScriptové :)

Nahoru Odpovědět 16.4.2013 18:28
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
jardazero:

Dále je tam také nedostat v odesílacím formuláři. Ten se odešle i když není vyplněno žádné pole. Potřeboval bych aby se odeslal jen v případě když jsou vyplněna všechna pole, nevíte jak na to?

 
Nahoru Odpovědět 16.4.2013 18:30
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na jardazero
Jiří Gracík:

když nám ukážeš kód jak to posíláš tak ti určitě poradíme ;)

Nahoru Odpovědět 16.4.2013 18:31
Creating websites is awesome till you see the result in another browser ...
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

POšli kód, jak se říká. Jinak základním řešením je nastavit inputůn atribut

required="required"

. To ti zajistí částečnou ochranu. Částečnou proto, že uživatel může zadat blbost, zajistí to pouze, že musí to pole být něčím vyplněné. Správnou validci budeš msuet provést už na straně serveru. Ještě jedna otázka. Máš zajištěnou anti-spam ochranu ?

Nahoru Odpovědět 16.4.2013 18:34
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
Odpovídá na Petr Nymsa
jardazero:

děkuji v programování jsem opravdu začátečník, ochranu proti spamu nemam..

 
Nahoru Odpovědět 16.4.2013 18:38
Avatar
jardazero
Člen
Avatar
jardazero:

<div id="form">
<form action="odesli­mail.php" method="post">
<input name="predmet" value="Vaše jméno" onfocus="if(this­.value=='Vaše jméno') this.value=''" type="text" id="jmeno">
<input name="od" type="text" value="E-mail" onfocus="if(this­.value=='E-mail') this.value=''" id="mail">

<textarea onfocus="igno­reKeys=true; if(this.inner­HTML=='Obsah zprávy') {this.innerHTML=}" onblur="ignoreKeys=false; if('this.innerHTML==) {this.innerHTML='Ob­sah zprávy'}" name="zprava" id="obsahz">Obsah zprávy</textarea>
<input type="submit" id="odeslat" value="ODESLAT">
</div>

 
Nahoru Odpovědět 16.4.2013 18:40
Avatar
jardazero
Člen
Avatar
jardazero:

<div id="form">
<form action="odesli­mail.php" method="post">
<input name="predmet" value="Vaše jméno" onfocus="if(this­.value=='Vaše jméno') this.value=''" type="text" id="jmeno">
<input name="od" type="text" value="E-mail" onfocus="if(this­.value=='E-mail') this.value=''" id="mail">

<textarea onfocus="igno­reKeys=true; if(this.inner­HTML=='Obsah zprávy') {this.innerHTML=}" onblur="ignore­Keys=false; if('this.inner­HTML==) {this.innerHTML='Ob­sah zprávy'}" name="zprava" id="obsahz">Obsah zprávy</textarea>
<input type="submit" id="odeslat" value="ODESLAT">
</div>

Editováno 16.4.2013 18:42
 
Nahoru Odpovědět 16.4.2013 18:41
Avatar
jardazero
Člen
Avatar
jardazero:
<div id="form">
<form action="odeslimail.php" method="post">
<input name="predmet" value="Vaše jméno" onfocus="if(this.value=='Vaše jméno') this.value=''" type="text" id="jmeno">
<input name="od" type="text" value="E-mail" onfocus="if(this.value=='E-mail') this.value=''" id="mail">

<textarea onfocus="ignoreKeys=true; if(this.innerHTML=='Obsah zprávy') {this.innerHTML=}" onblur="ignoreKeys=false; if('this.innerHTML==) {this.innerHTML='Obsah zprávy'}" name="zprava" id="obsahz">Obsah zprávy</textarea>
<input type="submit" id="odeslat" value="ODESLAT">
</div>
 
Nahoru Odpovědět 16.4.2013 18:42
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

Prosím rychle zedituj zprávy a kódy vlož do tagu code, tlačítko dvou papírků zprava nad polem pro zprávu . Děkujeme !

Editováno 16.4.2013 18:42
Nahoru Odpovědět 16.4.2013 18:42
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
jardazero:
<?php
if ( isset($_POST['zprava'])         //existuje prom_nná se zprávou?
     && ($_POST['zprava']!='')      //a je v ní n_co? (prázdné zprávy nechceme)
     && isset($_POST['od'])         //existuje prom_nná s adresou? (bez adresy se obejdeme)
     && isset($_POST['predmet']) )  //existuje prom_nná s p_edm_tem? (p_edm_t také není nutný)
mail('simek@illusion-pictures.cz',$_POST['predmet'],$_POST['od'].': '.$_POST['zprava'],'Content-Type: text/plain; charset="windows-1250"');
header('Location: index.html');


?>
 
Nahoru Odpovědět 16.4.2013 18:42
Avatar
jardazero
Člen
Avatar
jardazero:

omlouvám se, trošku jsem to zaspamoval xD

 
Nahoru Odpovědět 16.4.2013 18:44
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

No inputům nastav ty atributy jak jsem psal výše. A nemáš vůbec antispamovou ochranu, když někdo bude chtít, udělá ti v emailu velkou neplechu. Stačí dát další input a třeba se zeptat jaký je rok. Když bude opdpověď správná, odešleš email, jinak ne.

Nahoru Odpovědět 16.4.2013 18:45
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
jardazero:
<div  id="form">
        <form action="odeslimail.php" method="post">
        <input name="predmet" required="required" value="Vaše jméno"  onfocus="if(this.value=='Vaše jméno') this.value=''" type="text"  id="jmeno">
        <input name="od" required="required" type="text" value="E-mail"  onfocus="if(this.value=='E-mail') this.value=''" id="mail">

        <textarea onfocus="ignoreKeys=true; if(this.innerHTML=='Obsah zprávy') {this.innerHTML=''}" onblur="ignoreKeys=false; if('this.innerHTML=='') {this.innerHTML='Obsah zprávy'}" name="zprava" required="required" id="obsahz">Obsah zprávy</textarea>
        <input type="submit" id="odeslat" value="ODESLAT">
        </div>
 
Nahoru Odpovědět 16.4.2013 18:49
Avatar
jardazero
Člen
Avatar
jardazero:

jo tak uz to funguje..sice jsou pole automaticky vyplnena takze kdyz nekdo klikne na odeslat a neklikne predtim do formulare stejne se to odesle..ale to snad nevadi

 
Nahoru Odpovědět 16.4.2013 18:52
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

Místo value jako nápovědu co tam má být použij placeholder, je to všude na webech. Když je input prázdný zobrazí se tato nápověda, když začneš psát nápověda zmizí

Nahoru Odpovědět 16.4.2013 18:59
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
jardazero:

Jo bomba, díky

 
Nahoru Odpovědět 16.4.2013 19:03
Avatar
jardazero
Člen
Avatar
Odpovídá na Jiří Gracík
jardazero:

Podařilo se mi zredukovat počet chyb na 5..jak to tak vypadá validátoru se nelíbí tagy PLACEHOLDER a REQUIRED, které mam u formuláře. Nevíš co stím?
http://validator.w3.org/check?…

 
Nahoru Odpovědět 16.4.2013 21:45
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

Mám dojem že tyto dva atributy jsou záležitostí až HTML5, tedy deklarovat stránku jako HTML 5.

Nahoru Odpovědět 16.4.2013 21:48
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Petr Nymsa
Jiří Gracík:

Jop, zirko má pravdu, první řádek, místo

<!DOCTYPE HTML fajiejiajfeaofjeaofasf>

jenom

<!DOCTYPE HTML>
Nahoru Odpovědět 16.4.2013 21:53
Creating websites is awesome till you see the result in another browser ...
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na jardazero
Jiří Gracík:

Když si přečteš popis toho eroru, tak ti to většinou dojde v čem je chyba, tam je napsaný něco jako špatnej document type :)

Nahoru Odpovědět 16.4.2013 21:57
Creating websites is awesome till you see the result in another browser ...
Avatar
jardazero
Člen
Avatar
jardazero:

kdyz tam prcnu <!DOCTYPE HTML> vyhazi to chyb 27 :D

 
Nahoru Odpovědět 16.4.2013 22:02
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na Jiří Gracík
Petr Nymsa:

Ono zase deklarovat to jako HTML 5 když tam neí ani jedinej HTML 5 tag tak není úplně růžový :D

Nahoru Odpovědět 16.4.2013 22:21
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
jardazero
Člen
Avatar
jardazero:

No tak tam hold nechám 5 chyb..on se svět nezboří

 
Nahoru Odpovědět 16.4.2013 22:30
Avatar
Honza Bittner
Redaktor
Avatar
Odpovídá na jardazero
Honza Bittner:

5 ERRORŮ je docela špatné ... navíc pro HTML5 máš 17 errorů !!
http://validator.w3.org/check?…

Editováno 16.4.2013 22:43
Nahoru Odpovědět 16.4.2013 22:42
Ptejte se mě na cokoli na https://github.com/HoBi/ama a followujte mě na Twitteru https://twitter.com/tenhobi. :-)
Avatar
Kit
Redaktor
Avatar
Odpovídá na jardazero
Kit:

Asi je to vhodný okamžik pro opravu těch 27 chyb.

Nahoru Odpovědět 17.4.2013 9:16
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Kit
Redaktor
Avatar
Odpovídá na Petr Nymsa
Kit:

Deklarovat HTML 5 je určitě lepší, než HTML 4.01 nebo XHTML. Používání nových tagů není povinné.

Nahoru Odpovědět 17.4.2013 9:19
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Kit
Redaktor
Avatar
Odpovídá na Honza Bittner
Kit:

Jeden profesionální web má "88 Errors, 119 warning(s)" a provozovatele to evidentně netrápí ani po mém upozornění.

Nahoru Odpovědět  +1 17.4.2013 9:22
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Kit
Redaktor
Avatar
Odpovídá na Honza Bittner
Kit:

Na té stránce je také hromada chyb, které validátor nehlásí, protože je ani nemůže hlásit.

Nahoru Odpovědět  +1 17.4.2013 9:58
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
jardazero
Člen
Avatar
jardazero:

HTML 5 má zřejmě problém s mým zápisem obrázků s lightboxem http://validator.w3.org/check?…
Co stím?

 
Nahoru Odpovědět 17.4.2013 15:09
Avatar
Petr Nymsa
Redaktor
Avatar
Odpovídá na jardazero
Petr Nymsa:

To není HTML 5, chybí ti uzavírací tag tj

<img src="a.jpg" ...... />
Nahoru Odpovědět 17.4.2013 15:10
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
Kit
Redaktor
Avatar
Odpovídá na jardazero
Kit:
  • Keyword copyright is not registered - to snad nemusím překládat
  • rel="lightbox[ro­adtrip]" - takový atribut rel není definován. Proč ho nevyhodíš?
  • u obrázků ti chybí povinný atribut alt="..."
Nahoru Odpovědět 17.4.2013 15:33
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
jardazero
Člen
Avatar
jardazero:

Můj zápis vypadá takto

<div  id="citybike"><a href="images/citybike/citybike_big.jpg" rel="lightbox[roadtrip]" title="CITY BIKE - Moderní grafika"></a>
        </div>

img src= tam nemam..

 
Nahoru Odpovědět 17.4.2013 15:50
Avatar
jardazero
Člen
Avatar
jardazero:

rel="lightbox[ro­adtrip]"
tento atribut tam mám proto, aby se reference otevřela v lightboxu..

 
Nahoru Odpovědět 17.4.2013 15:52
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 44 zpráv z 44.