@MariekafSubmitted 5 months ago
Latest solutions
Testimonial-grid-section using HTML and CSS
Submitted 5 months agoFull code on review.Let me know what can be cleaned up, better organized, and just all around any detailed suggestions you can possibly leave me with to help make me a better programmer. Thank you for your feedback.
Four-card-section using HTML and CSS
Submitted 5 months agoFull code on review.Let me know what can be cleaned up, better organized, and just all around any detailed suggestions you can possibly leave me with to help make me a better programmer. Thank you for your feedback.
recipe-page-main using HTML and CSS
Submitted 5 months agoI would like help on table bordering and centering the text and aligning the items.
Socila-links-profile-main using HTML and CSS
Submitted 5 months agoI would like help on to make my code more efficent and less code. I also want know through accessibility perspective of my code.
Blog-preview-card-main
Submitted 5 months agoI would like help on how to design the button in this project with less code than i used maybe if there is different way. I would like to use more flex or grid in styling the pages. Can you suggest where can i use the flex or grid in any way in this project. Thank you for your feedback.
Latest comments
- @Manisha3196Posted 5 months ago
Hi.For the name i think headings tag will suit better and testimonial text i think p tag suits better than div. I think you used the style for name and graduate in HTML but you can style it in directly in css style so that you will not be repeating the code again and again in HTML. I think you used divs more than you want.you can try checking out other HTML tags like h1,h2,h3 or section or p or article tags. It is not responsive page as you used fixed div tags.rather that you can wrap each one of the testiomonial in div and use flex for mobile and then use grid to arrange the testimonial cards.It will be more responsive and efficient.
1 - @marliedevSubmitted 5 months agoWhat specific areas of your project would you like help with?
Is there a more elegant and fluid responsive solution?
- @bmeinert8Submitted 5 months agoWhat are you most proud of, and what would you do differently next time?
The thing that I'm most proud of in this project is the customization that I added to the project. I believe that customizing each and every one of these projects with your own ideas is a true challenge in itself, that really helps push the project to the next level. These challenges take time and skill as it is, but to be able to really make it your own, implement your ideas, and see the finished product come to life, is where the fun is at for me!
What challenges did you encounter, and how did you overcome them?The biggest challenge I had in the project was getting the JS feature to work. I'm fairly new to JS at this stage still so doing a lot of googling, youtubing, and trial and error to get the cart function working was a big obstacle at first for me, but also very fun learning experience and overly satisfying to see completed and working.
What specific areas of your project would you like help with?Full on code review. Let me know what can be cleaned up, better organized, and just all around any detailed suggestions you can possibly leave me with to help make me a better programmer. As always, thanks for looking and taking the time, it's truly appreciated :)
@Manisha3196Posted 5 months agoI think your code looks clean and readable.kudos to you for customizing your own ideas. Suggestions that i can give as iam still new to this are
- picture tag for product-image in HTML. <picture> <source srcset="./images/image-product-desktop.jpg" media="(min-width: 800px)" /> <img src="./images/image-product-mobile.jpg" alt="perfume-image" /> </picture>
So that it selects image according to width.
-maybe try to use grid maybe.I think it is more efficient.
I would also like to know how did you get to know the that precise image/margin width and height in rem like .212rem.Is that all trial and error or else is there any other way you did it.
Marked as helpful1 - @izma13Submitted 5 months ago@Manisha3196Posted 5 months ago
Hi, Your code looks clean and readable.for the line border-bottom maybe u=you can use the color stone150 given in style.md. Also for class instructions maybe you can play with margin or padding so that the li items will start at same place for every line.
0 - @BoffdubSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
I was able to make it responsive. I've always put it off in the past but it wasn't as scary as I thought it would be.
These challenges have been easier too.
What challenges did you encounter, and how did you overcome them?I was trying to figure out if it was a or an tag. I tried it both ways and the tag ended up making more sense given the screenshot I was given.
What specific areas of your project would you like help with?If there are ways to clean up the code a bit.
- @dhimanharshSubmitted 6 months ago@Manisha3196Posted 5 months ago
Hi @dhimanharsh, I think your solution looks good but for the height maybe you can use min-height: 100vh in body style.It really helped me.
0
