@ClaudiaRamirezDSubmitted about 2 months ago
Alpaar
@AlpaarXAll comments
- @AlpaarXPosted about 2 months ago
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!
0 - @ChimiRinzin-HWRSubmitted 3 months agoWhat 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.
@AlpaarXPosted about 2 months agoLooks 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 helpful1 -
- @LordgebrioSubmitted 2 months ago@AlpaarXPosted about 2 months ago
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 helpful0 - @immark007Submitted 2 months agoWhat 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
@AlpaarXPosted 2 months agoToo 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.
0 - @Guddi8Submitted 2 months ago@AlpaarXPosted 2 months ago
It looks good overall, but CSS for the card is a little overcomplicated in my opinion, could've made it simpler.
0 - @shreeharsh1141Submitted over 3 years ago
- my corners are not rounder. why?
@AlpaarXPosted over 3 years agoyou need to use border-radius: top left bottom right; in order to get rounder corners wherever you want.
Marked as helpful0
