• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Proszę o ocenę jakości kodu oraz strony internetowej.

VPS Starter Arubacloud
0 głosów
629 wizyt
pytanie zadane 17 marca 2017 w Rozwój zawodowy, nauka, praca przez Artek Stary wyjadacz (11,800 p.)
Witam wszystkich. Jestem w trakcie tworzenia strony dla kolegi, który pracuje w firmie sprzedającej/serwisującej kotły grzewcze jacuzzi itd.

Strona jeszcze nie jest gotowa, póki co dostępne są zakładki "firma" i kontakt. Chciałbym się za wczasu dowiedzieć czy popełniłem jakieś błędy aby poprawić się w przyszłości.

Proszę o szczegółowe uzasadnienie zgłaszanych wątpliwości.

http://sh194765.website.pl/
komentarz 18 marca 2017 przez Arkadiusz Waluk Ekspert (287,550 p.)
@Artek gdy chcesz się odnieść do czyjejś odpowiedzi to rób to proszę w komentarzu do niej, a nie kolejnej odpowiedzi, jak robiłeś dotychczas. Odpowiedzi, jak sama nazwa mówi, to mają być odpowiedzi innych na pytanie. Poza tym robi się w dyskusji straszny bałagan - przed chwilą coś połączyłem, ale do tej pory nie wiem czy dobrze to zrobiłem. A tak mamy pod każdą odpowiedzią komentarze, które jej dotyczą.
komentarz 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Usunąłeś parę wypowiedzi. Możesz je przywrócić? Były przydatne...
komentarz 19 marca 2017 przez Arkadiusz Waluk Ekspert (287,550 p.)
Niczego nie usunąłem. Wszystkie Twoje odpowiedzi (i komentarze pod nimi) zamieniłem na komentarze pod najlepszą odpowiedzią - jak zrozumiałem to co w nich było, dotyczyło właśnie tej odpowiedzi. Jeśli zrozumiałem źle to sorki, ale jak pisałem, przez dodanie przez Ciebie kilku odpowiedzi zrobił się taki bałagan, że ciężko było się w tym połapać.
komentarz 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Mój bład, Część komentarzy się schowała i trzeba rozwinąć.

5 odpowiedzi

+3 głosów
odpowiedź 17 marca 2017 przez xmentor Nałogowiec (49,520 p.)
wybrane 18 marca 2017 przez Artek
 
Najlepsza
<link href='http://fonts.googleapis.com/css?family=Athiti&subset=latin,latin-ext' rel='stylesheet' type='text/css'>
<link href='http://fonts.googleapis.com/css?family=Archivo+Narrow&subset=latin,latin-ext' rel='stylesheet' type='text/css'>

Czcionki można pociągnąć jednym linkiem: family=czcionka1|czcionka2

Atrybut type zbędny.

Skrypty(jQuery itp.) możesz przenieś na koniec ciała, tuż przed </body>.

<title>Nord Team Service</title>
<meta charset="utf-8">

Wydaje się, że meta z kodowaniem powinna być pierwsza.

<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">

chrome=1, jest zbędne - Chorme Frame umarł.

Do ikon lepiej używać znacznika span niż i - https://comandeer.github.io/blog/a11y/html-css/2017/02/28/o-ikonkach-slow-kilka.html

Sposoby kontaktu(mowa o divie z numerami telefonu i e-mailem) można by wrzucić do jakiejś listy np. dl - sam e-mail powinien być linkiem z href-em zawierającym mailto:twój@mail.pl.

Menu powinno być oparte na liście ol/ul - zamiast na mniejszych ekranach wyświetlać menu oparte na select, możesz ukryć menu zbudowane na liście ol/ul i dodać jakiegoś hamburgera, który będzie kontrolował pokazywanie tego menu.

Brakuje atrybutu alt w większości obrazków, jeżeli nawet są to elementy dekorujące, powinny mieć pusty alt.

Brakuje mi paragrafów.

Odwiedź nas na  <a target="_blank" href="https://www.facebook.com/NORD-TEAM-238591496164650/?fref=ts"><i class="fa fa-facebook-square"></i></a>

Pusty znacznik a, żadnej treści nie niesie ze sobą - dodatkowo brak aria-hidden.

<div id="pipes-image" class="fading-images"></div>
<div id="jacuzzi-image" class="fading-images"></div>

Tutaj tez pusto.

<input id="user-email-newsletter" placeholder="e-mail" type="text" name="user-email">

