Moja pierwsza strona, prosze o ocene.

Question

Witam

prosze o opinie na temat mojego projektu:

matcygal.github.io

pewnie masa bledow ktore doswiadczone oko wychwyci od razu.

jest to typowy template wiec nie  przywiazywalem wagi do contentu. 

Pozdrawiam

Comments

Moje w ogóle niedoświadczone oko wychwyciło brak alt przy obrazkach

---

Przy obrazkach warto też ustawiać atrybut draggable na false.

---

Spróbuj nawigować po stronie za pomocą tab. Można? Nie? Popraw to!

---

nie zabardzo rozumiem co masz na mysli :)

---

Wciskasz tab i jedziesz po stronie. W większości przypadków nie widać gdzie znajduje się fokus, lub tekst pod nim jest niewidoczny. Wystarczy w większości przypadków ustawić taki sam styl co dla hover dla focus.

Answer 1

prosze tez o ocene designu

Answer 2

Popraw sobie następujące rzeczy:

1.Przy załadowaniu strony menu u góry nieco zlewa się ze zdjęciem, na moim ekranie jedna opcja jest całkowicie na białym tle i jej nie widać.

2. Przy buttonie "contact us" mógłbyś dodać jakieś opóźnienie, żeby strona "sama się przescrollowała" do formularza

3. Przy wysłaniu formularza występuje błąd 405 (nie wiem czy to miało działać czy nie, ale w razie czego zwracam uwagę, że nie działa)

4. Na wersji mobilnej pole message jest moim zdaniem nieco zbyt duże.

5. Tak jak ktoś wcześniej wspomniał brak 'altów' w znaczniku img. W HTML5 są one wymagane.

Ale poza tym strona jest bardzo ładna. Kolorki, czcionka, obrazki, wszystko ze sobą współgra. Nie wiem jak innym ale mi się podoba 

Answer 3

1. Na przyciskach i social icons dobrze by było użyć transition.
2. "WHERE TO FIND US?", moim zdaniem ta sekcja wygląda trochę pusto. Spróbuj może dać mapę jako tło sekcji i na tym umieścić ten tekst.
3. W sekcji CONTACT użyj może jaśniejszego koloru czcionki, np. #f5f5f5
4. Po scrollu w dół tło headera jest (jak dla mnie) za bardzo przeźroczyste tło. Możesz spróbować dać ciemny kolor czcionki i tło mniej przeźroczyste, a bardziej białe tło + dodać dolny cień do headera.

Poza tym wygląda świetnie. 

Comments

w where to find powinna byc mapka googla

---

i jest, tylko chodzi mi o rozciągnięcie jej na całe to szare tło i informacje umieścić na mapce (usunąć też Google GUI z mapki).

Teraz zauważyłem, że jeszcze wszędzie używasz float i czyścisz to w ::after. Spróbuj używać display: inline-block

---

nie, nie wsadzam, robi to za mnie bootstrap:)

Answer 4

ok, wpadlem, 2 problemy:

1. testowalem strone na iphonie zony i backgroud-image wyglada kiepsko, nie skaluje go i widac tylko jakies 10% obrazu - lewy gorny rog. na moim androidzie wygloda ok.

2. czasem zeby zobaczyc mapke w sekcji where trzeba odswierzyc kilka razy.... ktos wie moze czemu?

Comments

1. Widzę, że sobie poradziłeś.

2. 

<script src="https://maps.googleapis.com/maps/api/js?key=AIzaSyBV6q-QfMqsizkXknLvHi6gtTfZHdrunEk&amp;callback=initMap" async="" defer=""></script>

ładujesz przed main.js a funkcję initMap() definiujesz w tym właśnie pliku. Najprościej: Daj tą linijkę na koniec body.
*odświeżyć