- Strona dostępna pod 4 adresami. www, non-www, index.html, non- index.html. Wypadałoby ujednolicić adres. Unikasz wtedy duplikowania treści i problemów z Same Origin Policy. No i jest porządek : )
-
<link href='https://fonts.googleapis.com/css?family=Roboto+Condensed:400,700,300' rel='stylesheet' type='text/css'>
Od HTMLa 4. atrybut type jest niepotrzebny
-
<div class="blur"></div>
Mam wrażenie, że to można zakodować w nawigacji, unikając tym samym niepotrzebnego diva.
-
<a class="anchor" id="sprawdz"></a>
Dodawaj id / kotwice bezpośrednio do odpowiednich sekcji. W ten sposób unikasz tworzenia pustych elementów.
-
<a href="#sprawdz" title="Brand" class="brand">Szymon Ryczek</a>
Tak na moje oko artrybit title nie powinien brzmieć Brand : d
-
Menu nie da się korzystać z poziomu klawiatury, przez co jest kompletnie niedostępne. Powinno być również widoczne, gdy wyłączy się JavaScript ( to samo tyczy się elementów portfolio )
-
Te nieklikalne ikonki (puste elementy!) w menu .. to już pominę : D
-
W animacji tła, skakanie kadru o kilka pixeli w lewo i prawo - parę razy na sekundę - jest drażniące.
-
Logicznie - nie pasuje mi aktualny outline dokumentu. Głównym nagłówkiem w tym przypadku powinno być Twoje imię i nazwisko.
-
<h1>Nowoczesne i szybkie <span class="block">strony internetowe</span></h1>
<hr>
Zamiast <hr> border-bottom dla h1 - lub stworzenie linii na pseudo-elemencie. <hr> ma swoje semantyczne znaczenie:
The HTML <hr> element represents a thematic break between paragraph-level elements (for example, a change of scene in a story, or a shift of topic with a section).
-
<div class="poster">
<img src="assets/images/header-background.jpg" alt="">
</div>
Co nie niesie ze sobą żadnej treści - wywal do CSSa.
-
To samo tutaj
<img src="assets/images/mrgeek.png" alt="">
Swoją drogą na tekście w obrazku - jest zdecydowanie za mały kontrast.
-
Całkiem ładnie zrobiona sekcja "Umiejętności" : )
-
<div class="gap"></div>
<div class="gap"></div>
Co to, po co i do czego? Z pewnością, da się bez tego.
-
<a href="http://musicfreekz.com">Sprawdź</a>
Treść linków powinna jednoznacznie wskazywać na element docelowy. Technologie asystujące tworząc dodatkowe menu z linków na stronie, stworzą kilka pozycji o nazwie Sprawdź.. ok..ale co mam sprawdzić?
-
Obowiązkowo dodaj label do inputów w formularzu.
-
<input class="button-send" type="submit" value="Wyślij">
Sam oznaczyłeś ten element jako button (button-send) - jeśli coś wygląda jak button - w kodzie też powinno być buttonem.
Pro-tip: Jeśli w formularzu występuje tylko jeden button, domyślnie jest on submitem, a atrybut type="submit" można pominąć.
-
<footer>
<hr>
Znów hr : d
-
Nie filtrujesz danych. Po wysłaniu formularza bez podania maila wyskakuje komunikat, że mail został wysłany.
-
Co do SweetAlert - denerwuje mnie znikanie scrollbara - cały układ strony zaczyna skakać.
-
Na mobile, rozdzieliłbym inputy - jeden input w jednej linii - robi się tam teraz bardzo ciasno.
-
Wypadałoby przypisać CSS hoverów do CSSa focusów. W wielu miejscach focus jest zbyt mało widoczny.
-
Focus nie powinien znikać gdzieś w otchłań, podczas przeglądania sekcji Portfolio.
-
Page Speed Insights - tam jeszcze kilka porad ( jak dla mnie wynik 90 to minimum )
Ogólnie całkiem ładna strona. Wiele rzeczy zostało poprawnie zrealizowanych.