@prajwal18
Submitted
Can someone review my work and give feedback, best practices, how you would do certain aspects differently. Any feedback would be highly appreciated.
Looking to hire developers?
@tomiwaorimoloye
@prajwal18
Submitted
Can someone review my work and give feedback, best practices, how you would do certain aspects differently. Any feedback would be highly appreciated.
@tomiwaorimoloye
Posted
Nice work bro, but don't forget to put in some placeholders. By the way, your calculation, for the total price seems to be a bit flawed, try to cross check that too.
@ThanhVuong0904
Submitted
-Hi everyone ! -I first coded in scripts2.js and then I improved the code in scripts.js. -Please tell me if there is something to improve in my code.
@tomiwaorimoloye
Posted
Nice work bro! You should also take a look at your accessibilities / HTML issues. Regardless, you did a great job.
@johnpaul18
Submitted
I'm new learning React, please guide me.
@tomiwaorimoloye
Posted
Awesome work bro👍
@GerriEzeocha
Submitted
Any feedback is welcome, especially regarding JS.
@tomiwaorimoloye
Posted
Hey, Your work looks great! However you should try fixing the responsive layout at (775px - 1175px) screen width And you also got some accessibility issues that need to be resolved. Regardless, nice work
@gerardguydavis
Submitted
This was mostly completed with SCSS and Vanilla JS.
I wasn't quite sure what the design prompt was intending with the blue subscription card. I thought it was supposed to be a selector, and I kind of like it better that way, but the preview looks a little off because of it 😅 (The footer background is set to fixed, which I think also caused some issues with the preview. I guess I could have made that background image its own separate div.)
I'm mostly happy with how this turned out, but I'm almost certain there was a better, more DRY way to handle the drop down menu on the signup page. I probably should have used React for this. Oh well...
@tomiwaorimoloye
Posted
Hey, great work! but don't forget that the Pro pricing component has a background-color: blue
by default
@manojks092
Submitted
Any feedback to mitigate or manipulate the code is highly appreciated.
@tomiwaorimoloye
Posted
Try to set a max width for the main container and center it so it doesn't stretch on wider displays.
@aguiimaraes
Submitted
@tomiwaorimoloye
Posted
Try generating a new screenshot so ur solution is easier to view directly from the solutions tab because currently it shows a 404 page on the screenshot.
Marked as helpful
@matthewjdavison
Submitted
@tomiwaorimoloye
Posted
Your design is near-perfect but don't forget to add the background image. Apart from that, great job!
Marked as helpful
@caioalvesp
Submitted
@tomiwaorimoloye
Posted
Nice work! But you missed some touches. Don't forget to add the box-shadow property to both the component and the primary button and put the background-image behind the whole component. With that, I'd say ur work will be better.
Marked as helpful
@arkharman12
Submitted
Any feedbacks are welcome :)
@tomiwaorimoloye
Posted
Your solution looks pretty accurate. You could however check for responsiveness. For example on a 2560 x 900 display, the site starts to look a bit off. You can tackle this by using chrome dev tools and adjusting the width of your container. A hack is to set a max width of the container and center it. And wrap it up by fixing your accessiblity issues.
@LucasDamasceno-S
Submitted
I had considerable difficulty getting the card centered in the beginning. But in the end I think I managed to do something relatively ok. The site has a certain degree of responsiveness and can adapt to cell phones with 300px width
@tomiwaorimoloye
Posted
You did good. Nice job! An easier way to center it is to use flexbox on the container of the item you want to center.
@mghadieh
Submitted
I have used lots of nested flex box displays. I am not sure if that is a good thing or not, but it got the job done.
@tomiwaorimoloye
Posted
I think it's okay to use a lot of nested flex containers in this particular project. From the description of this project, the purpose was to test your layout skills using either Grid or FlexBox.
@jmichaelseri
Submitted
I would be happy to receive some feedback on the positioning of the various elements and the use of css grid.
@tomiwaorimoloye
Posted
Your version looks good. I think a bit more padding could help.
Marked as helpful
@trostjan
Submitted
My second challenge, now really starting to understand how powerful mobile-first approach in combination with using rems and ems can be for quickly building a responsive website. Any feedback is highly appriciated. :)
@tomiwaorimoloye
Posted
Everything looks good but you should set the border radius so the site looks more identical with the design.
Marked as helpful
@Diego-Mike
Submitted
Look at my first project using react.js, flexbox, grid, and react router, the website is pretty simple but i used react to keep learning it and get used to it !
I added a new section at the beggining, you're going to get crazy with it, and know you will !
If you have any advices, i would love to read them and aply them for i'm still learning web design !
@tomiwaorimoloye
Posted
I love the creativity.
@Barsheyr
Submitted
Hi Feel free to give me a comment i would really appreciate it
Thanks
@tomiwaorimoloye
Posted
Your work looks great. But your cards look squashed so try to put in some whitespace to make it look better and put a border-radius: 5px; to you cards to align it with the original design
@silfrido-lab
Submitted
your feedback would help me a lot
@tomiwaorimoloye
Posted
Your work is very accurate. Congrats. Just try to fix your HTML and Accessibility issues. I think where you missed was the active states. The links don't respond to the hover action so try to fix that as well.
Hi! Stop-start newbie dev here. Looking to add some consistencies to my learning. This is my first challenge, please kindly help review my code and provide some feedbacks on how to write code better. Thanks!
@tomiwaorimoloye
Posted
Congrats on your first submission. Your work looks good. Try to fix the background in order for it to align with the main content.
@AxelZelenskiy
Submitted
I didn't understand where to put 'curve.svg'.
@tomiwaorimoloye
Posted
Your work is nice. Try to fix the HTML issues.
@gitdxb
Submitted
Any feedbacks are appreciated. I'm totally new to Web design and on the learning curve atm
@tomiwaorimoloye
Posted
Your work is very accurate so well done. Your page looks a little bit darker so try to get the colors right.
@Mahesh-yadav
Submitted
I want to have a single set of slide controls (next and prev buttons). I want to know how to fix position of these controls relative to image. I can do it at a fixed viewport width but as I change width the content adapt responsively and then button controls position changes.
@tomiwaorimoloye
Posted
you should probably make the picture relative and make the controls absolute to the picture then use the "bottom" attribute rather than top.
@CleytonRR
Submitted
I would like to know if the code organization is good.
@tomiwaorimoloye
Posted
It's good overall. But you should also pay attention to the HTML and accessibility issues. Add more padding to the first section with the picture, it looks a little bit too slim
@rfilenko
Submitted
Almost done with this landing page, a couple small things to finish. Had fun with css Grid, built with NextJS, working routing, form validation and modals. Will appreciate the feedback how to improve.😉
@tomiwaorimoloye
Posted
I've been waiting to see someone do their solution on this particular project. Thanks for doing it and it's actually so good. I look forward to seeing you post more premium projects
@tomiwaorimoloye
Posted
Great work! You were very accurate on this one.