Forum PHP.pl

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

Witam !

Pociąłem ten lay i zakodowałem go pod xhtml . Zapraszam do oceny kodu .

mam tu coś co mnie zainteresowało :

- czcionka 0 ?

i po co takie cuda wlepiac? duzo tego masz.
- ...


w IE w textarea mam scrolla

Nie wiesz co to jest clear:both; , po za tym jakąś przerwę między postem muszę zrobić .



Taki mały bug był i to musiałem dodać, aby poprawnie działało



Tutaj co jest źle zrobione ?

[PHP] pobierz, plaintext 
<?php
<div class="ShoutboxTrescNick">
<a href="#">Artu123</a>
</div>
<div class="ShoutboxTrescData">
07.05/20:23
</div>
?>
[PHP] pobierz, plaintext 

a nie daloby rady przerobic tego na spany albo p ?
A nie ciagle walisz tymi divami. Mozesz dzieki temu zastosowac dziedziczenie i ograniczysz o polowe dlugosc kodu

[PHP] pobierz, plaintext 
<?php
<div class="divnr1">
<span>Moj nick</span>
<p>jakas data</p>
</div>
?>
[PHP] pobierz, plaintext 

w css:

[PHP] pobierz, plaintext 
<?php
.divnr1 {...}
.divnr1 span {...}
.divnr1 p {...}
?>
[PHP] pobierz, plaintext 

zamiast tego mozesz dac margin-top lub cos w tym stylu

A właśnie się tu mylisz .

post

przerwa

post

przerwa

post

Tak musi być, a jakbym dodał margin to było by:

przerwa

post

przerwa

post

przerwa

post


A i nie zadawaj pytania dlaczego

Dać to w ul, li.



Odwołać się do p poprzez #KtoChat p { font-weight:bold; }, bo tekst "Chatują" nie wydaje się ważny dla wyszukiwarek



Skoro dajesz klasę to po co width i height pisane inline? Dlaczego resztę nazw piszesz stylem Camel a tutaj KRZYCZĄC?



Po co w p?



Linki jako listę.

Podsumowując. Po raz kolejny Twoje ulubione słowo to "class" oraz "inline".

Więc się przyczepie ponownie...

Po każdym poście dajesz



i TUTAJ ! właśnie powineneś dać w postach margin-top:30px; zamiast zaśmiecać!





A to po co? po co dokladasz top i down obrazka? zrób to jako jeden obrazek.




W tym szablonie masz 18 divów CLEAR.

Nie chce widzieć jak wyglądają pocięte przez Ciebie szablony stron...




Nie chce się zagłębiać dalej w Twój kod. Pochwalić mogę za komentarze w html i css będę musiał takie coś zastosować u siebie bo często się gubię i nie wiem gdzie jestem



ps. jak widzę coś takiego ->


to jestem przerażony... gdzieś widziałem tutorial gdzie pisali że TRZEBA czegoś takiego używać - dla mnie głupota i śmietnik!

Ten post edytował Poker 12.05.2009, 12:54:57

1. Przerzuć się na angielskie nazewnictwo.

2. [HTML] pobierz, plaintext 
   <h1><a href="#" title="GamesPlay.pl - Shoutbox"></a></h1>
   [HTML] pobierz, plaintext 

   Chyba zapomniałeś o treści dla A, a link jako # jest złym rozwiązaniem. Daj jako link do głównej.

3. [HTML] pobierz, plaintext 
   <p><strong>Chatują</strong> :</p>
   [HTML] pobierz, plaintext 

   A co jest takiego w tym ważnego, że dajesz to aż w STRONG?

4. [HTML] pobierz, plaintext 
   <div id="WymienKtoChat">
    
   			<a href="#" title="Artu123">Artu123</a> , <a href="#" title="Artu123">Artu123</a> , <a href="#" title="Artu123">Artu123</a> , <a href="#" title="Artu123">Artu123</a>
    
   		</div>
   [HTML] pobierz, plaintext 

   To jest przecież lista, więc wyświetl to jako listę.
5. Przed nadaniem nazwy identyfikatora czy klasy zastanów się czy ma ona jakiś sens: WymienKtoChat, WymienKtoChat1?
6. Cierpisz na divitis i classists
7. Kod HTML powinienen reprezentować strukturę danych, a nie jej wygląd - więc pozbądź się jakiś "ContentLewa" (ponglisha też)
8. Po co co chwile jakiś DIV z clearem?
9. Staraj się nie stosować atrybutu style, chyba że to konieczne - tutaj nie jest.
10. line-height:0px;font-size:0px; co to ma być?

