Nikt tego jeszcze nie zrobił to ja w takim razie się tego podejmę :P
Tzn postaram się ocenić kod.
1. Jesteś już na takim poziomie, że wypadałoby pilnować, czy dajesz "..." czy '..'
@import "./node_modules/normalize-scss/sass/normalize/import-now";
@import 'vendor/all';
2. Nie lepiej tutaj dać skrócony zapis dla **background**?
background-color: black;
background-image: url('../assets/phone-screen-min.png');
background-size: cover;
background-repeat: no-repeat;
background-position: center center;
background-attachment: fixed;
3. Po co aż coś takiego:
&_heading,
&_addressInfo,
&_contactForm,
&_socials,
&_paragraph {
position: relative;
z-index: 5;
width: 100%;
}
Jeśli chodziło o to przyciemnienie stopki to trzeba było dać z-index: -1 na pseudoelemencie (tam gdzie masz to ciemniejsze tło) + z-index: 1 dla całej stopki i zadziała.
4. Ojj nie za ładnie:
&_addressInfo {
}
&_contactForm {
}
A takich pustych miejsc masz naprawdę sporo, po co tak?
5. Takie coś:
&_serviceItem {
width: 100%;
margin-top: 3em;
}
@include screen(mobile) {
&_serviceItem {
width: 80%;
}
}
@include screen(mobileBig) {
&_serviceItem {
width: 70%;
}
}
@include screen(tablet) {
&_serviceItem {
width: 60%;
}
}
Możesz spokojnie zamienić na:
&_serviceItem {
width: 100%;
margin-top: 3em;
}
@include screen(mobile) {
width: 80%;
}
@include screen(mobileBig) {
width: 70%;
}
@include screen(tablet) {
width: 60%;
}
6. Jestem w 100% pewna, że jest fajna skrócona wersja dla tego zapisu, ale po prostu zapomniałam jaka :<
grid-template-rows: 20vw 20vw 20vw 20vw 20vw 20vw 20vw 20vw 20vw 20vw;
7. Zamiast do (prawie) każdego elementu dodawać box-sizing możesz na początku kodu zaznaczyć:
* {
box-sizing: border-box;
}
Przynajmniej nie będziesz musiał tego powtarzać :)
8. Pozamieniałabym nazwy plików "all.scss" na jakieś inne bo tak średnio to wygląda. Może bardziej "core.scss" ? A jak nie to zawsze też możesz od razu importować do głównego pliku.
9. Może warto byłoby się zainteresować czymś takim jak ITCSS? Fajnie można ogarnąć strukturę plików.
10. Dość ciężko ogarnąć Twoje nazewnictwo klas. Może jednak bardziej BEM?
Ale ogólnie powiem Ci, że fajnie się przeglądało kod bo jednak trzeba było bardziej się na nim skupić no i na pewno nie są to błędy typowe dla strony wrzucanych tu do oceny. Działaj dalej!