Strona Ocena ?

pytanie zadane 29 lipca 2015 w Nasze projekty przez ReksetoDev Gaduła (4,530 p.)

Pisze ostatnio strone dla klienta i chcialbym wiedziec czy trzeba cos poprawic narazie ona wyglada tak

http://www.skyup.pl/easydrop/index.html :) dziekuje za wszelkie poprawki które mozecie mi zaproponowac :) i milego kodowania

9 odpowiedzi

odpowiedź 29 lipca 2015 przez niezalogowany
wybrane 31 lipca 2015 przez ReksetoDev

Najlepsza

No, powiem ci, że stronka jeśli chodzi o fazę testową to fajna ;)

No, ale miały być poprawki i zastrzeżenia, to masz:

nie masz ikonki w zakładce strony, łatwe do zrobienia ;)
przyspiesz odrobinkę przejście z białego koloru do szarego po najechaniu myszką na napisy w menu (i może przy okazji zmień kolor przejścia na jeszcze trochę ciemniejszy)
a może menu daj pod ten zielono-szary obrazek (i zmień ten obrazek na coś konkretniejszego ;)), chyba będzie lepiej to wyglądało
a przycisk startu powinien być pierwszy w kolejności w menu
zmień czcionkę, ta jest trochę brzydka
wyśrodkuj w pionie napis w stopce na dole strony

Fajnie, że masz przyklejane menu ;) Tylko mówię, daj je pod ten obrazek.

To ja ci życzę miłego kodowania i stylizacji strony!

PS: Minecraft, kiedy ja w to ostatnio grałem ;)

komentarz 31 lipca 2015 przez ReksetoDev Gaduła (4,530 p.)

ikonke dodaje mój klient gdyż sam jest ogarniety w HTML i CSS co to tego obrazku to mysle ze wstawie tam baner :) przycisk start niedlugo naprawie nad czcionka mysalem ale nie wiem jaka ;x nota bene mój klient(raczej dobry przyjaciel i wlasciciel strony ;x dla którego to robie) jest coś na rodzaj grafika wiec pomysli sie a narazie dziekuje za taka opinie i mam nadzieje ze nastepne strony przypadna lepiej do gustu :)

komentarz 31 lipca 2015 przez niezalogowany

O, no to widzę, że zrobisz postępy i już wiesz przynajmniej co zrobić ;) A ci jeszcze kolega pomoże robić grafiki, to będzie super. Jakby co, to po skończeniu strony znowu dasz ją do końcowej oceny, to chętnie sprawdzimy ;)

Oby ci ta strona fajnie wyszła! I żeby na serwerze było fajnie ;)

odpowiedź 29 lipca 2015 przez Dorion300 Szeryf (90,250 p.)

Dupy nie urywa ale wygląda jak porządna strona z roku 2014/2015 więc jest dobrze.

komentarz 29 lipca 2015 przez niezalogowany

Ty, napisałeś to samo co ja, tylko że w jednym zdaniu :D

odpowiedź 29 lipca 2015 przez M4tthew Gaduła (4,480 p.)

odpowiedź 29 lipca 2015 przez Michau Xlow Pasjonat (15,190 p.)

odpowiedź 29 lipca 2015 przez Ivan Maniak (60,650 p.)

1. Strona nie jest responsywna

2. Wysokość menu jest ustawiona na sztywno i przy mniejszych ekranach jeden z linków ląduj na dół bez tła. [nie responsywna]

3. Ranking -> Input -> Wyszukaj gracza . Zamiast dawać value="..." możesz dać placeholder="..." Będzie lepiej wyglądało i nikt nie wyśle Ci formularza a "Wyszukaj gracza"

4. Sklep -> Div.buy Zastępuje to diva możesz dać <button></button> Załatwi to sprawę np. Rozjechanego tekstu "Kup" będzie on na środku i nię będziesz musiał się męczyć.

5. Jeżeli strona startowa istnieje to lepiej aby była na pierwszym miejscu w menu. Jest to już taki przyzwyczajenie użytkowników. Nie musi być ale wg. mnie powinna.

odpowiedź 29 lipca 2015 przez Hokage2910 Początkujący (360 p.)

odpowiedź 29 lipca 2015 przez Tomatosoup Pasjonat (18,530 p.)

Ocena bardziej od strony kodu i UX:

<div class="menu"> ? Użyj znacznika <nav>
<script src="js/jquery-2.1.3.min.js"></script> - Skrypty umieszczaj na samym dole strony, czyli przed zamknięciem </body>
Zamiast stosować <br /> użyj marginesów dla <p>, każdy paragraf wpakuj pomiędzy kolejne <p></p>, użyj także na stronie głównej nagłówka a pod nim paragrafów ( <h1></h1> )
Powinien być odnośnik do rankingu na stronie głównej w miejscu w którym wspominasz o nim
"START" w nawigacji powinno być po lewej stronie
"EKIPA SIECI SKYUP" powinno być także w tabeli w znaczniku <caption>
Jeżeli już używasz <tbody> to powinno być także <thead> i <tfoot>
Brakuje też nagłówków poszczególnych komórek w tabeli ( <th> , dodaj display:none; w css jeżeli nie chcesz ich wyświetlać.
style="width: 50px; height: 50px; border-radius: 3px; border: 1px solid black;"
Dlaczego jest to zrobione w inline css ? Stwórz klase .avatar, wrzuć tam co trzeba i dodaj do każdego img w tabeli
<div class="footer"> --> <footer>, <div class="header"> --> <header>
Na przyszłość - style.css ładuj na samym końcu, czyli po fontello.css czy frameworku - przeglądarka czyta od góry do dołu, twoje style przeczyta na końcu więc będą miały największy priorytet i będą nadpisywać inne, a w końcu o to chodzi.
<title> dla każdej podstrony inny - np. Sklep - EasyDrop :: SkyUP.pl

No i strona nie jest responsywna, ale o tym już pisali

komentarz 29 lipca 2015 przez Comandeer Guru (601,590 p.)

<th> , dodaj display:none; w css jeżeli nie chcesz ich wyświetlać.

To po co je wstawiać? Do takich rzeczy wykorzystuje się .visuallyhidden z H5BP

komentarz 29 lipca 2015 przez Tomatosoup Pasjonat (18,530 p.)

Czy w takim razie google nie czyta tego co ma display:none; ?

komentarz 29 lipca 2015 przez Comandeer Guru (601,590 p.)

Czyta, ale nie bierze pod uwagę, bo jest to element całkowicie ukryty dla wszystkich przeglądarek i czytników ekranowych

Zresztą Google mnie nie obchodzi - bardziej się martwię o osoby niepełnosprawne ;)