Forum PHP.pl > [www] Prosze o słowo krytki

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [www] Prosze o słowo krytki - strona zakonu

piotao

22.04.2007, 21:57:19

Zapraszam na stronę http://force-crusaders.org/ - proszę o krytykę, choć to bardzo stary template i nie mam nowszego projektu, żeby całość zmienić

Cały kod pisany przeze mnie, działa dla małej społeczności już ponad rok z hakiem.

starach

23.04.2007, 09:45:22

Tabelki .... a fuj ...
http://validator.w3.org/check?uri=http://z...s.org/index.php
o_O Poza tym na stronie głównej obrazek jest źle wypozycjonowany.
Takiego spamu to ja dawno nie widziałem

CODE

<tr>
<td valign="bottom" width="100%"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="200" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="20"></a></td>
</tr>
<tr>

<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-1c.jpg" border="0" height="101" width="99"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
</tr>
<tr>

<td colspan="2" valign="bottom">
<table align="center" border="0" width="70%"><tbody><tr><td class="fortune">W naszym Zakonie są tacy, którzy żyją dzięki świadomości Mocy, oraz są tacy, którzy żyją dzięki jej nieświadomości.<br> </td></tr></tbody></table>
</td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-2b.jpg" border="0" height="148" width="84"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-2c.jpg" border="0" height="148" width="99"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-2d.jpg" border="0" height="148" width="61"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>

</tr>
<tr>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-3a.jpg" border="0" height="120" width="258"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-3b.jpg" border="0" height="120" width="84"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-3c.jpg" border="0" height="120" width="99"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/intro-3d.jpg" border="0" height="120" width="61"></a></td>
<td valign="bottom"><a name="Kliknij aby wejść dalej!" href="http://zakon.force-crusaders.org/"><img src="images/intro/blank.gif" border="0" height="1" width="1"></a></td>
</tr>

Strona główna za szara za bura i skoro tyle linków masz w menu to mogłeś już zastosować javascript i menu rozwijane.

revyag

23.04.2007, 13:11:39

Niezbyt to wygląda. Graficznie cienko. Kolorystyka do przemeblowania. Czarna czcionka na tym tle wygląda dosyć obskurnie, potestowałbym jakieś ciemne odcienie szarości albo ciemny brązowy.
Te zielone i złote ozdobniki kompletnie nie pasują. Nie wiem na jaki kolor bym to zmienił, trzeba by zrobić parę testów.

piotao

24.04.2007, 06:05:30

Failed validation, 444 errors

WOW!!! Wielkie dzięki za tego linka walidującego, to doskonale uczy pokory

Cóż, będę powolutku poprawiał, nie ma innej drogi...

Dziękuję za krytykę!

Apropo tego javascriptu z menu rozwijanym. Nie znam się na tym, ale szukałem w sieci, znalazłem wiele różnych skryptów chociaż żaden jakoś w moich rękach nie zadziałał tak, żebym był zadowolony. Czy ma ktoś z Was może jakiś "sprawdzony" skrypt do generowania takiego menu? Coś, co jest - nie ukrywam - pewnego rodzaju gotowcem (proste przede wszystkim)... ? Tak tylko pytam, nie chcę robić offtopa tutaj.

starach

24.04.2007, 10:45:25

Skrypcik jest bardzo prosty.

[HTML] pobierz, plaintext 
<script language="javascript" type="text/javascript">
function show_hide(id)
{
  var disp=document.getElementById(id).style.display
  if(disp=='none') {
    document.getElementById(id).style.display = 'block';
  } else {
    document.getElementById(id).style.display = 'none';
  }
}
</script>
[HTML] pobierz, plaintext

Dajesz to między znacznikami <head></head>,
a w body robisz dwie warstwy hmm powiedzmy listy wypunktowane jedna to menu główne pozostałe to menu podrzędne. W menu głównym w linkach dajesz <a href="java script: show_hide('identyfikator')">pokaz zamknij menu</a> A w menu którym chcesz manipulować za pomocą tego linku dajesz
<ul id='identyfikator'>
<li></li>
<li></li>
</ul>

piotao

24.04.2007, 17:50:33

Dzięki za odpowiedź!

Acha, i jak rozumiem cały <ul> zniknie gdy jego styl będzie miał display ustawiony na none, dobrze rozumiem? Hmmm, no to pięknie, ale czy takie coś można zrobić dla elementów TD tabeli, którą teraz w tamtym designie wykorzystuję? Dla przykładu mam tam coś takiego:

TD class=main - nagłówek działu (klikalny i prowadzący do strony)
TD class=podmenu1 - podstrona działu
TD class=podmenu1 - podstrona działu
...

itp. I teraz czy dałoby się użyć tego, lub podobnego javascriptu, żeby pokazane było tylko TD z klas main (odpowiednie id mogę im ustawić), a po kliknięciu tego TD wyświetlana byłaby reszta podrzędnych pozycji...? Jednym słowem, na UL i LI da się to prosto zrobić, ale mój nieszczęsny design siedzi mocno w tabelach, i stąd takie "niewinne" pytanie

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.