Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [kod css/xhtml] kod xHTML
sebekzosw
post
Post #1





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


Proszę o ocenę kodu xHTML

Link
Go to the top of the page
+Quote Post
webber
post
Post #2





Grupa: Zarejestrowani
Postów: 107
Pomógł: 19
Dołączył: 30.10.2007

Ostrzeżenie: (0%)
-----


Pierwszy spojrzenie i... divitis. Ale jedziemy po kolei.

Kod
class="floatleft"


Ciekawa nazwa klasy winksmiley.jpg

Kod
<a href="index.html"><img src="images/aktualnosci.jpg" alt="Aktualności" class="floatleft" /></a>
            <a href="filmy.html"><img src="images/filmy.jpg" alt="Aktualności" class="floatleft" /></a>
            <a href="wycinki.html"><img src="images/wycinki_prasowe.jpg" alt="Aktualności" class="floatleft" /></a>

            <a href="linki.html"><img src="images/linki.jpg" alt="Aktualności" class="floatleft" /></a>


To jest lista.

A zamiast stosować klasę floatleft dałbyś div#logoikony img { float: left; }


Kod
<div id="logo">
        <div id="tytul_logo">Zapisy</div>
        <div id="tresc_logo">Przykładowy tekst cd; Przykładowy tekst cd; Przykładowy tekst cd; Przykładowy tekst cd :-)</div>
    </div>


Myślę że lepiej było się posłużyć znacznikami h i p i zamiast dawać kolejne divy i id dać div#logo p { styl }, div#logo h2 { styl }

Kod
<div id="menu">
        <ol class="menulist">
            <li><a href="index.html">Aktualności</a></li>
            <li><a href="http://forum.tkd-bielsko.pl/">Forum</a></li>
            <li><a href="filmy.html">Filmy</a></li>

            <li><a href="galeria.html">Galeria</a></li>
            <li><a href="Zapisy.xhtml">Treningi</a></li>
        </ol>

        <div class="text1">
            przykładowa tresc
        </div>
        <div class="text2">
            przykładowa tresc
            <div class="linia"></div>

        </div>
    </div>


Do menu mogłeś odwołać się poprzez div#menu ol { }, a te divy o klasie text1 i 2 eh... masakra smile.gif

Kod
<div class="newsbar">
            <div class="newstitle">
                Lorem ipsum dolor sit amet
            </div>

            <div class="newsinfo">
                <div class="zegar"></div>
                28/04/2009, 13:46
            </div>
        </div>
        <div class="newsinside">
            Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed vel velit sed odio aliquet suscipit. Integer tincidunt augue at dui. Morbi leo. Quisque quis sem sed ipsum rutrum aliquam. Suspendisse potenti. Suspendisse potenti. Praesent dapibus adipiscing justo. Sed consequat. Quisque consequat. Morbi venenatis dapibus libero. Nam nec nisl. Duis quis purus non nibh varius viverra. Class aptent taciti sociosqu ad litora torquent per conubia nostra, per inceptos himenaeos. In facilisis, urna et imperdiet tempus, mauris tellus ornare tellus, id semper odio leo sed ipsum. Nunc suscipit dignissim erat. Vestibulum rhoncus magna eu dolor sagittis imperdiet. Proin aliquam. Vivamus ante arcu, faucibus non, mollis ac, scelerisque eu, metus. Nam ornare pellentesque nisi.
            <div class="clearboth"></div>
        </div>
        <div class="newsend"></div>


Znów pominąłeś h, nie używasz span a przydałoby się do zegara, no i tekst bardziej w p...

Kod
<div id="zwiazki_kluby">
            <a href="http://www.pztkd.lublin.pl/" id="obr03" class="rollover"><img src="images/png/pztkd_szary.png" alt="Polski Związek Taekwon-Do" /></a>
            <a href="http://www.ksdragon.pl/" id="obr01" class="rollover"><img src="images/png/dragon_szary.png" alt="Klub Sportowy Dragon" /></a>
            <a href="http://www.tkd-itf.org/" id="obr02" class="rollover"><img src="images/png/itf_szary.png" alt="International Taekwon-Do Federation" /></a>
        </div>


To też jest lista. Dodaj width i height do obrazków.


--------------------
"CSS is like a girlfriend. No matter how hard you try, she will always interpret it a different way."
Go to the top of the page
+Quote Post
sebekzosw
post
Post #3





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


