Ocena strony www [ankieta]

pytanie zadane 26 lipca 2016 w Nasze projekty przez weats Początkujący (470 p.)

Witam,

prosiłbym o ocenę poniższej strony www. Jest to moja pierwsza strona stworzona "nie w celach naukowych".

http://kamieniarstwogorczewscy.pl

Zależy mi przede wszystkim na ocenie:

kodu
design'u
czytelności

Z góry dziękuję!

Możliwe odpowiedzi:

Bardzo dobrze (2 głosów, 8%)

Dobrze (16 głosów, 62%)

Średnio (8 głosów, 31%)

Źle (0 głosów)

Bardzo źle (0 głosów)

8 odpowiedzi

odpowiedź 26 lipca 2016 przez Comandeer Guru (607,980 p.)
wybrane 26 lipca 2016 przez weats

Najlepsza

Brak outline'u dla linków.
Przekieruj wersję bez www na tę z – inaczej masz duplicate content.
Nie sądzisz, że za dużo słów kluczowych w kodzie?
Czcionki od Google można dodać jednym arkuszem.
Skrypt ciasteczek można wywalić na koniec body.
Topbar z numerami telefonu wygląda jak address > dl z dodatkiem np. odpowiedniego słownictwa ze Schema.org
.menu-trigger to powinien być przycisk.
A samo menu można "oetykietować" przy pomocy [aria-labelledby], wskazując na przycisk.
Nie styluj po [id].
div#header_left można wywalić i style z niego przenieść na samo h1.
Plus za ładnie zrobione logo!
W menu powinno być li > a, nie a > li!
#line można zamienić h2 – chociaż tutaj warto się zastanowić nad tym jaką hierarchię nagłówków dostaniemy.
#img wygląda na #content::before.
#content wygląda na main.
Punktor w liście lepiej zrobić przy pomocy li::before. W ostateczności dowalając [aria-hidden=true] do obecnych span.
Ten pusty div[style] można zamienić na #container::after.
Nie krzycz w stopce – mamy text-transform: uppercase; w CSS.
Dodanie [onblur] i [onfocus] w formularzu to po prostu skrajne niezrozumienie działania [placeholder]…
.stars powinny mieć etykietę (np. [aria-label="Pole wymagane"]).
Wgl w kontakcie lewa strona wygląda jak lista dl – nagłówki tam nie pasują!
div#top_bar_tel span.tel_num vs .tel_num – utrzymuj minimalną specyficzność selektorów!

odpowiedź 26 lipca 2016 przez Jaroslaw Roj Obywatel (1,990 p.)

odpowiedź 26 lipca 2016 przez jpacanowski VIP (101,940 p.)

Pierwsze co mi się rzuciło w oczy to ładujące się tło. 4,2 MB to strasznie dużo. Zoptymalizuj tło https://tinyjpg.com/

Pliki .js ładuj na dole, przed </body>

Nie pisz z dużych liter, od tego jest text-transform: uppercase;

pousuwaj te  

	<div style="clear:both;height:52px;"></div>
	<footer>

Usuń tego pustego diva, a footerowi ustaw górny margines

footer {
    margin-top: 52px;
    clear: both;
}

odpowiedź 26 lipca 2016 przez IBB Gaduła (3,020 p.)

'Srona' główna? Połknąłeś 'T'.

Na mniejszych ekranach (np. 1366 x 768) na stronie głównej widać jedynie numery telefonów, logo, menu i zdjęcie. Treść poniżej zdjęcia jest całkowicie niewidoczna i nie ma niczego (oprócz paska przewijania), co by sugerowało, że strona ta nie kończy się na banerze. Jest to jedna z rzeczy, która często wychodzi przy badaniach z użytkownikami - wiele ludzi tę treść totalnie przeoczy.

Przycisk 'Menu' do uruchamiania menu na telefonach powinien być dodany do strony przy użyciu JavaScript; gdy obsługa JavaScript jest wyłączona, element ten nie pełni żadnej funkcji, jest więc niepotrzebny. Zamiast przycisku powinno być w tej sytuacji widoczne menu.