add task solution#564
Open
232567 wants to merge 3 commits into
Open
Conversation
raulriato
suggested changes
Jul 8, 2026
raulriato
left a comment
raulriato
left a comment
There was a problem hiding this comment.
You are missing some key points from the chacklist.md.
Also, you didn't push the index.html. Run git add index.html and then commit and push it.
Some points missing in your page:
- Add favicon
- Menu on mobile should contain navigation links. Menu example
- Logos should become bigger on hover and lead to the first section
- Icons in the “about us” block should be animated on hover
- The arrow icon in the footer should increase when hovering and when clicked smoothly scroll the page to the top
- Under the button “see more about us”, the strip should stretch to the full width when hovering and lead to the CrazyBaby website
- All interactive elements(links, buttons, pictures, icons) should have a hover effect and cursor pointer. User must intuitively understand that he can interact with them
- When you try to send the form there is no 405 error and the form is automatically cleared after submit and is scrolled to the top of the page or the page is reloaded
Remember to run npm run deploy after pushing the changes.
If you need any help, please join a Bate papo com o mentor meeting, or ask for help in the chat
Comment on lines
+4
to
+9
| &__content { | ||
| display: grid; | ||
| grid-template-columns: 1fr 1fr; | ||
| gap: 120px; | ||
| align-items: start; | ||
| } |
There was a problem hiding this comment.
This is causing your mobile layout to break.
Comment on lines
+11
to
+13
| &__logo-image { | ||
| width: 180px; | ||
| } |
There was a problem hiding this comment.
All your Logo images should have a hover effect to get bigger.
Comment on lines
+29
to
+36
| .benefit { | ||
| text-align: center; | ||
|
|
||
| &__icon { | ||
| width: 42px; | ||
| height: 42px; | ||
| margin: 0 auto 40px; | ||
| } |
There was a problem hiding this comment.
All __icon should be animated on hover. You can choose the animation style.
Comment on lines
+37
to
+41
| &__top { | ||
| position: absolute; | ||
| top: 120px; | ||
| right: 0; | ||
| } |
Comment on lines
+43
to
+54
| &::after { | ||
| content: ''; | ||
|
|
||
| position: absolute; | ||
| bottom: -8px; | ||
| left: 0; | ||
|
|
||
| width: 60px; | ||
| height: 1px; | ||
|
|
||
| background-color: $c-accent; | ||
| } |
There was a problem hiding this comment.
- Under the button “see more about us”, the strip should stretch to the full width when hovering and lead to the CrazyBaby website
added 2 commits
July 9, 2026 19:12
DanielCamposDevX
approved these changes
Jul 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DEMO LINK