@sumaiyakawsar
Latest solutions
Product-List-With-Cart Solution
#react#viteSubmitted 3 days agoI can't think of anything for now, however I am open to feedback on my work.
Results-Summary-Component Solution
#react#viteSubmitted 9 days agoNone that I can think of for now, however I am open to any sort of feedback on my work.
Bookmark-Landing-Page Solution
Submitted 17 days agoI am proud of this project so I can't think of anything for now, however I am open to any sort of feedback on my work.
Room-Homepage Solution
Submitted 25 days agoNone that I can think of for now, but I am open to feedback on my work.
Loopstudios-Landing-Page Solution
Submitted about 1 month agoEven though it looks good, the length of my bottom borders in the navigation links hover-state isn't as short as the one in the design specs, perhaps someone can let me know how to achieve the desired result.
NFT-Preview-Card-Component Solution
Submitted about 1 month agoNone that I can think of for now, however I am always open to some feedback on my work.
Latest comments
- @Tonye-Onuoha
@sumaiyakawsar Good job. I love your work, plus the animations were a nice touch.
- @Legaz-le@Tonye-Onuoha
@Legaz-le Nice work here. I was wondering if you could tell me how to include the "data.json" file that was provided to us in the production build using vite. I am having trouble loading the dessert products dynamically from the "data.json" file and I am sure that it's because it isn't being included as part of the build. Any suggestions might be helpful.
- @yrjeb@Tonye-Onuoha
@yrjeb Good work. Only needs some minor adjustments to make it better.
-
You forgot to add border-radius to each of the skills listed.
-
The top-left and top-right corners of the purple score component/container don't need any border-radius specified for the mobile-layout. You need to remove them in the mobile-layout but add them for the desktop-layout.
-
You didn't add the correct color for the "Summary" heading. Set it to "hsl(224, 30%, 27%)" as specified in the style-guide README.md file.
-
You didn't add the linear gradient for the "Continue" button hover effect. You can set it to "background-image: linear-gradient(hsl(252, 100%, 67%), hsl(241, 81%, 54%));"
-
Finally for the most obvious one, you need to set the background color of the body to the specified color like this : "background-color: hsl(221, 100%, 96%)"
Hopefully this makes things much better.
-
- @Legaz-le@Tonye-Onuoha
@Legaz-le Please how did you get the design layouts for this project? My starter template does not come with the mobile and desktop design layout images.
- @TetianaAleksWhat specific areas of your project would you like help with?
I’d really appreciate constructive feedback.
@Tonye-Onuoha@TetianaAleks Lovely work Tetiana. You wrote the JS email validation but it's not working. This is probably due to the fact that HTML form validation is still enabled. You can disable it by adding "novalidate" as an attribute to your form tag i.e <form novalidate></form>
This would enable your JS form validation to take effect.
Marked as helpful - @PyaePhyo1999@Tonye-Onuoha
@PyaePhyo1999 Really nice work on desktop, I love your animations too and hopefully I'll learn that someday. However on mobile, the blue background behind the main hero-image begins to stray away from the image and even block the elements below it. Feel free to look into that when you have the time. I recommend you put the blue background and the image in the same container and the position it absolutely to the parent container (that way it never leaves the boundaries of that container and blocks other elements), the you can add a z-index:-1 to position it behind the image. Also you forgot to add the hover states as seen in the design specs.