Cytat(webber @ 15.05.2009, 09:17:06 ) *
Pierwszy spojrzenie i... divitis. Ale jedziemy po kolei.


Może jakaś sugestia na rozwiązanie ;> smile.gif?
Go to the top of the page
+Quote Post
osl
post
Post #4





Grupa: Zarejestrowani
Postów: 260
Pomógł: 41
Dołączył: 6.04.2009
Skąd: Gdańsk/okolice

Ostrzeżenie: (0%)
-----


przecież masz mnóstwo innych tagów niż div...
Go to the top of the page
+Quote Post
sebekzosw
post
Post #5





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


A co złego jest w DIV? One są najbardziej uniwersalne... smile.gif
Go to the top of the page
+Quote Post
Poker
post
Post #6





Grupa: Zarejestrowani
Postów: 205
Pomógł: 3
Dołączył: 20.04.2009
Skąd: Jaworzno

Ostrzeżenie: (0%)
-----


moze i uniwersalne ale do formatowania tekstu uzywa sie czegos innego niz DIV!


--------------------
:) na miliony przyjdzie czas...
Go to the top of the page
+Quote Post
sebekzosw
post
Post #7





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


czego się używa? smile.gif
Go to the top of the page
+Quote Post
pyro
post
Post #8





Grupa: Zarejestrowani
Postów: 2 148
Pomógł: 230
Dołączył: 26.03.2008

Ostrzeżenie: (0%)
-----


kurshtml.boo.pl, zobacz


--------------------
ET LINGUA EIUS LOQUETUR IUDICIUM
Go to the top of the page
+Quote Post
webber
post
Post #9





Grupa: Zarejestrowani
Postów: 107
Pomógł: 19
Dołączył: 30.10.2007

Ostrzeżenie: (0%)
-----


Cytat(sebekzosw @ 15.05.2009, 15:41:47 ) *
czego się używa? smile.gif


To Ty na pewno wiesz co robisz? smile.gif

Pytasz o podstawy podstaw...


--------------------
"CSS is like a girlfriend. No matter how hard you try, she will always interpret it a different way."
Go to the top of the page
+Quote Post
Crozin
post
Post #10





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


Generalnie rzecz biorąc. DIVów powinno być jak najmniej - bo nie niosą one sobą żadnej semantycznej treści.
Go to the top of the page
+Quote Post
sebekzosw
post
Post #11





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


Wybierałem między <table> a <div>

Wybrałem div i teraz jeszcze jakieś wyjątki...
Cytat(Crozin @ 15.05.2009, 20:40:45 ) *
Generalnie rzecz biorąc. DIVów powinno być jak najmniej - bo nie niosą one sobą żadnej semantycznej treści.


to jest konkretna odpowiedź, a nie jakieś podawanie linków itp :|
Go to the top of the page
+Quote Post
Crozin
post
Post #12





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


Właśnie dlatego strasznie denerwuje mnie jak słysze "strona na divach" czy coś podobnego. Bo w tym nie chodzi o to by strona była robiona przy użyciu DIV. Chodzi o to by korzystać z całej palety dostępnych znaczników XHTMLa. By gdy potrzebujesz przedstawić listę zawodników, ich wiek, płeć, ilość czegoś tam i czegoś jeszcze byś skorzystał z TABLE. Gdy potrzebujesz cytatu z Q, BLOCKQUOTE, listy DL, UL czy OL, a gdy masz zwykły tekst P.
Go to the top of the page
+Quote Post
MWL
post
Post #13





Grupa: Zarejestrowani
Postów: 493
Pomógł: 32
Dołączył: 14.04.2008
Skąd: Lenkowski.net

Ostrzeżenie: (0%)
-----


np. nagłówki muszą być pisane w h1, h2 itd. pomaga to np w pozycjonowaniu.
Apropo uniwersalności, każdy tag może być taki sam. Sam link (a) może działać jak div, jeśli przypiszesz mu display:block;

Ten post edytował MWL 30.05.2009, 18:14:58


--------------------
Wpadaj na mój kanał o PHP. Dużo mięsa 🥩!
Go to the top of the page
+Quote Post
230005
post
Post #14





Grupa: Zarejestrowani
Postów: 316
Pomógł: 36
Dołączył: 2.04.2008

Ostrzeżenie: (0%)
-----


