@ShreyashParab
Latest solutions
Summary component (React)
Submitted about 1 year agoSuppose I have a large number of local images. How can I use them dynamically without importing all of them?
Latest comments
- @RadaidehDaniel
Good work, building this project without a framework is a fantastic skill.
- @ManojSinghDashauni@RadaidehDaniel
Good job, Manoj.
I have two comments regarding the logic. You missed the total mortgage amount over the term. Consider splitting the JSX in the Container into components.
I hope I helped.
- @QayyaxWhat specific areas of your project would you like help with?
Avoiding prop drilling Planning of components before starting
@RadaidehDanielGood job. The site is responsive. To me, the logic is sound. You still need to add the confirmation order feature. Look for <dialog></dialog>, it may help. MDN dialog
Marked as helpful - @SabaMarghania1@RadaidehDaniel
Good work.
One thing to point out is that the code (interactive-price-component) is not for the site (summary component), so I will review the live site only.
The design is perfect, and it is responsive but if you change the view for things like 820*1180 (tablet view), you will notice that there is a space on top. I think you use (display: flex) to center the app. You can add media query to change the display to block or inline to make the app stick to the top border of the screen.
Marked as helpful - P@Glenda9031@RadaidehDaniel
Great work.
The App is working as intended, but there is a missing timer. The question should have a timer.
Marked as helpful - @EliasFlorian@RadaidehDaniel
Good job.
