Quantcast
Channel: Forum Pasja Informatyki - Najnowsze pytania i odpowiedzi
Viewing all articles
Browse latest Browse all 83215

Odpowiedź: Code review strony

$
0
0

Polecam ponawigować po stronie przy pomocy klawiatury.

<li><a class="nav-a" href="#" onclick="scroll_to('#nav-introduction');">Wstęp</a></li>

Link powinien być linkiem a nie taką atrapą. http://pornel.net/onclick + http://codepen.io/Comandeer/pen/bEPbqg

To, że masz obrazki w SVG, nie zwalnia Cię z obowiązku podawania tekstu alternatywnego (czy to via span.visuallyhidden, czy przy pomocy [aria-label] bezpośrednio na svg).

<canvas id ="canvas" width="400" height="60">
		    		<div class="txtStyle">Damian Idczak</div>
				</canvas>

To nie ma sensu. Jak już, to bym pokombinował ponownie z SVG tutaj.

BTW z imienia i nazwiska zrobiłbym raczej część nagłówka.

<figure id="nav-introduction" class="container-img">
							<img src="images/b1t.png" alt="część grafiki mózgu" class="brain color-one" />
							<img src="images/b2t.png" alt="część grafiki mózgu" class="brain color-two" />
							<img src="images/b3t.png" alt="część grafiki mózgu" class="brain color-three" />
							<img src="images/b4t.png" alt="część grafiki mózgu" class="brain color-four" />
						</figure>

Bezsensowny [alt]. Prawdę mówiąc ten mózg jest ozdobnikiem, więc obrazki powinny mieć puste [alt], a samo figure tutaj nie pasuje.

Artykuły i sekcje bez nagłówków są bez sensu. Podział na nie jest bowiem robiony właśnie wg nagłówków. Jeśli masz artykuł bez nagłówka to a) nie jest to artykuł b) powinieneś umieścić nagłówek.

<div class="skills"><img src="images/html.png" alt="obrazek z grafiką html" class="image-skills"/></div>

Twoim skillem jest "obrazek z grafiką html"? No właśnie… Poza tym style to nie figure. To, że coś jest obrazkiem, nie oznacza, że trza to wsadzić w figure (ba, praktycznie żaden obrazek na tej stronie – oprócz Twojego zdjęcia – nie powinien być w figure).

<p class="vertical-text">Umiejętności</p>

To powinien być nagłówek sekcji, umieszczony na jej początku, nie końcu.

<article class="article-five">
					<aside class="sidebar">
						<p class="vertical">Chcę zdobywać wiedzę</p>
					</aside>

					<figure class="container-want-learn">
						<img class="images-want-skills" alt="obrazek z grafiką JavaScript" src="images/js.png">
						<img class="images-want-skills" alt="obrazek z grafiką bootsrap" src="images/bootsrap.png">
						<img class="images-want-skills" alt="obrazek z grafiką github" src="images/github.png">
					</figure>

				</article>

To nie ma sensu. aside jest nagłówkiem, a figure nie powinno wgl tutaj być.

Ikonki powinny mieć [title]– nie każdy kojarzy wszystko po logo (zwłaszcza, że np. dla JS używasz niestandardowego logo).

<aside class="aside-six">
					<h2>Każdy kiedyś zaczynał.</h2>
				</aside>

To nie jest nagłówek.

Formularz musi mieć label.

<aside class="footer-sidebar-two">
					<figure class="container-logo-footer">
						<a href="#"><img class="logo-footer" src="images/wolf-logo-footer.png" alt="logo w stopce strony" /></a>
					</figure>
				</aside>

Kod totalnie bez sensu. Zacznijmy od tego, że zrobiłeś z tego poboczną treść sekcji z kontaktem. Dodatkowo znów pojawia się figure, które tutaj nie ma sensu (ten obrazek nie jest "załącznikiem" do jakiejś treści, bo sam jest tą treścią!). Co więcej – [alt] jest całkowicie bezsensowny. Ten link nie prowadzi do "logo w stopce strony". Polecam poczytać choćby w specce HTML5 jak należy opisywać obrazki → https://www.w3.org/TR/html5/embedded-content-0.html#alt

Poza tym kontakt wyciągnąłbym poza stopkę.

Po co Ci 2 wersje jQuery – zwłaszcza, że nowszą nadpisujesz starszą? Wgl jQuery powinno być załączane na końcu body.


Viewing all articles
Browse latest Browse all 83215