Aram Hagen
@AramHagenAll comments
- @aayushyadavzSubmitted 4 months ago
- @EGUERREROP0Submitted 9 days ago@AramHagenPosted 9 days ago
Good job! Here are a few points: The space at the top and bottom doesn’t match the design. It would be better to use a single grid container instead of two (main and aside).
0 - @GinoGallardoSubmitted 14 days agoWhat are you most proud of, and what would you do differently next time?
Que maneje mejor el resonsive en tailwind
What challenges did you encounter, and how did you overcome them?el implementar grid en tailwind y con la ayuda de la ia y la documentacion pude resolverlo
What specific areas of your project would you like help with?tailwind
@AramHagenPosted 14 days agoGood job! I want to recommend to use
font-size:clamp(1.3rem, 1.3rem + 1vw, 2.2rem)
for thoseh2
for having responsive font-size.0 - @loiccapeSubmitted 23 days agoWhat are you most proud of, and what would you do differently next time?
I think i did a good job on this project , i start to getting faster on the way to prepare my coding.
What challenges did you encounter, and how did you overcome them?I did not realy encouter some challenges on this project.
What specific areas of your project would you like help with?i dont realy know when to use max or min width or height
@AramHagenPosted 23 days agoGreat job! I have some recommendation. Try to use css variables in your project. Check this link
https://www.w3schools.com/tags/tag_picture.asp
you can use picture tag for using different image in different device size.Marked as helpful0 - @CamrynTidsworthSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
This was my first time writing my own media queries without a tutorial so I'm proud of that. Next time, I want to get more clear on which units (em, rem, %, vh, etc.) to use when for responsive design.
What challenges did you encounter, and how did you overcome them?I encountered a few new concepts in this project including using the HTML tag and making a slightly more complex responsive design. I used google and youtube to answer my questions.
What specific areas of your project would you like help with?Any tips on responsive design or more general constructive critiques are welcome!
@AramHagenPosted about 1 month agoGood job! I wanna recommend this approach for import the google font to your project
I learned how to use the
&
symbol to import two fonts in my HTML(family=Outfit:[email protected]&family=Young+Serif)
. I also learned that the<link>
approach is faster and more efficient, while using@import
in CSS introduces delays and additional requests.@import url('https://fonts.googleapis.com/css2?family=Outfit:[email protected]&family=Young+Serif&display=swap')
; in the CSS file.- Adding
<link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Outfit:[email protected]&family=Young+Serif&display=swap" />
to the<head>
tag."
2 - @jeremymouzinSubmitted about 1 month agoWhat are you most proud of, and what would you do differently next time?
I used clamp() and vw unit to avoid using media queries for managing font sizes of text on mobile/desktop
@AramHagenPosted about 1 month agoGreat! I really like the use of clamp(). That was new to me and a great learning opportunity!
1 - @YoussefGaafarSubmitted about 1 month ago@AramHagenPosted about 1 month ago
Great! If you add padding to your container class is better so you don't need to add margin to your image and all of your box separately.
0 - @vengeance-makSubmitted about 1 month agoWhat challenges did you encounter, and how did you overcome them?
- Understanding the box model and how to leverage it to build the solution, took some time
I have tried my best to create this website. It would be better if we can improve up on it, in terms of simplifying the CSS or leveraging other tools to get a better output.
So, It would be better to know good tips about how to make the CSS better to get the output as intended, would be beneficial for next projects.
Thanks for this beautiful project
@AramHagenPosted about 1 month agoThe height of the card is not match with design. You didn't need to use for container. font-size of title is 22px not 18px.
Marked as helpful0 - @HDgamerH2W9Submitted about 1 month ago@AramHagenPosted about 1 month ago
Great! It will be better if you work on its responsive. Also, try not to have scroll you can fix the footer the top.
0 - @jeremymouzinSubmitted about 1 month ago@AramHagenPosted about 1 month ago
Great! You can use css variables. `:root { //colors--------------- --PattensBlue: #d5e1ef; --white: #ffffff; --SlateGrey: #68778d; --CatalinaBlue: #1f3251;
//breakpoints------------ --mobileSize: 375px; --desktopSize: 1440px;
//font weight-------------------- --light: 300; --regular: 400; --medium: 500; --bold: 700;
//font family--------------------- --fontFamily: "Outfit", sans-serif; }`
1