
Adán Maidana
@AdanMaidanaAll comments
- @able4servicesSubmitted about 1 month ago@AdanMaidanaPosted 15 days ago
To fix the issue of stacked testimonials when the resolution is higher than 375px, consider switching the layout from mobile to desktop when the resolution is higher than or equal to, for example, 992px.
0 - P@socratesioaSubmitted 17 days agoWhat are you most proud of, and what would you do differently next time?
I knew how to code it using flexbox but I wanted to try CSS Grid which turned out easier and cleaner.
What challenges did you encounter, and how did you overcome them?I had to brush up on my CSS Grid knowledge mainly grid areas. Great practice.
What specific areas of your project would you like help with?Is the responsiveness fine? I had a bit of a hard time deciding the break points.
@AdanMaidanaPosted 16 days agoI would also recommend using the <img/> tag for SVG, the code is much more readable that way because otherwise you have the full SVG and since it is so long it can be confusing to maintain the code, especially in larger projects.
Marked as helpful1 - P@socratesioaSubmitted 17 days agoWhat are you most proud of, and what would you do differently next time?
I knew how to code it using flexbox but I wanted to try CSS Grid which turned out easier and cleaner.
What challenges did you encounter, and how did you overcome them?I had to brush up on my CSS Grid knowledge mainly grid areas. Great practice.
What specific areas of your project would you like help with?Is the responsiveness fine? I had a bit of a hard time deciding the break points.
@AdanMaidanaPosted 16 days agoI would recommend you to put the breakpoints with a little more space since the content looks very tight, for example 1024px and also consider using the max-inline-size property for the paragraph below the h1/h2, keep coding and improving!
Marked as helpful1 - P@angie-createSubmitted 16 days agoWhat specific areas of your project would you like help with?
I’d love to know how to make the cards the same size across different devices, I used max-width and max-height but one of the cards has a different height.
Also, I usually design in px first and then convert the measurements to rem— is there a better practice for this?
I’d also love to know how you decide when to use Flexbox or Grid. In this case, I used Flexbox, but I could have used Grid too.
Any suggestions on how to improve my code are welcome!
@AdanMaidanaPosted 16 days agoGood work, but you can try building your desktop layout with a grid without flexbox, because that gives you better control of the content, doing it with 3 columns and 4 rows is very easy to do the challenge.
0 - @nik-p12Submitted 17 days ago@AdanMaidanaPosted 17 days ago
Hi, here are some tips to improve your code.
- Avoid using absolute values. If the screen doesn't support those values, the layout starts to break. Try using max-weight/max-width to avoid that and also try relative measurements.
- The fonts aren't linked correctly. Try looking at the fonts.google page to see how to do it, as they explain how to add them to the head of your HTML.
Marked as helpful0 - @free-careSubmitted 17 days ago@AdanMaidanaPosted 17 days ago
Add some margin between the list items so they don't look so close together, and a little line height in the paragraphs
0 - @lucasfs022Submitted 17 days ago@AdanMaidanaPosted 17 days ago
make sure you put the transition in the element and not in the hover state of the element because the animation will look arrupt, put this transition: background-color 1s ease, color 1s ease in your social-link class, and not in the social-link:hover state and you will see more fluid transitions on hover for text and background of the anchor links
Marked as helpful0 - P@H3NRY-JRSubmitted 18 days ago@AdanMaidanaPosted 17 days ago
Great job with the animation on the shadow on hover but you forgot the detail of the cursor that has to change to a pointer when you hover it
Marked as helpful0 - @kodaicoderSubmitted 21 days agoWhat are you most proud of, and what would you do differently next time?
May be I can adding a loading transition be fore every thing shown up or made some skeleton component to preload a component
What challenges did you encounter, and how did you overcome them?Writing a test spec with Playwright