Ocena strony, co w niej zmienić?

Question

Witam, tak jak w temacie jak oceniacie stronkę, co w niej zmienić? Najgorzej moim zdaniem wyszła mi oprawa strony, jak dobierać kolory, żeby najlepiej do siebie pasowały? Jakieś porady co do tworzenia wyglądu strony? Nie zwracajcie uwagi na adres strony oraz na to jakiej tematyki ona jest(Robię to co lubię, czyli rolnictwo, pisanie stron ewentualnie aplikacji oraz tworzenie filmików o tematyce rolniczej i nie tylko)

Adres strony: http://kujawynaroli.cba.pl/

Z góry dziękuję za odpowiedź, przyjmę każdą konstruktywną krytykę. :)

Answer 1

Witam, ja bym hiperłącza dał na trochę inny odcień żeby się różniły od normalnego tekstu.

Answer 2

co do kodu strony to błędy te same co i u większości osób, które wstawiały tu stronę do oceny, więc proponuję użyć wyszukiwarki na forum ;)
co do wyglądu to cóż...wygląda jak strona stworzona kilka ładnych lat wstecz, więc też proponuje rozejrzeć się po internecie i zwrócić uwagę w jakim stylu są teraz tworzone strony ;)

Comments

Z tego co mi się wydaje to jest moda na okienkowy design. Jakieś zdjęcie jako div, 1/5 jego powierzchni zajmuje jakiś pasek, na którym jest nagłówek, a całość to hiperłącze.

Answer 3

Moim zdaniem kolorystyka nie jest taka zła, ale zmieniłbym na pewno tło. Strona jest dosyć czytelna, responsywna (nie sprawdzałem na komórce). Slider napisałeś sam czy wziąłeś od Mirosława? Jeśli napisałeś sam to nieźle. Więcej nie napisze bo nie jest to jakaś super rozbudowana strona. Co do doboru kolorów polecam korzystać z palet  (znam ten ból):  http://www.colourlovers.com/palettes

http://www.color-hex.com/color-palettes/

Comments

Właśnie dobór koloru wychodzi mi najgorzej, więc na pewno skorzystam z tych palet.

Answer 4

Jak już @Noemi wspomniała, błędy te same co u większości osób wstawiających stronę do oceny.

co w niej zmienić?

Odnosząc się do kodu, to wszystko!

• Skrypty JS lepiej przenieść na koniec ciała,
• dobrze by połączyć wszystkie style w jeden plik i go zminifikować,
• nie używać np. onClick tylko addEventListener,
• poczytaj o ikonach i ich dostępności,
• jeżeli coś wygląda jak perfidna lista to lepiej zbudować to na liście ul/ol,
• nadużywasz atrybutu target,
• obrazki nie posiadają atrybutu alt,
• znacznik <br> raczej używany niezgodnie z jego przeznaczeniem,
• niepotrzebne stylowanie inline,
• divitis, gdzie znaczniki HTML5?
• Próbowałem poruszać się klawiaturą po stronie, jedynie menu ma w miarę widoczny focus,
• znacznik label jest etykietą dla pól formularzu, a Ty użyłeś go do.. w sumie nie wiem czego:

  <label class="linki_yt_fb" style="clear: both;">
    Jeżeli chcesz do mnie napisać lub znaleźć więcej informacji na mój temat zapraszam na media społecznościowe:<br>
    <a class="linkfbyt" href="http://bit.ly/2k94FRs" target="_blank">YT - Kujawy Na Roli</a><br>
    <a class="linkfbyt" href="http://bit.ly/2jFcP5F" target="_blank">YT - Spalinowe Trąbki</a><br>
    <a class="linkfbyt" href="http://bit.ly/2k3ea2h" target="_blank">FB - Kujawy Na Roli</a><br>
    <a class="linkfbyt" href="http://bit.ly/2l49J7l" target="_blank">FB - Spalinowe Trąbki</a><br>
  </label>

   

Comments

ej takiego typu informacje powinny być gdzieś wyświetlane przed wstawieniem strony do oceny :p zaoszczędziłoby to trochę czasu niektórym

---

Dzięki za rozbudowaną odpowiedź, postaram się to poprawić :)

---

A czy zamiast dodawania JS na koniec ciała można dać np. do head i dodać defer?
Czyli <script src="plik.js" defer></script>

Answer 5

W efekcie przejścia w slajderze widać tlo