Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [www] serwis o perukach
acztery
post
Post #1





Grupa: Zarejestrowani
Postów: 945
Pomógł: 7
Dołączył: 15.03.2005
Skąd: katowice

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


co myślicie o http://www.pcabin.home.pl/perukaorg/ jeszcze domena nie jest podpięta
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi (1 - 7)
Kreton
post
Post #2





Grupa: Zarejestrowani
Postów: 345
Pomógł: 3
Dołączył: 20.02.2005

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


Co do wyglądu jest ok, czysto, ładnie.
Boksy google są wyszczególnione takim kolorem że wogule nie widać ich na LCD. Same reklamy tak, ale nie otoczkę.
Zdjęcie w stopce zajmuje zdecydowanie za dużo. Nie dało by rady tego zmienić ? praktycznie 80% zajmuje jeden i ten sam gradient, dopiero na końcu jest ta ładna pani za mgłą.
Szkoda że stronka troszkę się ładuje przez te grafiki, top, footer i horyzontalne menu.

  1. </body>
  2. </html>
  3. </div></body>
  4. </html>
Eeeee ?
Go to the top of the page
+Quote Post
Moli
post
Post #3





Grupa: Zarejestrowani
Postów: 662
Pomógł: 45
Dołączył: 26.03.2007
Skąd: Warszawa

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


Wygląd jest ładny. Kod ujdzie, ale popracuj nad nim. Te tabelki z aktualnościami są strasznie oczo-walne. W galerii zamiast wyskakiwania okienka użyj LightBox'a.
Go to the top of the page
+Quote Post
acztery
post
Post #4





Grupa: Zarejestrowani
Postów: 945
Pomógł: 7
Dołączył: 15.03.2005
Skąd: katowice

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


"Kod ujdzie, ale popracuj nad nim" tzn? rozwiń to przecież kod jak dla mnie jest czytelny i zgodny z XHTML

  1. </body></html></div></body></html>
poprawione
Go to the top of the page
+Quote Post
Moli
post
Post #5





Grupa: Zarejestrowani
Postów: 662
Pomógł: 45
Dołączył: 26.03.2007
Skąd: Warszawa

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


np. wszystkie funkcje JS umieść w jednym, oddzielnym pliku i wczytanie daj w sekcji head. Tak po za tym, to kod jest ok.
Go to the top of the page
+Quote Post
Kreton
post
Post #6





Grupa: Zarejestrowani
Postów: 345
Pomógł: 3
Dołączył: 20.02.2005

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


Dla mnie troszkę za dużo div-ów. Czasem nawet jest ich 5 poziomów, wydaje mi się że można by to spokojnie upchnąć na mniejszej ilość.

Jeżeli masz:
  1. <div class="aktualnosci">
  2. <div class="news">
  3. <div class="news">
  4. <div class="lastnews">
  5. </div>

To lepiej nadać nadać właściwości rodzicowi w sposób : div.aktualnosci div { tutaj style }, a osobno utworzyć styl dla lastnews. W przyszłości gdy będziesz miał 20 takich linków różnica będzie znaczna.

Ten post edytował Kreton 12.05.2007, 12:58:06
Go to the top of the page
+Quote Post
acztery
post
Post #7





Grupa: Zarejestrowani
Postów: 945
Pomógł: 7
Dołączył: 15.03.2005
Skąd: katowice

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


lastnews i news to calkiem inne atrybuty ostatnia aktualnosc rozni sie rozmiarem ... można zawsze cos tam jeszcze wymyśleć ale po co
Go to the top of the page
+Quote Post
Kreton
post
Post #8





Grupa: Zarejestrowani
Postów: 345
Pomógł: 3
Dołączył: 20.02.2005

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


No ale gdybyś miał listę 20 linków to dla każdego stosowałbyś <a class=""> zamiast nadać to rodzicowi ? Np. div#links a ? Przecież to oszczędność miejsca.
Przecież zaproponowałem żeby właśnie dla tego ostatniego specjalnego newsa nadać mu class="lastnews". Ale dla reszty nie jest to uzasadnione, lepiej nadać to rodzicowi.

Ten post edytował Kreton 12.05.2007, 13:05:40
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: 20.08.2025 - 05:54