Skip to content

Comments

Challenge#1#12

Open
karolguzik wants to merge 6 commits intoD0man:masterfrom
karolguzik:homepage
Open

Challenge#1#12
karolguzik wants to merge 6 commits intoD0man:masterfrom
karolguzik:homepage

Conversation

@karolguzik
Copy link

No description provided.

@IamMK
Copy link

IamMK commented Jan 22, 2019

Więc zacznę ja. Co do samego HTML nie mogę się dużo przyczepić.

<img class="logo__img-corner" src="assets/coming-soon-ribbon.png" alt="">

jedynie ta nazwa mi się nie podoba. Logo w rogu? Trochę myląca nazwa.

pattern="/^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*{2,8}$/"

Ten pattern chyba ode mnie :D poznaję po {2,8} ;)

<h1 class="logo__title">code academy</h1>

Jeśli się doczepić na siłę, to Code Academy ;) Należy w pliku zachować trochę poprawności w pisowni.

@media(min-width: 768px) and (orientation: portrait)

Podoba mi się ta linijka ;)

@karolguzik
Copy link
Author

  1. faktycznie, nazwa klasy rownież niepoprawna
  2. tak, pattern twoj
  3. bede pamietal, juz ktos mi wspomnial o tym na grupie
  4. ale tego to ja nie rozumiem :)
    btw dzieki MK za code review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants