- Symulator MaSzyna -
Hyde Park => Bocznica => Wątek zaczęty przez: Robert89 w 23 Maja 2018, 11:19:11
-
Cześć,
zauważyłem dział WWW, więc chciałem podzielić się z Wami moją stroną internetową - http://najazdowe.pl/.
Co o niej uważacie?
Jak podoba Wam się grafika?
Co byście zmienili?
To moja pierwsza strona, więc liczę na Wasze przychylne oko :D
Opis działu jest chyba jednoznaczny do czego jest przeznaczony:
Wszelkie sprawy związane z funkcjonowaniem strony WWW (www.eu07.pl).
Bocznica.
adsim
-
- brak wersji mobilnej strony :(
- podstrona "pełna oferta wag" nie działa
- składanie zamówień: użycie funkcji "mailto" wygląda słabo. Liczyłem na to że zostanę przekierowany do formularza ze spisem produktów i możliwością kontaktu przez formularz.
- "opinie klientów", nawet jak są w pełni autentyczne to w mojej ocenie jest to januszowe podejście.
- dizajn ciekawy. bez zbędnych bajerów obciążających działanie strony
- przydałaby się zakładka ze zrealizowanymi projektami, coś na wzór https://www.puczynski.pl/places
-
Ten dział nie służy do prezentowania swoich stron, tylko konkretnie strony www związanej z MaSzyną. Ale jak już napisałeś, to zrobię Ci code review. Błędy:
- szkielet strony bazuje na tabeli zamiast na divach - ogromny błąd,
- strona nie jest responsywna,
- istnieje od lat HTML5 (https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/HTML5), z którego nie korzystasz,
- nie odseparowałeś warstw tekstu i prezentacji od siebie(CSS umieszczony w kodzie HTML),
- stosujesz przestarzałe znaczniki, np. <font> (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/font),
- stylujesz niektóre znaczniki inline, np. <td style="width: 306px; height: 198px" class="style10" valign="top">,
- używasz znacznika <br /> w pseudostopce (używaj znacznika <footer> (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/footer) ze standardu HTML5) do zwiększenia pionowej odległości pomiędzy liniami z tekstem,
- nazwy klas powinny cokolwiek mówić o samej klasie (najlepiej w języku angielskim),
- atrybut alt w znaczniku <img> (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img) służy to przechowywania tekstu opisującego obrazek, który ma być wyświetlony, jeśli z jakiś powodów przeglądarka nie jest w stanie załadować obrazka,
- separujesz tekst wielokrotną spacją niełamliwą . Całe menu powinno być umieszczone w <nav> (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/nav) (znowu kłania się HTML5).