To ja może trochę z innej strony - obok loga klubu masz "wycinki pracowe", a powinno być chyba prasowe smile.gif. Generalnie czcionka, której użyłeś do tych czterech buttonów na górze jest bardzo ładna, ale (może tylko ja mam takie zboczenie na tym punkcie) przecież piszesz stronę po polsku więc powinieneś użyć takiej która ma te wszystkie ą i ę. Tak samo w logo jest "sekcja klubu oswiecim". Nie wiem jak innych ale mnie to strasznie bije po oczach.
Go to the top of the page
+Quote Post
sebekzosw
post
Post #15





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


Poprawiłem wg Waszych sugestii, oto link:

Link

Proszę o komentowanie...

Wyniki Walidatora CSS W3C - Dokument ten jest poprawnie napisanym arkuszem CSS wersja 2.1 !

Wyniki Walidatora (x)HTML - This document was successfully checked as XHTML 1.0 Strict!
Go to the top of the page
+Quote Post
Crozin
post
Post #16





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


Menu w Twoim przypadku jest listą nieuporządkowaną (UL). Ale dlaczego #nadlogiem nie jest listą? Poza tym identyfikator "nadlogiem"? XHTML to struktura dokumentu, nie jego prezentacja.
Kod
<div style="width:88px; float:left;">&nbsp;</div>
Takie cuda...
Kod
<div class="text1"> przykładowa tresc </div>
                      <div class="text2"> przykładowa tresc
                          <div class="linia"></div>
                      </div>
                </div>
A od tekstu to jest chyba P? A od linii CSS, ew. jeżeli ma ona semantyczne znaczenie HR - tu nie ma.

Cierpisz na syndrom przedawkowania atrybutu class.
Kod
<div class="news">
                          <h3 class="tytul_newsa">Lorem ipsum dolor sit amet</h3>
                          <span class="data">28/0/2009, 13:45</span>
                          <p class="tresc_newsa"> .....</p>
                      </div>


Kod
<ol id="dol_menu">
                            <li><a href="#">Aktualności</a> / </li>
                            <li><a href="#">Filmy</a> / </li>
                            <li><a href="#">Treningi</a> / </li>
                            <li><a href="#">Galeria</a> / </li>
                            <li><a href="#">Historia klubu</a> / </li>
                            <li><a href="#">Wycinki prasowe</a> / </li>
                            <li><a href="#">Kontakt</a> / </li>
                            <li><a href="#">Linki</a></li>
                        </ol>
Ale po co te "/" jako tekst?


Tak na pierwszy rzut oka
Go to the top of the page
+Quote Post
sebekzosw
post
Post #17





Grupa: Zarejestrowani
Postów: 437
Pomógł: 42
Dołączył: 16.04.2007

Ostrzeżenie: (0%)
-----


Cytat(Crozin @ 16.05.2009, 16:04:39 ) *
Cierpisz na syndrom przedawkowania atrybutu class.


Pozbyłem się zbędnych "class" ;p więcej to chyba nie da się usunąć

Cytat(Crozin @ 16.05.2009, 16:04:39 ) *
Ale po co te "/" jako tekst?


No a co mam dać? Musi to być...

Jakieś inne sugestie?
Go to the top of the page
+Quote Post
Crozin
post
Post #18





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


Cytat
No a co mam dać? Musi to być...

Jakieś inne sugestie?
Jak dla mnie ten ukośnik jest elementem dekoracyjnym czyli powinien być wstawiany za pomocą CSS. Ale lepiej jest wstawić to nie jako tekst tylko zwykły obrazek tła.

Aktualności na stronie tworzą listę.
Z własnego doświadczenia podpowiem Ci, że warto coś takiego:
Kod
<div class="news">
                          <h3><a href="#">Lorem ipsum dolor sit amet</a></h3>
                          <span>28/0/2009, 13:45</span>
                          <p>...</p>
                      </div>
Zamienić na:
Kod
<div class="news">
                          <.. class="meta">tytuł, data</>
                          <... class="body"><p>tresC</p></>
                      </div>
Prędzej czy później obecny podział stanie się problematyczny
Go to the top of the page
+Quote Post
PawelC
post
Post #19





Grupa: Zarejestrowani
Postów: 1 173
Pomógł: 121
Dołączył: 24.09.2007
Skąd: Toruń

Ostrzeżenie: (0%)
-----


W górnym menu masz zamiast wycinki prasowe, to masz wycinki pracowe
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 21.08.2025 - 10:00