[www] kodowanie Opcje

[Artu123]

Witam!

Mam do oceny tylko moje kodowanie . Lay nie ja robiłem więc się nie czepiajcie . Co poprawić itp.. ? Mam pytanie, ile w tej chwili na moje umiejętności mogę brać za kodowanie pod xhtml/css ? Na ile oceniacie .

Edit://

Tutaj link do kodowania:

http://test.grafa.netarteria.eu/projekty/umaping/

Ten post edytował Artu123 19.04.2009, 14:03:42

[kantek]

Kod ładny i czytelny. Nie podoba mi się tylko, że masz dużo definicji stylu w kodzie html,
chodzi mi o <div style="...."> od tego jest css.

Takie coś mi się nie podoba:

[HTML] pobierz, plaintext 
<div class="clear" style="height:4px;"></div>
<div style="text-align:center;"><input type="reset" value="Resetuj" class="submit" />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<input type="submit" value="Zaloguj" class="submit" /></div>
[HTML] pobierz, plaintext 

div dajesz pusty który ma 4px wysokości - możesz to zrobić marginem innego divu, a czy
ie6 się nie wykrzacza na tym bez overflow:hidden (nie mam teraz jak sprawdzić) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

i te spacje to samo

Ogólnie : 9/10 (też robię wcięcia w css (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) )

Pozdrawiam

Ten post edytował kantek 19.04.2009, 16:38:20

[Kildyt]

Tytuł jest jednym z najważniejszych rzeczy w pozycjonowaniu. Ten nic nie mówi.
Puste div'y są brzydkie. Po co podajesz alt'y jeżeli pozostawiasz je puste? Zielone światełko w walidatorze za wszelką cenę to bezsens.
Jak już kantek napisał: za dużo stylów w html-u.
Zacznij używać label-ów.

[HTML] pobierz, plaintext 
       
[HTML] pobierz, plaintext 

Bardzo źle to wygląda.
Używasz h3, gdy nie ma h2.
Jeżeli w całym znaczniku używasz wielkich liter to zastosowałbym polecenie w css-ie, żeby się nie męczyć z Caps Lock'iem. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ogólnie za dużo div-ów. Całkiem fajnie. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[webber]

"style="padding-top:1px;""

Styl inline.

"id="NadTop", class="NadTopMenu" - już i mi zwrócono uwagę. Nie używaj takich nazw. Użyj "menu" "subMenu" "headerMenu".

