Chceš se vypracovat až k výdělku 160.000 Kč/měsíc? Důležité je začít. Staň se programátorem díky rekvalifikačnímu kurzu! Začni dnes!
Hledáme nové posily do ITnetwork týmu. Podívej se na volné pozice a přidej se do nejagilnější firmy na trhu - Více informací.

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

Jak se ti líbí web?
3 hlasů
Přejít na web:
http://www.simek.illusion-pictures.cz
Aktivity
Avatar
Jaroslav Šimek
Neregistrovaný
Avatar
Jaroslav Šimek:16.4.2013 17:00

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
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na Jaroslav Šimek
Neaktivní uživatel:16.4.2013 17:07

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
16.4.2013 17:07
Neaktivní uživatelský účet
Avatar
Honza Bittner
Tvůrce
Avatar
Honza Bittner:16.4.2013 17:17

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
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na Honza Bittner
Neaktivní uživatel:16.4.2013 17:19

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

Nahoru Odpovědět
16.4.2013 17:19
Neaktivní uživatelský účet
Avatar
Honza Bittner
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Honza Bittner:16.4.2013 17:20

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
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovídá na Jaroslav Šimek
Petr Nymsa:16.4.2013 17:24

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
16.4.2013 17:24
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na Honza Bittner
Neaktivní uživatel:16.4.2013 17:25

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
Neaktivní uživatelský účet
Avatar
Honza Bittner
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Honza Bittner:16.4.2013 17:26

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

Nahoru Odpovědět
16.4.2013 17:26
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
jardazero
Člen
Avatar
jardazero:16.4.2013 17:34

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
Tvůrce
Avatar
Odpovídá na Jaroslav Šimek
Petr Nymsa:16.4.2013 18:08

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:16.4.2013 18:26

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
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:16.4.2013 18:28

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:16.4.2013 18:30

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
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na jardazero
Neaktivní uživatel:16.4.2013 18:31

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
Neaktivní uživatelský účet
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:16.4.2013 18:34

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:16.4.2013 18:38

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:16.4.2013 18:40

<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:16.4.2013 18:41

<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:16.4.2013 18:42 
<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
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:16.4.2013 18:42

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:16.4.2013 18:42 
<?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('[email protected]',$_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:16.4.2013 18:44

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

 
Nahoru Odpovědět
16.4.2013 18:44
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:16.4.2013 18:45

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:16.4.2013 18:49 
<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:16.4.2013 18:52

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
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:16.4.2013 18:59

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:16.4.2013 19:03

Jo bomba, díky

 
Nahoru Odpovědět
16.4.2013 19:03
Avatar
jardazero
Člen
Avatar
Odpovídá na Neaktivní uživatel
jardazero:16.4.2013 21:45

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
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:16.4.2013 21:48

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
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na Petr Nymsa
Neaktivní uživatel:16.4.2013 21:53

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

<!DOCTYPE HTML fajiejiajfeaofjeaofasf>

jenom

<!DOCTYPE HTML>
Nahoru Odpovědět
16.4.2013 21:53
Neaktivní uživatelský účet
Avatar
Neaktivní uživatel
Tvůrce
Avatar
Odpovídá na jardazero
Neaktivní uživatel:16.4.2013 21:57

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
Neaktivní uživatelský účet
Avatar
jardazero
Člen
Avatar
jardazero:16.4.2013 22:02

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

 
Nahoru Odpovědět
16.4.2013 22:02
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Petr Nymsa:16.4.2013 22:21

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:16.4.2013 22:30

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
Tvůrce
Avatar
Odpovídá na jardazero
Honza Bittner:16.4.2013 22:42

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
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Kit
Tvůrce
Avatar
Odpovídá na jardazero
Kit:17.4.2013 9:16

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
Tvůrce
Avatar
Odpovídá na Petr Nymsa
Kit:17.4.2013 9:19

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
Tvůrce
Avatar
Odpovídá na Honza Bittner
Kit:17.4.2013 9:22

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
17.4.2013 9:22
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Honza Bittner
Kit:17.4.2013 9:58

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
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:17.4.2013 15:09

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
Tvůrce
Avatar
Odpovídá na jardazero
Petr Nymsa:17.4.2013 15:10

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
Tvůrce
Avatar
Odpovídá na jardazero
Kit:17.4.2013 15:33
  • 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:17.4.2013 15:50

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:17.4.2013 15:52

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.