Skocz do zawartości
Pecetowicz Forum komputerowe

Projekt zakodowany z PSD do oceny.


Rekomendowane odpowiedzi

3 godziny temu, SeNioR napisał:

@Tworest Ale jak mamy ocenić kod bez live demo? 🙂 

Przepraszam już daje. Przypominam strona nie ma jeszcze skalowania robiłem ją na zoom 50% mam dość mały monitor i no jakoś tak wyszło 

http://salvarp.ct8.pl Dałem na starej domenie nie mogłem zrobić drugiej 

Edytowane przez Tworest
Odnośnik do odpowiedzi
Udostępnij na innych stronach
  • Założyciel

Składnia kodu jest w porządku, używasz angielskich nazw klas, kod jest schludny i przejrzysty, oddzielasz poszczególne sekcje komentarzami.

Niestety brakuje responsywności, tekst w niektórych miejscach ma za duży rozmiar, dolna sekcja do poprawy.

Tekst Placeholder jest zbyt duży, przycisk mógłbby być zaokrąglony a napis równo, w stopce napis mógł być jednolitego koloru oraz znacznie mniejszy. 

JwwuvPV.png

A i duże grafiki dodawaj w nowym formie WebP, dla szybszego wczytywania.

Odnośnik do odpowiedzi
Udostępnij na innych stronach

@Tworest błąd:

<h1 class="col-1">Stwórz juz teraz<span> własną markę</span></h2> 
<h2 class="col-2">razem z nami !</h3>

Zazwyczaj nagłówka poziomu H1 używa się jedynie raz na stronie. Co do RWD możesz skorzystać z frameworka typu Bootstrap 5, gdzie łatwiej będzie Ci napisać wersję responsywną z podejściem mobile first.

Screenshot_2021-04-13 Loyaut NEW.png

Odnośnik do odpowiedzi
Udostępnij na innych stronach
  • Ekspert
1 godzinę temu, CzareK napisał:

Zazwyczaj nagłówka poziomu H1 używa się jedynie raz na stronie.

Jest już obalone to stwierdzenie.
Nie mam co prawda pod ręką artykułu na ten temat, ale `h1` możesz użyć także wewnątrz `article` i SEO na tym nie traci.

Ale cenna uwaga, bo sam heading jest źle zbudowany (zamykany).

Odnośnik do odpowiedzi
Udostępnij na innych stronach
  • Ekspert
  1. Nie stylujemy po ID!!!
  2. <button><a href="">Poznaj nas</a></button>

    Albo button, albo link. Nie ma nic pomiędzy.

  3. <h1 class="col-1">Stwórz juz teraz<span> własną markę</span></h2>
    <h2 class="col-2">razem z nami !</h3>

    Brak zamkniętego znacznika h1, nie mówiąc już o tym że h2 i h3 to tutaj nie jest potrzebne.

  4. <h1>Damian</h1>
    <h2>Grafik</h2>

    To też jest źle zrobione. Znaczniki H muszą mieć jakieś znaczenie, a nie pojedyncze wyrazy.

  5. <footer>
    	<span>2021©</span><h1>Tworest i Mateo</h1>
    	<i class="fab fa-facebook"></i>
    </footer>

    W stopce h1? A po co?

  6. Inputy nie są podpisane (Nie mają label),

  7. <br>

    Nie używaj już stylowania z poprzednich wersji HTML. Użyj do tego CSS.

  8. W CSS staraj się używać flexa lub grida. Unikaj używania float jeżeli nie jest to konieczne.

  9. Zachęcam do korzystania ze zmiennych w CSS 🙂

  10. Brak RWD,

  11. href nie może zostawać pusty. Zamiast tego w zawartość wpisz #,

  12. Swoje prace wstawiaj na Github,

  13. Do deployu strony możesz się posłużyć Vercel lub Github Pages

Edytowane przez aXenDev
Odnośnik do odpowiedzi
Udostępnij na innych stronach

Problem wciąż nierozwiązany? Dodaj swoją odpowiedź

Jeśli chcesz dodać odpowiedź, zaloguj się lub zarejestruj nowe konto. Jedynie zarejestrowani użytkownicy mogą komentować zawartość tej strony.

Zarejestruj nowe konto

Załóż nowe konto. To bardzo proste!

Zarejestruj się

Zaloguj się

Posiadasz już konto? Zaloguj się poniżej.

Zaloguj się
×
×
  • Dodaj nową pozycję...