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

Projekt strony. Ocena.

VPS Starter Arubacloud
0 głosów
490 wizyt
pytanie zadane 17 lutego 2018 w Nasze projekty przez Michał Gibas Pasjonat (19,610 p.)
Witam!

Na codzień nie zajmuję się webdevem, ale postanowiłem się trochę przyuczyć frontendu. Poniżej wysyłam linka do jednego z moich pierwszych frontendowych projektów i proszę o ocenę designu strony i kodu. Jestem gotowy na konstruktywną krtykę ;)

https://madrussianhacker.github.io/PdEngineWebsite/index.html

Pozdrawiam!

5 odpowiedzi

+1 głos
odpowiedź 17 lutego 2018 przez Daniel90 Pasjonat (17,970 p.)
Na pewno bym ściaśnił te menu bo od lewa do prawa trochę trzeba nalatać myszką ;p
+1 głos
odpowiedź 17 lutego 2018 przez Wionek Użytkownik (670 p.)
Fajna tylko nad RWD popracuj i będzie git :)
+1 głos
odpowiedź 17 lutego 2018 przez Beginer Pasjonat (22,110 p.)
Oglądam na tablecie.  Tutaj proporcje elementów w stosunku do ekranu są  dobre. Strona wyświetla się prawidłowo, choć z jednym, istotnym mankamentem. W układzie pionowym tytuł strony"Palladium engine" wyświetla się zbyt małą czcionką - mniejszą od pozostałych napisów, i nie jest wyśrodkowany. W układzie poziomym jest wszystko w porządku.

Od strony forhendu witryna wygląda ładnie - schludnie, poważnie, porządnie.
+1 głos
odpowiedź 17 lutego 2018 przez imklau Nałogowiec (42,090 p.)

jeśli chodzi o kod to zerkaj sobie od razu tutaj: https://validator.w3.org/nu/?doc=https%3A%2F%2Fmadrussianhacker.github.io%2FPdEngineWebsite%2Findex.html

<h2><a href="https://www.libsdl.org/license.php">[SDL license]</a></h2>

to raczej nie ma sensu, wystarczy sam link

<nav id="hornav">
            <a href="index.html">Home</a>
            <a href="license.html">License</a>
            <a href="#">Download</a>
            <a href="docs.html">Documentation</a>
            <a href="#">Tutorials</a>
            <a href="javascript:void(0);" class="icon" onclick="toggleResponsiveMenu()">&#9776;</a>
</nav>

to tak czy inaczej powinna być lista <ul>

<span class="scrollToTop fixed" onclick="scrollToTop()"><i class="icon-up-open"></i></span>

kliknij sobie jakiś przycisk a nie <span> :D tutaj akurat powinien też być link + nie stosuj onclick tylko nasłuchuj na zdarzenia za pomocą addEventListener

<div style="padding: 10pt;">
                <i>"Spreading the idea of freedom is a big job — it needs your help."</i> ~ Richard Stallman
</div>

serio przypisujesz tak style? zapewne nie przeglądałeś ocen innych stron na forum? a bardzo to polecam :D i dlaczego tam jest znacznik <i> ?

I serio przejrzyj projekty innych osób, bo błędy się powtarzają :)

Powodzenia dalej!

+1 głos
odpowiedź 17 lutego 2018 przez Tomek Sochacki Ekspert (227,510 p.)

Parę sugestii jeśli chcesz iść we front webowy:

  1. zapoznaj się z webpack (np. na blogu nafrontendzie.pl kolega Bartek ma kilka artykułów wprowadzających w tajniki webpack). Ewentualnie inne, gulp itp. ale polecam webpack bo sam z nim pracuję, a innych za dokładnie nie znam.
  2. skrypty JS ładuj na końcu, przed zamknięciem </body>. Nie ma sensu wstrzymywać wczytywania strony na ładowanie skryptów. Ewentualnie poczytaj o atrybutach async/defer.
  3. Docelowo zapoznaj się z minifikatorami kodu css i js, np. w formie pluginów do wspomnianego już webpacka.
  4. Do takiego projektu jQuery spoko, może być, ale jeśli chciałbyś coś poważniej myśleć o pracy fronta to musisz poznać czysty JS. Ale to już zależy do jakich celów się uczysz "webu".
  5. Nie stosuj onClick w HTML tylko poczytaj o addEventListener i rozdziel treść/semantykę w HTML od interakcji/zdarzeń w JS.
  6. "href="javascript:void(0);"" na ikonce... a czemu nie po prostu element <button> skoro to de facto jest przycisk? Ostatecznie np. <div>, ale w pierwszej kolejności lepiej rozważyć button.
  7. "<div style="padding: 10pt;">" od tego jest CSS. Nie stosuj css inline. Osobiście dopuszczam to tylko sporadycznie np. do background-image ale to na prawdę ostrożnie i rzadko. 
  8. Nie stosuj w CSS jednostek PT. Polecam raczej rem/em, vw/vh i procenty, a w ostateczności px, ale to z ostrożnością ze względu na RWD.

To tak na szybko, gdyby coś było nie jasne to pisz.

Pozdrawiam

Podobne pytania

0 głosów
3 odpowiedzi 500 wizyt
pytanie zadane 4 lipca 2018 w Nasze projekty przez Layoutowiec Mądrala (5,470 p.)
0 głosów
0 odpowiedzi 278 wizyt
pytanie zadane 25 września 2017 w Nasze projekty przez Mistrz0000000 Bywalec (2,780 p.)
0 głosów
2 odpowiedzi 415 wizyt
pytanie zadane 18 lipca 2018 w Nasze projekty przez mgita Nowicjusz (120 p.)

92,453 zapytań

141,262 odpowiedzi

319,088 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!

...