Steven• 150
@Steven-Ang
Submitted
a-yemisi
@a-yemisi
All comments
- a-yemisi• 130
@a-yemisi
Posted
Looks alright
Well done.
0 - Woldu Tsegazgi• 910
@weldu0
Submitted
What specific areas of your project would you like help with?
I’d love to hear any suggestions you might have for things I could improve on!
a-yemisi• 130@a-yemisi
Posted
You called your media screen for screens too large. The four cards showed up like in the android format when I tried accessing your page on my laptop. It is a 14 inches laptop. Asides from that, I have no compliants
Marked as helpful
0 - Robel T. Hawelti• 100
@Rapbit27
Submitted
What challenges did you encounter, and how did you overcome them?
Changing the pictures when the view changed was quite difficult. The second thing was fitting the picture to full height of the card in desktop view.
What specific areas of your project would you like help with?
I would like help on the above mentioned problem. Changing the pictures when in different views and also with sizing them accordingly. Thanks in advance!!
a-yemisi• 130@a-yemisi
Posted
For the images, there were two images sent for the challenge, you could use media query. For the smaller screen size, ask it to use the mobile image, for larger, use desktop. You can use CSS to make to add an image to an HTML page, look up how that is done.
Asides from that well done.
1 @abdelkarim-el-manssouri
Submitted
a-yemisi• 130@a-yemisi
Posted
The HTML is fine, the CSS is there. I can see that you used the media query, but the website behaves the same way and has the same appearance for both smaller and larger screens. This should not be so from the design given.
Asides from that, it was fine.
0- Renzo2O99• 40
@Renzo2O99
Submitted
a-yemisi• 130@a-yemisi
Posted
Does the solution include semantic HTML? I do not necessarily see semantic HTML, but the solution is not complicated, so it's fine if there is no semantic HTML. . . Is it accessible, and what improvements could be made? The solution is accessible, the solution is close enough to the design but could be better. . . Does the layout look good on a range of screen sizes? The layout looks good on a range of screen sizes. . . Is the code well-structured, readable, and reusable? I am not sure about the structure of your code, your html is readable, but your CSS is a lot. It will be easier to study if it was simpler, I did not understand the need for 3 CSS files. . . Does the solution differ considerably from the design? The solution differed from the design, but generally it was fine.
In conclusion, your design is fine, my major concern was the structure of your CSS code, and that it differs a bit from the original design. Good job!
0 - Obare• 210
@simplyObare
Submitted
a-yemisi• 130@a-yemisi
Posted
Does the solution include semantic HTML? Fine enough Is it accessible, and what improvements could be made? It is Does the layout look good on a range of screen sizes? It should Is the code well-structured, readable, and reusable? Yes Does the solution differ considerably from the design? Yes, the card is not centralized on the page
Altogether, it is well done, just the issue of the position of the card.
0 @Akinloluwa-Adedijo
Submitted
What are you most proud of, and what would you do differently next time?
Proud of figuting out how to center a div and the text as well.
What challenges did you encounter, and how did you overcome them?
Centering the div
a-yemisi• 130@a-yemisi
Posted
Unfortunately, cannot access your code The layout is okay There seems to be some issues with rendering the image Generally, it's fine
0
