Finished all tasks#161
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Michale,
Całość prezentuje się bardzo dobrze! 👍
Miałem drobną uwagę dot. menu - szczegóły znajdziesz w komentarzu :)
| border-bottom-color: transparent; | ||
|
|
||
| shape-outside: polygon(0 0, 0% 100%, 100% 0); | ||
| margin-right: 10px; |
| "b-item" | ||
| "c-item"; | ||
| } | ||
|
|
| grid-template-areas: | ||
| "a-item a-item" | ||
| "c-item b-item"; | ||
| } No newline at end of file |
| referrerpolicy="strict-origin-when-cross-origin" | ||
| allowfullscreen | ||
| ></iframe> | ||
| </side> |
| </li> | ||
| <li class="dropdown__item"><a href="#">Item 2</a> | ||
| <i class="fa-solid fa-chevron-right arrow-right"></i> | ||
| <ul class="nav__dropdown-second dropdown"> |
There was a problem hiding this comment.
W wersji mobilnej zagnieżdżone elementy nie wyglądają za dobrze - wychodzą poza ekran z prawej strony, warto to poprawić :)
| <div class="header__container content"> | ||
| <h2 class="header__logo content__item"></h2> | ||
| <nav class="header__navbar content__item"> | ||
| <i class="burger fa-solid fa-bars"></i> |
There was a problem hiding this comment.
Warto ukryć "hamburgera" w wersji mobilnej :)
There was a problem hiding this comment.
Chodzi pewnie o wersje desktopową, w sumie to nie wiem co się stało, bo burger miał 'display: none'
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Michale,
Zmiany są ok 👍
Zastanawiam się jeszcze czy przypadkiem te elementy nie są za małe. Na telefonie może być mało wygodnie je klikać dlatego często pozycje menu wyświetla się jedną pod drugą (block). Już bym nie zmieniał, ale na pszyszłość pamiętał ;)
| background: var(--color-beta); | ||
| padding: 10px; | ||
| min-width: 120px; | ||
| min-width: 100px; |
2 participants
No description provided.