| Autor | Zpráva | ||
|---|---|---|---|
| Enngage Profil |
#1 · Zasláno: 5. 7. 2011, 22:46:16 · Upravil/a: Enngage
Delam na webu, kde se budou dat jednoduse vkladat clanky, hodnotit je, vest statistiky apod.. Budu na tom jeste dlouho pracovat a vyladovat detaily, Vase kritika by mi urcite hodne pomohla.
Je tam nejaky nedodelany balast a urcite spousta chyb, ale +- tam je zakladni funkcnost. Edit: Zejmena postrehy co zlepsit, nebo co je vylozene spatne me zajimaji nejvice. |
||
| Darker Profil |
#2 · Zasláno: 5. 7. 2011, 23:18:15
eknihovna.cekuj.net
Dodej testovací údaje. Design se mi zdá přehledný, statistiky zabírají moc místa (vím, dal sis na nich záležet). Titulek webu by se mohl měnit v závislosti na lokaci uživatele. Pamatovat v přihlašovacím formuláři by vypadalo lépe s velkým P. |
||
| Enngage Profil |
#3 · Zasláno: 5. 7. 2011, 23:19:25 · Upravil/a: Enngage
Diky za rychlou reakci:) Nejake data prave v tenhle okamzik doplnuji. Tim titulek mas na mysli title v meta tagu?
|
||
| Darker Profil |
#4 · Zasláno: 5. 7. 2011, 23:31:01 · Upravil/a: Darker
Enngage:
„title v meta tagu?“ Určitě ne v meta tagu :) V hlavičce. Koncepce je jednoduchá: Jsem na příspěvcích, title bude "Student - příspěvky" Registrace: "Student - registrace" A tak dále. Jo a pokud můžeš, používej prosím tady na diskusi háčky čárky. |
||
| Enngage Profil |
#5 · Zasláno: 5. 7. 2011, 23:34:29
Díky, hned zítra zjistím jak funguje jak se dá změnit ten titulek, tohohle jsem si ani vůbec nevšimnul.:)
|
||
| Darker Profil |
#6 · Zasláno: 6. 7. 2011, 00:00:26
Enngage:
„jak se dá změnit ten titulek“ Doporučuju vypisovat echem do titulku. |
||
| jenikkozak Profil |
#7 · Zasláno: 6. 7. 2011, 00:08:40
Darker:
> Jsem na příspěvcích, title bude "Student - příspěvky" > Registrace: "Student - registrace" Raději obráceně. Když má člověk otevřených více stránek jednoho webu ve více záložkách, v případě, že se mu zobrazí pouze část titulku, ví, na jaké podstránce se nachází. Enngage: 1) V IE se mi nezobrazí přihlašovací formulář. 2) Chybí klikací logo. Adresa eknihovna.cekuj.net/index.php?clanek=obsah je duplicitní k eknihovna.cekuj.net. Chtělo by to přidat podmínku, která by tomuto zabránila. 3) Vůbec nikde jsem se nedočetl, o co vlastně jde. To je nějaký blog? Hodnocení amatérských článků? Nebo seznam knížek? |
||
| Enngage Profil |
#8 · Zasláno: 6. 7. 2011, 09:46:44 · Upravil/a: Enngage
jenikkozak:
„3) Vůbec nikde jsem se nedočetl, o co vlastně jde. To je nějaký blog? Hodnocení amatérských článků? Nebo seznam knížek?“ Hodnocení amatérský článků, tohle všechno tam bude napsané na hlavní stránce, teď se zaměřuju na funkcionalitu a potom to podrobněji popíšu v informacích. Ten prihlasovaci formular je mi docela zahadou... Tak jsem to zkusil poupravit a uz se zobrazuje v IE.. jediny problem je v tom, ze IE nejspise nezvlada zaobleni bunek pomoci CSS, tak to vypada docela blbe takove hranate (po prihlaseni) |
||
| jenikkozak Profil |
#9 · Zasláno: 6. 7. 2011, 10:17:05
Enngage:
„vite cim by to mohlo byt?“ Používáš úplně nesmyslně tabulky. Tabulka se používá na data, která se čtou po řádkách, přičemž jednotlivé sloupce obsahují záznamy společných hodnot. Rozhodně nejsou na to, abys pomocí jich udělal blok v designu. Uděláš tabulku, v ní dvě buňky. Jedna je úplně prázdná, druhá obsahuje veškerá data. V ní je div, v ní další tabulka. Taky s jedinou buňkou. V ní je div, v ní form. A přitom by se to dalo udělat jen s tím formem. Svůj problém odstraníš tím, že té druhé nesmyslně vložené tabulce odstraníš atribut „width“ s hodnotou „100%“, která se tam evidentně nevejde, a tak zajíždí mimo obrazovku. |
||
| Enngage Profil |
#10 · Zasláno: 6. 7. 2011, 10:27:28
Enngage:
„2) Chybí klikací logo.“ Jediný způsob, který mě napadl jak tohle udělat je vložení další neviditelné tabulky/obrázku na pozici loga a udělat z toho odkaz, může to tak být? |
||
| jenikkozak Profil |
#11 · Zasláno: 6. 7. 2011, 10:30:38
Enngage:
„Jediný způsob, který mě napadl jak tohle udělat je vložení další neviditelné tabulky/obrázku na pozici loga a udělat z toho odkaz, může to tak být?“ Ty snad neposloucháš, co píšu. :-D Tabulek tam máš až nad hlavu. Odkaz se dělá z odkazu, ne z tabulky. Pokud chceš, aby se nechoval jako řádkový prvek, připiš mu vlastnost display:block, pak mu půjdou nastavit rozměry. |
||
| AlastorM Profil * |
#12 · Zasláno: 6. 7. 2011, 10:36:04
Enngage:
dám ti radu - zbav se těch tabulek. Je to množství kódu navíc, není to vhodné řešení, je to zastaralé a nepřehledné. Uvidíš sám, že si tím spoustu práce ulehčíš a i zdrojový kód bude přehlednější. Taky jsem ještě celkem nedávno (před půl rokem) používal tabulky na rozvržení. jenže to není a nebyl ideální způsob. |
||
| Enngage Profil |
#13 · Zasláno: 6. 7. 2011, 10:37:24
AlastorM:
„jenže to není a nebyl ideální způsob.“ A co je idealni na rozvrzeni? |
||
| jenikkozak Profil |
#14 · Zasláno: 6. 7. 2011, 10:42:37 · Upravil/a: jenikkozak
Enngage:
„A co je idealni na rozvrzeni?“ <table border="0"><tr><td valign="top"> </td><td><a href="">Text</a></td></tr></table> <a href="">Text</a> Rozdíl v délce a přehlednosti je patrný. Určitě se zbav těch jednořádkových a zároveň jednobuňkových tabulek. |
||
| AlastorM Profil * |
#15 · Zasláno: 6. 7. 2011, 10:46:35
Já používám divy - 2 nebo 3 - to úplně stačí. Jeden divy jako logo, jeden jako menu, jeden jako obsah, případně ještě 4 div pro obalení, těch 3 základních divů. Pak už jen tagy pro odstavce <p></p>, tagy pro nadpisy h1 až hx.
Třeba tato část - tvoje vrchní menu: <td align="left"> <a href="index.php?clanek=obsah">Novinky</a> <a href="index.php?clanek=zebricky">Žebříčky článků</a> <a href="index.php?clanek=zebricky_uziv">Žebříčky uživatelů</a> <a href="index.php?clanek=vyhledavani">Vyhledávání</a> <a href="index.php?clanek=faq">FAQ</a> <a href="index.php?clanek=online">Přítomní (0)</a> <td align="right"> bych udělal takto: <ul> <li><a href="index.php?clanek=obsah">Novinky</a></li> <li><a href="index.php?clanek=zebricky">Žebříčky článků</a></li> <li><a href="index.php?clanek=obsah">Novinky</a></li> <li><a href="index.php?clanek=obsah">Novinky</a></li> <li><a href="index.php?clanek=obsah">Novinky</a></li> </ul> A ve stylech bych udělal, aby to bylo na jednom řádku s mezerami mezi sebou místo tohoto: . A styly jsou další věc. Používáš tam externí soubor css, ale stále tam máš ještě dost zápisů, které by v tom css souboru být mohli také |
||
| Enngage Profil |
#16 · Zasláno: 6. 7. 2011, 10:47:15
jenikkozak:
K cemu patri ten table? trosku jsem se v tom ztratil. Jinak jsem, jak jsi poradil zkusil udelat ten klikaci odkaz jako logo. Myslel jsi to takhle (funguje to:)) : <td valign="top" width="935"><a style="display:block; height:95" href="index.php"></a> |
||
| Enngage Profil |
#17 · Zasláno: 6. 7. 2011, 10:51:22
AlastorM:
To mas pravdu, to by bylo urcite mnohem prehlednejsi. S ul a li jsem zatim vubec nepracoval a divum jsem nevenoval tak velkou pozornost, jejich dulezitost jsem objevil az docela pozde. |
||
| jenikkozak Profil |
#18 · Zasláno: 6. 7. 2011, 10:52:52 · Upravil/a: jenikkozak
Enngage:
U CSS hodnot se píšou jednotky (u nuly se psát nemusejí), takže ne „height:95“, ale „height:95px“. (Tím reaguji na tvůj výše uvedený kód.) „K cemu patri ten table?“ Podívej se, u kterých tabulek ve svém kódu nevyužíváš jejich tabulkovou strukturu. Tedy takové, které mají obsah v jediném políčku. Tyto tabulky jsou naprosto zbytečné. Můžeš místo nich použít třeba <div>, u kterého se nenastavují řádky a buňky. A teprve druhým krokem by mohlo být to, co doporučuje AlastorM, tedy odstranění i těch tabulek, které využívají více svých buněk. |
||
| Enngage Profil |
#19 · Zasláno: 6. 7. 2011, 11:04:26
Mam tam spoustu takovych tabulek.. pouzival jsem je hojně třeba na nadpisy... Jen tak na okraj - čemu to vadí, že tam mám takové tabulky, kromě toho, že mi to dalo víc práce napsat a pak to zabírá víc místa v kódu?
|
||
| jenikkozak Profil |
#20 · Zasláno: 6. 7. 2011, 11:10:08
Enngage:
„Jen tak na okraj - čemu to vadí, že tam mám takové tabulky, kromě toho, že mi to dalo víc práce napsat a pak to zabírá víc místa v kódu?“ To ti nestačí? Je to také nepřehledné, takže ti dá víc práce to dál rozvíjet. Když napíšeš <h1>Nadpis<h1>, je při strojovém čtení zdrojového kódu tomu zařízení jasné, že se jedná o nadpis první úrovně. Když dáš <table><tr><td>Nadpis</td></tr></table>, je to jen obyčejný text v tabulce. |
||
| AlastorM Profil * |
#21 · Zasláno: 6. 7. 2011, 11:29:38
Je to na tobě, ale ze své zkušenosti ti to doporučuji. Sám poznáš, že se v kódu brzy nevyznáš, jakmile doplníš obsah. Navíc s tím neuděláš vše tak, jak bys měl. třeba to logo: teď ho vidím jeden a půlkrát pod sebou - to asi nemělo být, že?
Kdybych měl náladu a udělal bych stejný web jako ty, měl bych kód minimálně o polovinu kratší, mnohem přehlednější a navíc jakékoliv úpravy by byly bez komplikací. Ale jestli tam potřebuješ ty tabulky prosím. Nechci vidět, jak budeš mít web dokončen, půl roku nešáhneš do zdrojového kódu a za půl roku budeš muset udělat nějakou úpravu, nebo budeš chtít něco přidat a budeš mít problém se v tom orientovat - k tomu ti nejvíce řekne co jsi napsal výše - "K cemu patri ten table? trosku jsem se v tom ztratil." a to tam ještě nic nemáš. Web jinak vypadá dobře, jen si prostě komplikuješ život |
||
| Enngage Profil |
#22 · Zasláno: 6. 7. 2011, 11:34:17
AlastorM:
„teď ho vidím jeden a půlkrát pod sebou - to asi nemělo být, že?“ Kde se ti to prosimte zobrazuje? V Opere, firefoxu a ted uz i v IE se zobrazuje spravne ( v IE trosku posunute, ale budiz) |
||
| AlastorM Profil * |
#23 · Zasláno: 6. 7. 2011, 11:47:25
Enngage:
V opeře 11.50, ve firefoxu 4.0 a v chrome 12.x - tedy nejnovější verze - IE9 bez problémů, ale to nic neznamená |
||
| Bubák Profil |
#24 · Zasláno: 6. 7. 2011, 13:05:02
Enngage:
„Kde se ti to prosimte zobrazuje?“ Na monitoru. tím chci naznačit, že problém souvisí s rozlišením, mi a třebas AlastorMovi se položky formuláře zobrazují "hezky" pod sebou, takže na výšku zabrají celkem dost místa. Proč sis nezkusil zmenšit okno, nemusíš jak peta na 500×500px, stačí v mezích rozumných velikostí, zda se ti web někde nerozpadá? |
||
| Enngage Profil |
#25 · Zasláno: 6. 7. 2011, 13:32:38
Bubák:
Zkousel jsem to v Opere, v ramci moznosti to vypada pouzitelne a nerozpada se. |
||
| AlastorM Profil * |
#26 · Zasláno: 6. 7. 2011, 14:08:50
Enngage:
TADY uvidíš, jak se tvůj web zobrazuje v několika prohlížečích - podívej se zejména na screenshot z opery, chromu a firefoxu, jak vypadá tvé logo |
||
| Enngage Profil |
#27 · Zasláno: 6. 7. 2011, 14:23:57
Diky, tahle stranka se mi bude do budoucna hodit. Skoda jen, ze nevim, kde je chyba, resp. jak ji napravit... Mozna zkusim cele to logo predelat jen za pomoci tech divu, jestli to nejak pujde.
|
||
| jenikkozak Profil |
#28 · Zasláno: 6. 7. 2011, 14:42:58 · Upravil/a: jenikkozak
Enngage:
„Skoda jen, ze nevim, kde je chyba, resp. jak ji napravit“ Piš prosím s diakritikou. Opakování obrázku na pozadí zabráníš pomocí vlastnosti background-repeat s hodnotou no-repeat. „Mozna zkusim cele to logo predelat jen za pomoci tech divu, jestli to nejak pujde.“ Třeba: <div style="background: #f00 url('logo.png') left top no-repeat; overflow: hidden">
<a href="/" style="display:block; float:left; width:50%; height: 51px">Eknihovna</a>
<form style="float: right; width: 50%; height: 51px"></form>
</div> |
||
| AlastorM Profil * |
#29 · Zasláno: 6. 7. 2011, 16:11:03
jenikkozak:
a proč takto? Když už tam má nějaký ten css soubor, tak by ho měl používat a stačí pouze: <div id="logo"> <a href="/"><img src="logo.png"></a> <form></form> </div> a ostylovat v css souboru, který už stejně má. Ale to už by bylo lepší udělat takto celou stránku znovu. Když odmyslím tu grafiku, tak by to vypadalo zhruba takto - příklad - je to jen kostra bez grafiky a pořádného nastylování |
||
| jenikkozak Profil |
#30 · Zasláno: 6. 7. 2011, 16:51:44
AlastorM:
„a proč takto?“ Protože sis nevšiml dvou věcí: 1) Vedle odkazu má být také formulář. Když tam strčíš přes celou šířku obrázek s odkazem, budeš nad něj muset formulář pozicovat. 2) Úmyslně jsem použil inline zápisy, aby Enngage viděl, co se k čemu vztahuje. Přepsání mých stylů do externího souboru je jednodušší, než z tvého layoutu s pevnými rozměry udělat flexibilní. |
||
|
Téma pokračuje na další straně.
|
|||
0