11. [HTML] pobierz, plaintext 
    <img src="images/AvatarTop.gif" alt="[Avatar - Border]" /><br />
    				<img src="images/Avatar.gif" class="AVATAREK" width="59px" height="52px" title="Artu123 - Avatar" alt="[Avatar]" /><br />
    				<img src="images/AvatarDown.gif" alt="[Avatar - Border]" />
    [HTML] pobierz, plaintext 

    Obraz wstawiany przez IMG to część "contentu", a nie dekoracji - więc wywal te "bordery".
    Cytat
    Skoro dajesz klasę to po co width i height pisane inline?
    Akurat dobrze, że to podał - strona nie będzie skakać przy ładowaniu. Jedynie niepotrzebnie dodał jednostkę (px).

12. [HTML] pobierz, plaintext 
    <div class="ShoutboxTrescNick">
     
    					<a href="#">Artu123</a>
     
    				</div>
     
    				<div class="ShoutboxTrescData">
     
    					07.05/20:23
     
    				</div>
    [HTML] pobierz, plaintext 

    HTML wychodzi do Ciebie z dziesiątkami elementów, a Ty się uparłeś tych DIVów. O CITE, BLOCKQUOTE, P i całej masie innych nie słyszałeś?

13. [HTML] pobierz, plaintext 
    <!-- Koniec Post -->
    [HTML] pobierz, plaintext 

    Polski jest językim, w którym niektóre części mowy/zdania się odmieniają. Wiem, że się czepiam.

14. [HTML] pobierz, plaintext 
    <textarea rows="0" cols="0"></textarea>
    [HTML] pobierz, plaintext 

    Poza tym, że zapomniałeś o atrybucie name, to zapomniałeś jeszcze o formularzu. No i gdzie jakiś opis (LABEL) na co jest to pole?

Jak gdzieś napisano, że tak trzeba to rzeczywiście głupota, ale dlaczego uważasz resetowanie domyślnych stylów przeglądarki za głupotę? Pozwala to uniknąć wielu niepotrzebnych nieścisłości.

1. Kod CSS jest kompletnie nieczytelny
2. Przy zerze nie dawaj jednostek
3. A... nie sprawdzam, bo nie chce mi się tego rozszyfrowywać...

Gratuluję Crozin'owi długiego posta.

Czy to jest twój design? Tragedia.

Staraj się nie wstawiać pustych znaczników. Komentarze chyba piszesz dla jakiegoś niezłego tumaka. Czy to ma sens?
Dlaczego alt'y img'ów piszesz w kwadratowych nawiasach?
Zainteresuj się ul-em.
Strong-a nie używamy do zaznaczenia pogrubionego tekstu, który nie ma znaczenia merytorycznego, a jedynie wizualny.
Tytuł strony mało mówi robotowi.
Zalecam przejście na UTF-8.

Ten post edytował Kildyt 12.05.2009, 15:41:08

Ułatwij sobie życie

Zamiastzrób tylko ostylowanie do tego co już masz:

Poker:

Oto styl główny PerfectionOrVanity.com Riddle'a. Tzn. początek:

[HTML] pobierz, plaintext 
html, body, h1, h2, h3, h4, h5, h6, div, span,
blockquote, p, address, form, fieldset, img,
ul, ol, dl, dt, dd, li, hr, table, td, th,
strong, em, sup, sub, dfn, ins, del, q, cite,
var, samp, code, kbd, tt {
	margin: 0;
	padding: 0;
	border: 0;
}
[HTML] pobierz, plaintext 

Ale pewnie jesteś mądrzejszy.

Nie ma potrzeby resetowania ostylowania jeśli później i tak je wykorzystujemy, ze zmienną wartością i nazwą, a tak jest w większości przypadków. To tak jakbyś wyważał drzwi PO wejściu do mieszkania.

O stronie... Zmień kodowanie, zastosuj się do rad podanych przez poprzedników i może wywal znaczniki meta, i tak są mało warte.

Zakodowane jest super

@Cypherq - uwierz, że w przypadku robienia takiego resetu dla stron opartych o CMS typu eZPublish potrafi to bardzo przeszkadzać... Nie mówie że to błąd, ale po prostu trzeba tego używać z głową.

I później mieć problemy z IE W wersji 6 nie obsługuje first-child. Nie wiem jak jest w 7.

Wiesz... mamy IE8. A jak bardzo zależy Ci na zgodności z IE6 to już lepiej skorzystać z czegoś w stylu http://code.google.com/p/ie7-js/

To nie jest żaden błąd.



Jest wydajniejsze niż.



I z tego powodu tak się pisze, jest to poprawne i wskazane ze względów wydajnościowych.

Pozdrawiam.

Ten post edytował pejott 14.05.2009, 14:34:13

Jakieś testy?

Do tego nie trzeba testów, bo przypisanie do każdego możliwego elementu atrybutu, na przykład.


Jest wolniejsze niż przypisanie do 50 określonych.
Nie potrzeba testów, aby to stwierdzić.

Ten post edytował pejott 14.05.2009, 18:56:48