P
samritbasnet• 180
@samritbasnet
Submitted
P
samritbasnet
@samritbasnet
All comments
- Psamritbasnet• 180
@samritbasnet
Posted
thank you for comment that helped me with my functionality
0 - LucasNgTg• 190
@LucasNgTg
Submitted
Psamritbasnet• 180@samritbasnet
Posted
looks perfect
0 - Gideon Akpan• 110
@gideonakpan
Submitted
What are you most proud of, and what would you do differently next time?
Proud of completing the project, and using javascript to validate my form
What challenges did you encounter, and how did you overcome them?
None
What specific areas of your project would you like help with?
How to make my input background change colour
Psamritbasnet• 180@samritbasnet
Posted
preety well done i would just recomend adjusting font size other than that looks good
0 What are you most proud of, and what would you do differently next time?
I used an SVG element for the popup triangle instead of the typical ::before and borders approach.
What challenges did you encounter, and how did you overcome them?
I faced difficulties in getting the image to match the height of the adjacent column in the desktop view. I resolved this by adjusting the margins and paddings of the right column, making both columns visually balanced and aligned in height.
What specific areas of your project would you like help with?
I would like to better understand how elements take up space and how they influence each other in terms of layout. Specifically, I'm interested in learning about the factors that affect the size and positioning of elements within a container and how these interactions can be managed.
Psamritbasnet• 180@samritbasnet
Posted
looks good
0- PDaniel Dominguez• 150
@dominguez-daniel
Submitted
Psamritbasnet• 180@samritbasnet
Posted
excellent job
0 - gk6294• 150
@gk6294
Submitted
Psamritbasnet• 180@samritbasnet
Posted
overall good work i would be adjustting font color for paragraph of kira whittle and jeanete harmon in your site and increase the gap between the cards.
0 - Musicalaudio• 160
@Musicalaudio
Submitted
Had some trouble with getting the right sizing in the desktop layout. Feel like the size of the cards are a bit too wide relative to the paragraph below the main heading.
As a whole it feels like it's too large, but that maybe due to my laptop having a small screen.
Any advice for sizing the desktop layout is appreciated.
Psamritbasnet• 180@samritbasnet
Posted
overall it looks good.for me i did this way @media (max-width: 1140px) { .card-container { grid-template-columns: 1fr; }
.card:nth-child(n) { grid-column: 1; grid-row: auto; } }.And for card container property .card-container { display: grid; grid-template-columns: repeat(3, 10fr); gap: 20px; margin-top: 60px; }
0 - PDavid de Alencar• 120
@dealencardavid
Submitted
Psamritbasnet• 180@samritbasnet
Posted
i see the main border color is white for the elements you can adjust that other than that 169.99 looks to have letter-spacing which you can adjust.Anyways the final product looks amazing good work.
Marked as helpful
1 - ClemFrance• 40
@ClemFrance
Submitted
Psamritbasnet• 180@samritbasnet
Posted
i have seen few issue first would be the selection of background color and finally the table element is not inside the body element make sure the div section is inside it and be sure you gave right padding and margin.
0 - EstebanReyes• 30
@reyes630
Submitted
What are you most proud of, and what would you do differently next time?
Las habilidades de escritura han mejorado, tratarĂa de usar menos codigo
What challenges did you encounter, and how did you overcome them?
No enfrente ningun desafio
What specific areas of your project would you like help with?
En la animacion al pasar el cursor
Psamritbasnet• 180@samritbasnet
Posted
The font color is not set properly for your project and the size of main content is not appropriate other than that spacing between element should be align properly and the sizing of the main element at bottom margin should be well kept.
0 - Jorge Cordero• 70
@jorgecorderozx
Submitted
Psamritbasnet• 180@samritbasnet
Posted
I see your hover effect is not working in the page you can do that by adding following properties .title:hover { color: hsl(47, 88%, 63%); cursor: pointer; }.Other than that everything looks good for me.
Marked as helpful
0 - alexrtm• 50
@alexrtm
Submitted
What are you most proud of, and what would you do differently next time?
I was able to get the component to look pretty similar to the design.
What challenges did you encounter, and how did you overcome them?
The main challenge with this one was making sure the image was an appropriate size and was not overflowing its container. Setting the image to block display and giving it a max-width of 100% took care of the sizing issues. Using auto margins for the image made sure it was centered.
What specific areas of your project would you like help with?
-
I would like feedback on the choice of my html elements and structure. Specifically, I would like to know if I chose the correct semantic elements.
-
In order to center the component in the body, I had to add this css rule:
html, body { height: 100%; }I am not sure why I had to do this, even when I made the body
display: flexand used center forjustify-contentandalign-items. -
Was it appropriate to give the body a flex display to center the component?
-
Did I properly size the component?
Psamritbasnet• 180@samritbasnet
Posted
I would suggest you to flex-direction column and min-height property:100vh to maintain the size of component also margin of 0.
0 -