type można zmienić na email.

Mało widoczny fokus np. na inputach w formularzu.

Podstrona Kontakt:

Duża ta czcionka.

Zrób z tych numerów i e-maila(e-mail powinien być linkiem), listę - jak już wyżej wspomniałem.

Inputy powinny mieć etykiety(label).

Warto również oznaczyć, które pola są obowiązkowe do wypełnienia.

Sposób weryfikacji(przepisywanie kodu z obrazka) jest niedostępny.

komentarz 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Ale nie wczytuje stylów czy po prostu nie podoba Ci się wygląd tego select'a? Jeżeli po prostu Ci się nie podoba to rozumiem, ale Win98 to chyba jednak przesada :)
komentarz 19 marca 2017 przez xmentor Nałogowiec (49,520 p.)
Stylowanie selecta jest mocno ograniczone, szczególnie tej części rozwijalnej - pogoogluj :)
komentarz 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Co do reszty uwag. Rzeczywiście na msdn rekomendują umieszczeni biblioteki jQuery i skryptów przed znacznikiem </body>. https://msdn.microsoft.com/pl-pl/library/samouczek-jquery--rozpocznij-prace.aspx

To zostanie poprawione.

Zastanawiam się tylko dlaczego nikt nie wpadł na to aby stworzyć screen-reader zdolny do odczytania atrybutu placeholder..
komentarz 21 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
A właśnie znalazłem całkiem spoko stronkę właśnie z selectem jako menu nawigacyjne.

Zastanawiam się czy rzeczywiście macie rację z tym aby tego nie używać w ten sposób...

http://polteknik.pl/pl/
komentarz 21 marca 2017 przez xmentor Nałogowiec (49,520 p.)
"Spoko"? Wystarczy zajrzeć w kod i już nie jest spoko.

Nie ma co się zastanawiać i po prostu zapomnieć o tym pomyśle.
+1 głos
odpowiedź 17 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
No dobrze. Dziękuję bardzo wszystkim za uwagi, są one dla mnie bardzo cenne gdyż bardzo zależy mi na popchnięciu skill'a do góry. Postaram się poprawić wszystkie niedoskonałości.

Oczywiście jeżeli ktoś chciałby coś jeszcze dodać to jestem otwarty na dalsze uwagi. :)
0 głosów
odpowiedź 17 marca 2017 przez Filius Gaduła (4,120 p.)
menu za duża czcionka, tekst lorem ipsum też troche zbyt duży
0 głosów
odpowiedź 17 marca 2017 przez jaca121212 Nałogowiec (40,760 p.)

►Warto zastosować https://css-tricks.com/the-current-state-of-telephone-links/

►fanpage możesz dać ciut większe albo zrobić tak jak tutaj na stronie  fajnie to wygląda,

►popraw stronę kontakt nie jest estetyczny,

►także do menu możesz zastosować i nie tylko do menu https://css-tricks.com/bubble-point-tooltips-with-css3-jquery/

zamiast 

<span id="e-mail" class="contact-info floated">nordteamservice@interia.pl</span>

użyj

https://css-tricks.com/snippets/html/mailto-links/

► mniejsza czcionka 

0 głosów
odpowiedź 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Interesuje mnie tylko jeszcze jedna kwestia. Nadal nie rozumiem dlaczego stylowanie po id jest złe. Rozumiem na czym polega wspomniana "specyficzność" w css, ale nie rozumiem w czym problem.
komentarz 19 marca 2017 przez xmentor Nałogowiec (49,520 p.)
Linkowałem to już wczesniej i linkuje jeszcze raz: https://forum.pasja-informatyki.pl/109776/html-class-czy-id%C2%A0

Komentarze pod wypowiedziami też poczytaj.
komentarz 19 marca 2017 przez Artek Stary wyjadacz (11,800 p.)
Brakuje konkretnych przykładów. Tutaj jest to opisane znacznie lepiej http://oli.jp/2011/ids/

Podobne pytania

+2 głosów
4 odpowiedzi 318 wizyt
0 głosów
6 odpowiedzi 416 wizyt
pytanie zadane 4 grudnia 2016 w HTML i CSS przez niezalogowany
0 głosów
1 odpowiedź 153 wizyt
pytanie zadane 24 października 2017 w PHP przez Paweł Antyporowicz Stary wyjadacz (11,470 p.)

92,452 zapytań

141,261 odpowiedzi

319,074 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...