Looking to hire developers?
@sansarj17
@MightyKale
Submitted
I am not sure if my styling in CSS is correct and/or consistent, especially how I positioned elements with absolute, relative and fixed. Also in the media query area, I used display: none to change the pattern divider between mobile and desktop view, and I am not sure if that's the correct way to do it. Any feedback for suggestions welcome, thank you guys in advance!
@sansarj17
Posted
Hello Kale, Congrats on completing the challenge.
My suggestion would be to light up the dice when the person hovers on it so the user knows that he can interact with it. As for the divider, I think using anyone would have been fine and you can scale up/down the divider image according to the viewport. However, I do like how you tackled the issue.
@Beats-Ayush
Submitted
Would like some valuable feedback on the responsive-ness of the site and where I can improve.
Spent a lot of time on the javascript part and added some states for the inputs. Any inputs on how I can improve the javascript will be highly appreciated.
Also facing a lot of trouble in styling in React. Any resources for learning how to manage that?
@sansarj17
Posted
Hey Ayush, Congrats on completing the challenge. It looks clean and good.
As for styling, i recommend styling it prior to making the app. Example, make the webpage using plain html,css and then convert it into react app.
Marked as helpful
@sansarj17
Submitted
In this one, I used CSS grid. With grid, it was easy to lay out the content. I mainly had one problem, in bigger viewports, the page was extending in x and y direction unnecessarily. Idk why, any suggestions
@sansarj17
Posted
One more issue was the background.
@AleksandarV91
Submitted
Any feedback is more than welcome!
@sansarj17
Posted
Congrats man. Looks clean and professional
Marked as helpful
