Tworest Opublikowano 10 Kwietnia 2021 Udostępnij Opublikowano 10 Kwietnia 2021 Projekt został zakodowany z layoutu pana @damson już to stare ale uczę się i myślę że nawet spoko mi to wyszło. lekko zmienione są niektóre rzeczy. ten ciemny pasek z lewej strony to od snapshota z opery 😕 Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
Założyciel SeNioR Opublikowano 11 Kwietnia 2021 Założyciel Udostępnij Opublikowano 11 Kwietnia 2021 @Tworest Ale jak mamy ocenić kod bez live demo? 🙂 2 Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
Tworest Opublikowano 11 Kwietnia 2021 Autor Udostępnij Opublikowano 11 Kwietnia 2021 (edytowane) 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 11 Kwietnia 2021 przez Tworest Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
Założyciel SeNioR Opublikowano 13 Kwietnia 2021 Założyciel Udostępnij Opublikowano 13 Kwietnia 2021 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. A i duże grafiki dodawaj w nowym formie WebP, dla szybszego wczytywania. 3 Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
CzareK Opublikowano 13 Kwietnia 2021 Udostępnij Opublikowano 13 Kwietnia 2021 @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. 3 Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
Ekspert -n3veR Opublikowano 13 Kwietnia 2021 Ekspert Udostępnij Opublikowano 13 Kwietnia 2021 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). 3 Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
Ekspert aXenDev Opublikowano 13 Kwietnia 2021 Ekspert Udostępnij Opublikowano 13 Kwietnia 2021 (edytowane) Nie stylujemy po ID!!! <button><a href="">Poznaj nas</a></button> Albo button, albo link. Nie ma nic pomiędzy. <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. <h1>Damian</h1> <h2>Grafik</h2> To też jest źle zrobione. Znaczniki H muszą mieć jakieś znaczenie, a nie pojedyncze wyrazy. <footer> <span>2021©</span><h1>Tworest i Mateo</h1> <i class="fab fa-facebook"></i> </footer> W stopce h1? A po co? Inputy nie są podpisane (Nie mają label), <br> Nie używaj już stylowania z poprzednich wersji HTML. Użyj do tego CSS. W CSS staraj się używać flexa lub grida. Unikaj używania float jeżeli nie jest to konieczne. Zachęcam do korzystania ze zmiennych w CSS 🙂 Brak RWD, href nie może zostawać pusty. Zamiast tego w zawartość wpisz #, Swoje prace wstawiaj na Github, Do deployu strony możesz się posłużyć Vercel lub Github Pages Edytowane 13 Kwietnia 2021 przez aXenDev 6 Odnośnik do komentarza Udostępnij na innych stronach Więcej opcji udostępniania...
Rekomendowane odpowiedzi
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ę