Kritika webu http://www.teamspeak4fun.eu
http://www.teamspeak4fun.eu
V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.
Člen
Zobrazeno 15 zpráv z 15.
//= Settings::TRACKING_CODE_B ?> //= Settings::TRACKING_CODE ?>
V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.
Já osobně bych tu stránku "zcvrkl" aby byla menší, na mém notebooku se vleza asi jenom 3/4 obsahu z toho 1/2 je hlavička a menu. Já bych zmenšil mezery mezi jednotlivými částmi a hlavičku bych taky zmenšil.
V kódu máš nějak moc section.
oprav si kód: <br><br<br>
jinak se mi ta stránka líbí. Je pravda že bych trochu uvažoval o těch
velikostech. Možná by stálo za to se zamyslet nad tím že hodně lidí má
nízké rozlišení nebo třeba tablety a nebaví je moc scrolovat. Ale při
rozlišení 1650 x 1050 je to ale super
Patičce bych dal stejnou barvu jako fixnímu menu nahoře, případně
obráceně Celý text v
patičce bych více spíš vycentroval a zvětšil ikony sociálních sítí, v
podstatě mi to nevadí ale ani tak jak to je. S velikostí stránky problém
nemám.
Jinak se mi to celkem líbí, v pravidlech bych ale použil odstíny té modré
co je v hlavičce, ikdyž nevím jak to bude vypadat.
Co se týče kódu je neodsazený a nepřehledný, jak říkal David Hartinger je tam dost section, a v pravidlech sis mohl ušetřit tu práci stylováním každého červeného odstavce zvlášť a vyřešit to s classem.
EDIT: neodolal jsem http://validator.w3.org/check?… je tam toho teda celkem dost
Takhle by to bylo lepší?
<div id="container">
<div id="banner">
<img src="images/banner.png" alt="TeamSpeak4fun.eu"> </a>
</div>
<div class="obsah">
<article class="uvod">
<header>
<h1>O nás</h1>
</header>
<section>
<p>Provozujeme TeamSpeak3 Server pro hráče browserové hry DarkOrbit , ale i jiných her a komunit.<br> Naše TS funguje na systému registrací , takže můžete mít klid , že sem nikdo jentak nevlítne a nezačne dělat nepořádek.<br> Vedení TS se k jeho chodu staví profesionálně a snaží se svoje zkušenosti vložit i do začínajích adminů. <br>Server je stabilní bez častých výpadků a jediné co za poskytnutý prostor požadujeme je dodržování <a href="pravidla.html">pravidel</a> , což v praxi znamená dodržování zásad slušného chování.
</p>
</section>
</article>
</section>
</div>
</div>
<div class="boxplace">
<article class="box1">
<header>
<h1> TS3 DarkOrbit</h1>
</header>
<section>
<p>Poskytujeme prostor pro komunikaci mezi hráči českých i jiných serverů hry DarkOrbit. Zakládáme nové místnosti pro klany i sekce pro servery.</p>
<a href="ts3server://89.29.14.194?port=9987"><span class="button">Připojit</span></a>
</section>
</article>
<article class="box2">
<header>
<h1>Public</h1>
</header>
<section>
<p>Krom hry DarkOrbit rádi přivítáme i hráče jiných her či členy komunit. Pro tyto potřeby existuje na našem serveru sekce.</p>
<a href="ts3server://89.29.14.194?port=9987"><span class="button">Připojit</span></a>
</section>
</article>
<article class="box3">
<header>
<h1>TS3 Forum</h1>
</header>
<section>
<p>Na fóru zveřejňujeme oficiální oznámení. Můžete zde žádat o UnBan , odebrání trestu či podat stížnost na uživatele nebo admina. </p>
<a href="http://forum.teamspeak4fun.eu"><span class="button">Přejít</span></a>
</section>
</article>
</div>
<footer>
<div class="patickavlevo">
<a href="http://facebook.com/teamspeak4fun.eu"><img src="images/facebook_32.png" class="opacity"></a> <a href="http://www.youtube.com/user/TeamSpeak4funEu" class="opacity"><img src="images/youtube_32.png"></a> <a href="mailto:[email protected]"><img src="images/email_32.png" class="opacity"></a>
</div>
<div class="copyright">
<p>Web by TS3Expert.maweb.eu (c) 2013</p>
</div>
</footer>
Když box1, box2 i box3 vypadají stejně, měla by to být jen jedna class. K čemu ji definovat 3×? Vlastně uvnitř "boxplace" jsou zbytečné.
na tie boxi pouzi nieco taketo
margin: 0px;
padding: 0px;
box-sizing: border-box;
.box {
width:274px;
float:left;
}
.box+.box {
margin-left:26px;
}
Díky všem...
No tak jsem to trochu poupravil , kouknul by jste se mi na to někdo prosím jestli jsem zas něco nepokazil? Doufám , že vás moc neprudím , ale nejlépe se učím praxí.
Upravil jsem ten kod a trochu jsem to zmenšil. Přemýšlím , že bych ještě napsal mobilní verzi...
Po grafickej stranke nebudem hodnotit, ale chcelo by to zmodernizovat.
Kazdy img co ta mas potrebuje alt ** popis to je povinne!!! a menu co
mas - prve musis dat li a az potom odkaz **a href
opacne to je chyba
<li><a href="index.php" title="O nás">O nás</a></li>
a ked davas plavuce divi tak si urci triedu
.left {
float : left;
}
podobne aj pre right
a potom napises
<div class"box left"></div>
Doplnil bych, že obrázky, u kterých mě nenapadá žádný vhodný "alt", nejspíš nepatří do HTML ale do designu, tedy do CSS.
Za vhodné alty nepovažuji "šipka", "pozadí", "roh" apod. Musí se týkat obsahu stránky.
Díky dost jste mi pomohli se rozhýbat... Myslím , že by to již mělo být správně.
Pořád je trošku zavádějící název třídy těch boxů dole, proč box-left ? Přejmenoval bych to na box, samozřejmě je to hovadina ale když už, tak už
Ok , díky... A co říkáš na design? Jelikož nemám geneticky dané cítění pro design, tak se ho snažím nějak natlouct do hlavy sám
nemyslel som box-left ale box left. Lebo potom nemusi davat v css vsade float:left or float: right, ale urobi si dve triedy pre float a ma. POtom v HTML da iba
<div class"box left"></div>
alebo right mas to univerzalnejsie a menej kodu v css.
Zobrazeno 15 zpráv z 15.