<b>Sprite</b> -> <strong> (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Kod logowania, formularza jest TRAGICZNY. Google.pl -> "clean forms xhtml" "forms web standards" (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Czemu nie używasz atrybutu "alt"?

[HTML] pobierz, plaintext 
<div class="PanelPadding">
 
								<ul class="NawigacjaPanel">
 
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
 
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
 
								</ul>
 
								<div class="clear"></div>
 
							</div><!-- Koniec .PanelPadding -->
[HTML] pobierz, plaintext 

Tu zamiast dawać klasę do UL mogłeś się odwołać do tego poprzez div.PanelPadding ul { }


Na IE 8 rozjeżdża się Tobie menu na topie.

Ile możesz brać? Na zlecenia.przez.net jakieś 200 od strony - tak myślę. Profesjonalna robota to to jeszcze nie jest (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

6/10

Tytuł jest jednym z najważniejszych rzeczy w pozycjonowaniu. Ten nic nie mówi.

Toć to nie jego sprawa jaki będzie tytuł strony!

Jak już kantek napisał: za dużo stylów w html-u.

sęk w tym że wcale ich tam nie powinno być

Puste div'y są brzydkie.

Choć skuteczne... :/

Ten post edytował webber 19.04.2009, 18:22:49

[Artu123]

"style="padding-top:1px;""

Styl inline.

"id="NadTop", class="NadTopMenu" - już i mi zwrócono uwagę. Nie używaj takich nazw. Użyj "menu" "subMenu" "headerMenu".

<b>Sprite</b> -> <strong> (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Kod logowania, formularza jest TRAGICZNY. Google.pl -> "clean forms xhtml" "forms web standards" (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Czemu nie używasz atrybutu "alt"?

[HTML] pobierz, plaintext 
<div class="PanelPadding">
 
								<ul class="NawigacjaPanel">
 
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
 
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
									<li><a href="#">Strona Główna</a></li>
 
								</ul>
 
								<div class="clear"></div>
 
							</div><!-- Koniec .PanelPadding -->
[HTML] pobierz, plaintext 

Tu zamiast dawać klasę do UL mogłeś się odwołać do tego poprzez div.PanelPadding ul { }


Na IE 8 rozjeżdża się Tobie menu na topie.

Ile możesz brać? Na zlecenia.przez.net jakieś 200 od strony - tak myślę. Profesjonalna robota to to jeszcze nie jest (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

6/10



Toć to nie jego sprawa jaki będzie tytuł strony!



sęk w tym że wcale ich tam nie powinno być



Choć skuteczne... :/

Za takie np. kodowanie 200zł ? (IMG:http://forum.php.pl/style_emoticons/default/sciana.gif)

Ja za to wziąłem 50zł (IMG:http://forum.php.pl/style_emoticons/default/aaevil.gif) (IMG:http://forum.php.pl/style_emoticons/default/haha.gif)

[mike]

Ja za to wziąłem 50zł (IMG:http://forum.php.pl/style_emoticons/default/aaevil.gif) (IMG:http://forum.php.pl/style_emoticons/default/haha.gif)

Mało wziąłeś. Bo tyle kosztuje amatorszczyzna.
To co dałeś tu do oceny niewiele różni się od wszystkich poprzednich "prac". Nie potrafisz dokonać prostego postępu.

[webber]

Więc nie masz szacunku do własnej pracy.

No chyba że pociąłeś to w godzinę (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

IT to nie branża w której chodzi o to by zarobić sobie na piwko, no chyba że traktujesz to jako hobby ;p

Staraj się wyeliminować używanie stylów wewnątrz kodu html, staraj się jak najmniej używać class (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował webber 19.04.2009, 18:32:50

[Artu123]

Ja robię tak:

dodaje w css klasę clear i później w html dodaj div z klasa clear i dodaje do tego style i ustalam height tylko ile chce . Robie style inline no, ale nie tworzę nowych klas . Co lepsze ?

Tworzyć nową klasę, czy dodawać do style wysokość ?

[webber]

Ja robię tak:

dodaje w css klasę clear i później w html dodaj div z klasa clear i dodaje do tego style i ustalam height tylko ile chce . Robie style inline no, ale nie tworzę nowych klas . Co lepsze ?

Tworzyć nową klasę, czy dodawać do style wysokość ?

Aaa tu nie chodzi o clear, tu chodzi o typowe marginy itd.

Co do Twojego pytania, nie wiem. Ale myślę że klasę. Wtedy zamiast zmieniać w wielu miejscach to i szukać po całym dokumencie otwierasz 1 plik .css masz wydzieloną strefę z wszystkimi "clear`ami" i sobie je tam edytujesz.

[Kildyt]

Mało wziąłeś. Bo tyle kosztuje amatorszczyzna.

Może z allegro wziął zlecenie? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

<b>Sprite</b> -> <strong> (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Tylko nie popadajmy w paranoję. Nie wszystkie "b" powinny być strong'iem. Strong do zaznaczenia ważniejszej treści, a b do designu.

Toć to nie jego sprawa jaki będzie tytuł strony!

No tak, ale dobre praktyki trzeba pokazywać na każdym kroku. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

sęk w tym że wcale ich tam nie powinno być

Ja wstawiam style gdy dany element występuje dość rzadko i jest ich dosyć mało. Nie ma sensu zaśmiecać nim css-a.

Choć skuteczne... :/

No tak. Dzisiaj nawet zauważyłem, że sam takowych używam. Czasami niestety brakuje odpowiedniego znacznika. Postaram się sam siebie zmobilizować do unikania tego typu sytuacji. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[webber]

Ja wstawiam style gdy dany element występuje dość rzadko i jest ich dosyć mało. Nie ma sensu zaśmiecać nim css-a.

Hm, ale zauważ że jednak masz łatwiejszy dostęp do tego, nie nazwałbym tego zaśmiecaniem .css

Wtedy jak ktoś ma edytować cokolwiek po Tobie otwiera sobie plik .css, uruchamia Web Developer do FF i sobie edytuje wszystko jak mu się podoba jedynie za pomocą .css :]

Ale poczekajmy na opinię kogoś bardziej zaawansowanego, czy stosować styl inline w niektórych przypadkach czy tworzyć klasę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Artu123]

w Alt mam wpisywać co to za obrazek ? np. Logo ?

[webber]

w Alt mam wpisywać co to za obrazek ? np. Logo ?

http://www.queensu.ca/www/wsaguide/development/alt.html

To Tobie pomoże (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[drake88]

Według standardów Twoje kodowanie jest jak najbardziej zgodne z xHTML.

[piotrooo89]

Według standardów Twoje kodowanie jest jak najbardziej zgodne z xHTML.

co nie znaczy że jest semantyczne. ~mike kiedyś powiedział że nawet jeśli gówno opakuje się w super paczke dalej będzie gównem (czy coś koło tego).

PS. bez aluzji.