Develop#2749
Conversation
There was a problem hiding this comment.
Great start :) remember about CHECKLIST
-
there're several BEM naming issues in your project. I've marked one example of each type of mistake but please fix all similar cases
-
consider adding max-width so that on screens wider than 1260px only side margins continue to grow because right now some sections become too stretched (e.g in Contact Us section the two content columns are pushed to opposite sides of the screen leaving large empty space between them
-
adjust image positioning so that it is cropped correctly
- change cursor to pointer for all buttons and links (make sure all addresses are clickable links)
| <main class="main"> | ||
| <section class="hours"> | ||
| <div class="hours--left"> | ||
| <div class="main__font main__font--highlight main__font--highlight-small"> |
There was a problem hiding this comment.
Typical BEM Mistakes: Using an Element Inside Another Block
| Museum hours | ||
| </h2> | ||
| </div> | ||
| <div class="hours--right"> |
There was a problem hiding this comment.
Typical BEM Mistakes: Using a Modifier Without the Belonging Class
2 participants
Index without navigation
DEMO LINK