Chyba coś się porozjeżdżało :)
Moim zdaniem wejście do posta powinno być dostępne zarówno przez klik na obrazek, tytuł, opis jak i czytaj dalej. Z punktu UX obecna forma jest wg mnie błędna.
Hover na tym buttonie czytaj dalej fajnie gdyby jakoś wskazywał, że mogę to kliknąć.. owszem, jest cursorek ale może jakiś border, background-color itp.?
I dlaczego nie jest to button skoro taką pełni funkcję? div -> li -> a to błędna struktura, li powino być zagnieżdżwone w ul albo ol.
Menu jest dla mnie kompletnie niewidoczne, a po za tym to dlaczego na desktopie nie dasz pełnego menu, a tzw. hamburger tylko dla mobile?
Osobiście bardzo drażni mnie jak lekceważy się UX i każe userowi latać po ekranie... na przykład klikam w hamburgera i szukam menu... nagle zauważam, że wylatuje z lewej strony... mega irytujące jak dla mnie.
Dlaczego przy przejściu np. na recenzje najpierw dostaję stronę, potem chwilowy preloader i znowu stronę? Coś z tym preloaderem jest nie tak bo odpala się chyba po fakcie :)
Po za tym wizualnie strona wydaje mi się strasznie pusta, bez jakieś treści, bez energii itp. Wchodzę i w sumie nic tu nie widzę..
<link href="https://fonts.googleapis.com/css?family=Eczar:500" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Arvo" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Lato" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Source+Sans+Pro:600" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Raleway:300i" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Slabo+27px" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=M+PLUS+Rounded+1c" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=PT+Sans" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Merriweather" rel="stylesheet">
<link href="https://fonts.googleapis.com/css?family=Palanquin" rel="stylesheet">
Naprawdę używasz wszystkich tych fontów :)? Zostaw tylko to, co faktycznie używasz, niepotrzebnie podciągasz userowi cała masę zbędnych czcionek... Po za tym te fonty raczej się nie zmieniają i można by je zaserwować lokalnie.
Po co Ci reset CSS? To robiło się kilka lat temu... dzisiaj aplikacje webowe tworzy się komponentowo, resety do niczego nie są potrzebne :)
Zastanawia mnie sens używania i podciągania jQuery w tym projekcie...:
$(function() {
$('.navigation-toggle').on('click', function(e) {
$('body').toggleClass('navigation-show');
});
});
w czym to jest niby krótsze i przyjemniejsze niż:
const nav = document.querySelector('.navigation-toggle');
nav.addEventListener('click', () => {
document.body.classList.toggle('navidation-show');
})
Nie ma co na siłę podciągać jQuery dla paru prostych zastosowań, niepotrzebny request dla usera.
Teraz laptopa możemy dostać w trzech kolorach:<br>
-złoty<br>
-srebrny<br>
-gwiezdno szary<br><br>
<br>
A dlaczego ten content nie posiada po prostu znaczników <p> i normalnych list, np. <ul> z elementami <li>? Mógłbyś wtedy bardzo łatwo stylować sobie wszystko dla całego komponentu posta.
<div class="hr2">
<br><br><br><hr width="20%;" style="margin-top: 27%; margin-right: 6%;">
</div>
co to za cudo:)? Kompletnie zbędny kod... po pierwsze to CSS w plikach CSS a nie inline, a po drugie to robisz coś bardzo nie tak skoro potrzebujesz w html takie coś...
Praktycznie w ogóle nie stosujesz znaczników <p> gdzie ich miejsce, tylko usilnie ładujesz divy... zła praktyka.
Najpierw:
<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.10.2/jquery.min.js"></script>
a potem:
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.2.4/jquery.min.js"></script>
zdecyduj się... nie dość że tu jQuery jest w ogóle zbędne dla tych paru linii kodu to nigdy nie podciagaj kilku wersji tej samej libki... to bardzo duży błąd... mojego CR na pewno być czymś takim nie przeszedł :)
Dlaczego podciągasz pusty plik script.js?
Popper, bootstrap, scrollreveal... serio.. tyle libek do tak malej strony...?
Generalnie moim zdaniem strona wymaga całkowitej przebudowy od strony kodu... wizualnie to kwestia gustu, ale kod wg mnie do poprawy zarówno html, jak i css oraz JS - zupełnie zbędne jQuery dla paru linijek kodu js...
