@christiano84
Latest solutions
Latest comments
- @Bashamega
Hello:) It is very good, and your code is clear and readably. The only thing I noticed was that you included the design folder and the README-template.md and the style guide and website-text but you should exclude this in your GitHub repo. Also try setting the active tab to be black but thee unactive tabs to grey
- @GermanGuerreroR@Bashamega
Hello:) It looks awesome. I have noticed 2 issues. First it is not very responsive on large screens, like it looks good on my laptop but horrible on my monitor. Also, you should add a hover effect for the button. and the confirmation the email should be the email that the user has entered, and you can achieve this with some js. However, the overall is great. I how you had a great time completing this challenge.
- @hanzala-bhutto@Bashamega
It is great, and awesome. I suggest that you try to use semantic html instead of divs like nav, main, and footer tags. Also I think the crew page should be a slideshow and the crew members images get a little bigger
- @LucianEmanuel@Bashamega
It is awesome, but I suggest make the images more bright by default
- @Twixii99@Bashamega
Hello:) It is very good and responsive. I suggest removing the default front end mentor style and also try using semantic HTML, for example use the footer tag instead of the div tag.
- @MrDevM@Bashamega
Hello Mulugeta I have checked it in my browser the responsive view, it is fine, but the image is stretchy. I suggest not specifying the height of the image or setting it to auto and this should resolve the issue
Marked as helpful
