Proszę o ocenę moich 2 repozytoriów na githubie.

0

Witam,
umieściłam na githubie 2 repozytoria. Html/css- webside.
Jakoś nikt tam zadnymi poprawkami mojego kodu nie jest zainteresowany:)
Zaczęłam się tego uczyć 3 miesiące temu, więc nie wiem czy są dobrze napisane.
Proszę o ocenę kodu. Nie wiem czy te strony będą prawidłowo wyglądały na innych urządzeniach.
https://justynaolszewska.github.io/webside2-html-css/
https://justynaolszewska.github.io/try-again/

Z góry dziękuję.

1

Nie chcę naruszać twojej delikatnej psychiki, ale tam praktycznie nic nie ma.

0

To jakis troll czy co.

na gicie 'I am bigginer.' nazwa pliku webside. lol

0

@wioza:
Jak wspomniał @czysteskarpety mało jest tego kodu, ale bierz się za dalszą naukę, jeśli tylko masz chęci.

  1. https://github.com/JustynaOlszewska/webside2-html-css/blob/master/webside2.html#L121
    Unikaj pisania CSSa w HTMLu jako inline, lepiej cały CSS pisać w osobnym pliku .css, a nie łączyć go z plikiem html. To jest zasada - nie należy pisać css inline.

  2. https://github.com/JustynaOlszewska/webside2-html-css/blob/master/webside2.html#L8
    Lepiej wstawiać href jako ...href="css/style.css"..., gdzie css to folder w miejscu, gdzie znajdują się twoje pliki html oraz sam plik CSS lepiej nazywać typu style.css i nie należy nazywać pliku jeszcze ze słowem css w jego nazwie ;) Wystarczy .css jako format pliku.

  3. W CSSie dodawaj sobie spację po nazwie znacznika czy klasy przed klamrą. Powinno być:

.class {
    font-size: 12px;
}

Tak samo po własności CSSa i dwukropku wstawiaj spację jak wyżej, a nie font-size:12px;

  1. Nazwa plików poczytaj jak należy prawidłowo nazywać pliki, bo nie wiem skąd wpadłaś na takie rozwiązanie ;)
0

@wioza: Spojrzałem na Twojego githuba i commity jakie teraz robisz. Nie commituj zmiany nazwy pliku jako nowy commit i tak po kolei, bo co by było, jakbym w swoim projekcie zmienił 10 drobnych rzeczy i commitował każdą osobno? Po paru dniach miałbym kilkaset commitów. Commitujesz po pewnych 'znaczących' zmianach. Przerzucasz css inline do pliku css, zmieniasz nazwy plików na prawidłowe i dopiero commitujesz sobie jak chcesz jako np Reorganize project structure to fulfil good practices.

You commit when you have reached a codebase state you want to remember. There are a lot of reasons why you might want to remember a particular codebase state, so there can't be hard-and-fast rules on when to commit. However, number of commits is definitely not a measure of quality or progress.

0

Nie ukrywam, że mi wstyd :)
Poprawiłam wszystkie błędy w website2, które wskazaliście.
Jeśli jeszcze macie coś do napisania, to piszcie.

1 użytkowników online, w tym zalogowanych: 0, gości: 1