@ClaudiaRamirezD
Latest solutions
Responsive recipe page using CSS Flex
Submitted 8 months agoI would love to get a feedback and potential shortcuts that I could've done for my CSS solution.
Latest comments
- @AlpaarX
Looks great! But here are some points I would improve:
- spacing between text elements is too big when viewing the website on fullscreen.
Hopefully it will help you improve your project!
- @ChimiRinzin-HWRWhat are you most proud of, and what would you do differently next time?
From a comment made on from my previous work, I learnt how to properly center HTML elements and also learnt to properly design a web page.
What challenges did you encounter, and how did you overcome them?Initially, it was really difficult to arrange the elements according to what I wanted but after patiently experimenting and tweaking with the CSS, I was able to easily arrange all the items according to my desires.
@AlpaarXLooks really great! There are some minor fixes that you might apply in your future projects:
-
ol li::marker, ul li::marker{ color: var(--color-brown-800) }Here you don't need to specify the type of list unless you are applying different styles, just li::marker is enough. -
for sizing and padding/margin try to use relative values as they give you more consistency and potentially responsiveness.
I hope these advices will be helpful for you!
Marked as helpful -
- @Lordgebrio@AlpaarX
Here are some points that might help you improve your project
- Use shorter duration for transition
- Use relative measurements for sizing text and elements (em, rem, %)
- Use CSS variables to assign colors, it might help you change colors by changing code only in one place, instead of changing it on every element
Marked as helpful - @immark007What are you most proud of, and what would you do differently next time?
Talvez usar um button pro learning
What challenges did you encounter, and how did you overcome them?sombreamento do componente
What specific areas of your project would you like help with?não sei
@AlpaarXToo many wrapping containers just for 1 child element. An element can be used as a container itself.
Failed to find styles file, even though it's mentioned in html code. Please make sure that all folders in your GitHub repo are visible.
- @Guddi8@AlpaarX
It looks good overall, but CSS for the card is a little overcomplicated in my opinion, could've made it simpler.
- @shreeharsh1141@AlpaarX
you need to use border-radius: top left bottom right; in order to get rounder corners wherever you want.
Marked as helpful
