Yusuf Jamal
@Jamal-DigitalAll comments
- @Abas-codeSubmitted 15 days ago
- @clyoSubmitted 18 days ago@Jamal-DigitalPosted 18 days ago
hi , you really tried about the design but the design is not functional , the javaScript is not working. you should work on that
0 - @christianb3llSubmitted 22 days agoWhat are you most proud of, and what would you do differently next time?
Good CSS grid practice. I went back and forth on the semantics for this one. I ultimately set the user name as the header for each card, rather than the large pull quote but I can see how it could go either way.
What challenges did you encounter, and how did you overcome them?Positioning the items on the grid that spanned multiple rows/columns took some trial and error.
What specific areas of your project would you like help with?Interested in seeing how other people did the semantic markup
- @LouaiKhodary13Submitted 3 months ago@Jamal-DigitalPosted 22 days ago
Hi Your really tried with your design , but for convenience and fast work flow, instead of typing or pasting of same code for each cards you could just give the cards double classes with one class the same class with all the four cards, so you font have to code same things for the other cards same again. eg
<div class="card card1"> this first card </div> <div class="card card2"> this second <div class="card card3"> this third card </div>card </div> <div class="card card4"> this fourth card </div>Marked as helpful0 - @stephany247Submitted about 1 month agoWhat challenges did you encounter, and how did you overcome them?
I didn't encounter any challenges.
What specific areas of your project would you like help with?I'd would really appreciate any feedback.
@Jamal-DigitalPosted about 1 month agoHi , you design looks good but i'll like to give you a suggestion. The easy way to put a line over a text is by using the text-decoration att ribute instead of using the pseudo code ::before or ::after.
p { text-decoration: line-through; } also about changing image during responsive. i'll suggest using <picture> tag
<picture> <source media="(min-width: 36.875rem)" srcset="images/image-product-desktop.jpg"><img src="images/image-product-mobile.jpg" alt="image product"> </picture>
Marked as helpful0 - @Augurk66Submitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
I am proud of the fact that I pay attention to small details.
What challenges did you encounter, and how did you overcome them?This project has a lot of details for a beginner. I divided the project into smaller junks to make the the project doable.
What specific areas of your project would you like help with?My CSS is always a bit chaotic. I need to learn to write consistent code.
@Jamal-DigitalPosted about 1 month agoHi, i really love the way you did yours, mine is a little bit hard, because i was not able to design my <ol> according to the design. But i learnt that from your code. Keep it up , you're great.
0 - @kentimusSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
I think I got pretty close to the preview image.
What challenges did you encounter, and how did you overcome them?This was similar to the two previous challenges, so not too difficult.
What specific areas of your project would you like help with?Any feedback is helpful.
@Jamal-DigitalPosted about 1 month agoyou really did a good job, mine is not even as perfect as this when i did mine. Nice! keep it up.
0 - @HM-JubaerSubmitted about 1 month agoWhat specific areas of your project would you like help with?
I’m open to any advice. Thanks!