My attempt#17
Conversation
| @@ -0,0 +1,226 @@ | |||
| @import url('https://fonts.googleapis.com/css?family=Lato:300,400,700&subset=latin-ext'); | |||
There was a problem hiding this comment.
Lepiej jest załadować fonty w w HTML.
There was a problem hiding this comment.
Ze względu na fakt, że czcionki ile by nie ważyły to dodatkowe dane, które muszą być zaciągnięte zanim cały plik css zostanie wczytany. Lepiej jest zrobić to w html ;)
| a{ | ||
| text-decoration: none; | ||
| } | ||
| button{ |
There was a problem hiding this comment.
Staraj się unikać stylowania po tagach.
| border: 2px solid red; | ||
| } | ||
|
|
||
| .contact__input:invalid ~ .contact__input-warning--error{ |
There was a problem hiding this comment.
Z tą częścią musisz pokombinować. Nie zadziałało tak jak miało. Od wejścia na stronę wyświetla napis invalid e-mail. Ale idziesz w dobrym kierunku. To może Ci się przydać: https://css-tricks.com/float-labels-css/
| text-decoration: none; | ||
| } | ||
| button{ | ||
| font-family: roboto, sans-serif; |
There was a problem hiding this comment.
Ustawiasz fonta, którego nie załadowałeś do projektu. Chciałeś chyba Lato wrzucić :)
There was a problem hiding this comment.
No tak, to copy/paste z resetu mojego jak zaczynam każdy projekt ;) Czcionkę przy okazji zmienia na bardziej przyjazną. Będę musiał pamiętać by to zmianiać. Swoją drogą, inherit zadziała dla buttona przy dziedziczeniu z body?
There was a problem hiding this comment.
Sam sobie odpowiem, inherit lepiej się w tym przypadku sprawdzi:)
IamMK
left a comment
IamMK
left a comment
There was a problem hiding this comment.
Podsumowując. Nazwy klas są dość ok. Idzie się połapać co jest czym bez patrzenia w plik html. Co do jakiś błędów - może mam tylko takie wrażenie, ale Twój kod wydaje mi się dłuższy niż powinien. Można go śmiało skrócić ;) Mimo to generalnie jest dobrze.
|
|
||
|
|
||
| Jak ktoś ma ochotę zrobić lepszy readme, to niech to zrobi :D | ||
| demo - http://vizman.ayz.pl/01-Landing-Page/ |
There was a problem hiding this comment.
Cały README usunąłeś... Unikaj tego, mało ko to lubi
There was a problem hiding this comment.
Poprawione, jednak dalej chyba jest konflikt? Czyli nic do readme nie mogę dodać, nawet linka do wersji podglądowej?
| font-size: 3rem; | ||
| line-height: 5rem; | ||
| } | ||
|
|
There was a problem hiding this comment.
nie wiem po co tutaj ta zabawa z minusowymi marginami. Niekoniecznie jest to dobre ;)
There was a problem hiding this comment.
Też już nie pamiętam po co:)
| @@ -0,0 +1,226 @@ | |||
| @import url('https://fonts.googleapis.com/css?family=Lato:300,400,700&subset=latin-ext'); | |||
There was a problem hiding this comment.
Ze względu na fakt, że czcionki ile by nie ważyły to dodatkowe dane, które muszą być zaciągnięte zanim cały plik css zostanie wczytany. Lepiej jest zrobić to w html ;)
| a{ | ||
| text-decoration: none; | ||
| } | ||
| button{ |
| main{ | ||
| flex-grow: 10; | ||
| } | ||
|
|
| <h1 class="header__title" >Code Academy</h1> | ||
| </header> | ||
| <main> | ||
| <form id="contact__submit--send" action="" class="contact"> |
There was a problem hiding this comment.
coś tu zmieniasz, że dodajesz modyfikator?
samo contact__submit wystarcza
There was a problem hiding this comment.
pod jsa, dwa przyciski są tak samo ostylowane a musiałem je jakoś wyselekcjonować w jsie. Jeden --send odpala dodanie maila do tablicy, a drugi --close zamyka okno z listą adresów.
| </div> | ||
| <input id="contact__submit--close" type="submit" class="contact__submit" value="ok"> | ||
| </div> | ||
| </aside> |
| e.preventDefault(); | ||
| tab.push(input.value); | ||
| tab.reverse(); | ||
| console.log(tab); |
There was a problem hiding this comment.
Jeśli udostępniasz, usuwaj debugi i console.log'i. Mnie to osobiście drażni. Chyba że jest ku temu jakiś powód
3 participants
html/css/js
demo - http://vizman.ayz.pl/01-Landing-Page/