Add task solution #1017
Add task solution #1017
Conversation
|
Demo link is unavailable.
|
src/index.html
Outdated
| </div> | ||
| <div class="footer__contact-information"> | ||
| <h3>Visit us</h3> | ||
| <p>2905 West Drive, Buffalo Grove</p> |
There was a problem hiding this comment.
it should be a link for a google maps url
src/styles/blocks/our-expertise.scss
Outdated
| color: #253757; | ||
| } | ||
|
|
||
| &--information{ |
There was a problem hiding this comment.
there is no hover effect on these elements
src/styles/blocks/testimonials.scss
Outdated
| @@ -0,0 +1,149 @@ | |||
| .testimonials{ | |||
There was a problem hiding this comment.
there is no hover effect on review elemets
src/index.html
Outdated
| </h3> | ||
| </article> | ||
|
|
||
| <article class=" |
There was a problem hiding this comment.
indentation on the entire html document is disrespecting the linting styling from the linting packages provided by Mate.
src/index.html
Outdated
| <input | ||
| type="text" | ||
| class="footer__message-input" | ||
| placeholder="Your message" |
There was a problem hiding this comment.
this input should be a textarea.
what if the user wants to send a 30 lines text? It would be very difficult to see the entire text on this small and unresizable field.
src/index.html
Outdated
| <p>004</p> | ||
| </div> | ||
| <div class="service__item--jobs"> | ||
| Digital Communications<br> |
There was a problem hiding this comment.
its perfectly fine to wrap each phrase on html tags to avoid the <br> tag.
src/index.html
Outdated
| <p>003</p> | ||
| </div> | ||
| <div class="service__item--jobs"> | ||
| Marketing Strategy<br> |
There was a problem hiding this comment.
its perfectly fine to wrap each phrase on html tags to avoid the
tag.
src/index.html
Outdated
| <p>002</p> | ||
| </div> | ||
| <div class="service__item--jobs"> | ||
| Сontent Production<br> |
There was a problem hiding this comment.
its perfectly fine to wrap each phrase on html tags to avoid the
tag.
src/index.html
Outdated
| <p>001</p> | ||
| </div> | ||
| <div class="service__item--jobs"> | ||
| Brand Development<br> |
There was a problem hiding this comment.
its perfectly fine to wrap each phrase on html tags to avoid the
tag.
| @@ -0,0 +1,14 @@ | |||
| @import 'utils'; | |||
There was a problem hiding this comment.
commenting here to not pollute the other scss files.
i don't see any hover and cursor styles across the styling.
There was a problem hiding this comment.
@MannyShussan the Header Menu is not opening on mobile screen
| border-radius: 8px; | ||
| background-color: #2060F6; | ||
| color: #fff; | ||
| border: 0; |
There was a problem hiding this comment.
add:
cursor: pointer
outline: none
property here
DEMO LINK.