[x] Nieprawidłowy użyty font, w projekcie jest Poppins do weryfikacji także wszędzie font-weight w projekcie.
[x] Brak box-shadow na hover w nawigacji.
[x] Gdy dropdown jest aktywny backdrop nie powinien przykrywać headera.
[x] Sekcja .header myśle, że lepiej by było tutaj zastosować ten element w tagu img zamiast w background img, na responsive można zastosować własność object-fit.
[x] Pewne niezgodności w font-size na elementach np. Umów się na bezpłatną wycenę.
[x] Sekcja Zobacz jak działamy zdjęcie nie powinno być w kontenerze, a lewa strona powinna być utrzymana w kontenerze. Możesz to uzyskać robiąc dodatkowa utils klasę, która będzie robiła pętle po elementach w breakponcie i dodawała padding-left do kontenera (zobacz jak tworzony jest np komponent grid).
[x] Sekcja Klimatyzatory - sprzedaż, montaż, serwis najlepsze marki została użyta animacja w sumie lepsza by było zastosowanie jakiegoś infiniteloop, bo w pewnym momencie mamy sytuacje, że elementów nie ma (możesz użyć jakiejś gotowej biblioteki)
[x] Sekcja SDM-Klient w liczbach plus w animacji mógłby zostać dodany w sumie na sam koniec jako callback.
[x] Linie poziome (gradient) powinny mieć wysokość 1px.
[x] W sumie niepotrzebnie dodajesz do projektu cała bibliotekę font-awesome tylko dla jednej ikony, możesz ją pobrać w .svg.
Uwagi scss/css
[x] Nie trzymasz się https://sass-guidelin.es/, elementy powinny zostać podzielone na komponenty w osobnych plikach. Kod byłby łatwiejszy w utrzymaniu.
[x] Kolor np rgb(27, 170, 248) powinin być utrzymany w wartości hexalnej oraz powinien być utrzymany w zmiennej, to samo tyczy się innych kolorów w projekcie.
[x] Wartości spacingów tak samo jak używasz w html typu m- lub p- możesz używać w formie zmiennych w pliku .scss.
[x] Elemenet .heading::after powinien w sumie nie być tak użyty, ponieważ tracisz kontrole nad tym jaka część gradienta zostanie wyświetlona, bo część jest zasłonięta. Użyj to jako :after od elementu .heading--small i wykorzystaj własności flex.
Uwagi js
[x] Gdy używasz eventów on scroll (resize) najlepiej jest zastosować debounce.
[x] Nie musisz stosować na element #myTab a tab('show') na click, jest to gotowe w komponencie bootstrap.
KrystianRysnik
commented
4 years ago
Uwagi ogólne
Jest pewna niezgodność w projekcie:
Poppinsdo weryfikacji także wszędziefont-weightw projekcie..headermyśle, że lepiej by było tutaj zastosować ten element w tagu img zamiast w background img, na responsive można zastosować własnośćobject-fit.font-sizena elementach np.Umów się na bezpłatną wycenę.Zobacz jak działamyzdjęcie nie powinno być w kontenerze, a lewa strona powinna być utrzymana w kontenerze. Możesz to uzyskać robiąc dodatkowa utils klasę, która będzie robiła pętle po elementach w breakponcie i dodawała padding-left do kontenera (zobacz jak tworzony jest np komponent grid).Klimatyzatory - sprzedaż, montaż, serwis najlepsze markizostała użyta animacja w sumie lepsza by było zastosowanie jakiegoś infiniteloop, bo w pewnym momencie mamy sytuacje, że elementów nie ma (możesz użyć jakiejś gotowej biblioteki)SDM-Klient w liczbachplus w animacji mógłby zostać dodany w sumie na sam koniec jako callback.font-awesometylko dla jednej ikony, możesz ją pobrać w.svg.Uwagi scss/css
rgb(27, 170, 248)powinin być utrzymany w wartości hexalnej oraz powinien być utrzymany w zmiennej, to samo tyczy się innych kolorów w projekcie.m-lubp-możesz używać w formie zmiennych w pliku.scss..heading::afterpowinien w sumie nie być tak użyty, ponieważ tracisz kontrole nad tym jaka część gradienta zostanie wyświetlona, bo część jest zasłonięta. Użyj to jako:afterod elementu.heading--smalli wykorzystaj własnościflex.Uwagi js
on scroll(resize) najlepiej jest zastosować debounce.#myTab atab('show') na click, jest to gotowe w komponencie bootstrap.@KrystianRysnik