P
@vietlong90Submitted about 22 hours ago
Professora Bianca
@ProfessoraBiancaAll comments
- @ProfessoraBiancaPosted about 20 hours ago
Your solution in AWESOME!!!!
One little thing: When the site is in a big screen, there is a scroll bar in the right. Probably because de
min-heightof the<body>. Maybe set it tomax-heightinstead?Anyway, congrats!!!!
Marked as helpful1 - P@MuskaanVashSubmitted 4 days agoWhat are you most proud of, and what would you do differently next time?
I’m really proud of how I implemented Flexbox to create a responsive layout that adapts well to different screen sizes, especially with the product preview card component. I feel like the CSS structure is clean, and the use of Flexbox has made the layout quite flexible across devices. I also feel good about using semantic HTML5 elements and ensuring accessibility, as I aimed to make the page usable for all users, which is a key aspect I’ve been working on improving.
Next time, I would try to implement CSS Grid more effectively, as I feel I could’ve used it in some parts of the design for better control and alignment, especially when arranging multiple elements in a container. I’d also explore using SASS or CSS custom properties to improve scalability and organization of my styles. While the current implementation works well, introducing variables and a more modular approach would make future projects easier to manage.
What challenges did you encounter, and how did you overcome them?Challenge 1: Aligning elements properly across different screen sizes At first, I had issues with centering the container both horizontally and vertically on larger screens. The layout wasn’t as fluid as I wanted it to be. I overcame this by thoroughly exploring Flexbox properties like justify-content and align-items, and using height: 100% for the parent elements to ensure proper centering. Adjusting the height of certain elements also helped me achieve the desired result.
Challenge 2: Managing content overflow on smaller screens I ran into problems where the content was overflowing on smaller screens, particularly when I had to allow for scrolling. The container padding was causing the text and buttons to spill out. I resolved this by adjusting padding and reducing line heights for better space management.
What specific areas of your project would you like help with?Code Quality and Flexbox/Grid Skills: I’m looking for tips on how to write cleaner and more maintainable CSS. I want to improve my organization and structure, especially when working with Flexbox and Grids. Any advice on best practices or design patterns for responsive layouts would be really appreciated.
Flexbox/Grids Strengths: I’d love to hear some feedback on how I can get stronger with Flexbox and Grid. Sometimes I feel like I’m not utilizing their full potential, and I’m unsure of when to choose Flexbox over Grid or vice versa. If anyone has suggestions on how to improve my use of these CSS properties for complex layouts, that would be very helpful.
@ProfessoraBiancaPosted 3 days agoHello! You did really well in this challenge!
Some things I think you can improve based on my take of the challenge:
You could try using the
<picture>tag in HTML. This offers the option of changing theimgbased on thewidthof the display. (That's why Frontend Mentor included 2 images as resources)Personally I used only
flexfor this challenge. I think It was a good decision of yours.A question:
The live site shows everything disconfigurated. The text overflows the card in desktop view and the
<button>overflows in the mobile. Did you checked this problem?Marked as helpful0 - @ChrisTariahSubmitted 26 days agoWhat are you most proud of, and what would you do differently next time?
I am happy about the level of similarity I was able to achieve between my solution ang the original design, and the minor detail I was able to style.
What challenges did you encounter, and how did you overcome them?The major challenge I faced while designing my solution was trying to separate sections and design some details, for example: the numbers on the table. Designing the table was also a bit challenging, I spent a lot of time trying to get the bottom border on my rows, without any results, before I figured out that the table borders need to be collapsed first.
What specific areas of your project would you like help with?My design is not quite perfect, you will notice that the links do not align with the headers like in the original design any suggestions on how I can fix that, I tried using margin, padding and text align.
@ProfessoraBiancaPosted 6 days agoHey @ChrisTariah ! I liked a lot your solution! Congrats!
I also had a lot of problems with this challenge...
Some things I discovered:
To make the dots align with the headers, you can use the
ul::before { padding: 0; margin: 0; }or
ul::marker { padding: 0; margin: 0; }I don't remember witch of them helped me, but you can test it and find out :D
And juuuuuust a little thing:
When we test the responsive design, the image doesn't take the full width of the
<main>seccion! Neither theborder-radiusis set to 0. (It was hard to do it, but you should give it a try).Anyway, congrats and good coding!
0 - @Dorm-24Submitted 8 days agoWhat challenges did you encounter, and how did you overcome them?
I didn't find anything difficult. But I think my font sizes and margins aren't perfect.
@ProfessoraBiancaPosted 8 days agoHey, Dorm! Your solution is really good!
I noticed that you didn't include the cursor color change effect when we hover the links. Why? Did you choose to not include it? Or it was hard to implement?
Anyway, I really liked your design!
0 - @Nayiraa9Submitted 15 days ago@ProfessoraBiancaPosted 9 days ago
Hey! Your solution is really good! The one thing I would change is the placement for the cursor type.
It's on the .card class, so it makes the cursor imediatly change to pointer the moment we hover over the card.. It's the same for the color change of the title.
The idea of making the shadow move... WOW. Amazing!
I just think that the cursor change and the color change would be better if they were bound only to the title and not to the card itself.
0 - @Mohit708Submitted 10 days ago
