@0605AbMu
Latest comments
- @smalancea
I think your solution is a little bit different than the original one. Have you tried starting with the basic main containers and working your way to the smaller components? The font size is also different and the font weight too. I think overall looks good but you just need organize it a little better.
Marked as helpful - @AmitKumar-001@smalancea
Your design is slightly different from the original. Colors, font-size, font-weight, padding and etc need to be adjusted. I took a look through your files on github and I could say that you need to rearrange the order of those files and folders. Html is good and CSS too, however I would say that you will need to get better and naming classes in a more understandable way and writing css properties in order. Overall looks good.
- @DanielAlcantara1997@smalancea
Design is pretty good and mobile version too. I took a look through your files and I like that you commented your CSS and have properties in order. However, you could simplify your JS by not repeating code. Also HTML needs some commenting too.
Marked as helpful - @Blazej-oWo@smalancea
The design looks somewhat close to the original one but there's some little differences like the font-weight and the border for the button.
I looked through your html and css and could say something: You need to clean index.html a little bit by having better classes and commenting code. Your CSS could also be a little bit cleaner by putting classes in order and properties too. Overall looks good.
Marked as helpful - @acoreyu@smalancea
Your code looks very clean. I took a look through your files and I would say good job for commenting. The design is very clean and also the mobile version looks good.
Marked as helpful - @smalancea@smalancea
Let me know if my code is clean and functional
