Islam abozeed• 240
@islamabozeed
Submitted
Islam abozeed
@islamabozeed
All comments
- Islam abozeed• 240
@islamabozeed
Posted
sure it is, I feel grateful of your comment, Thanks @Vishal Maurya, I put it all like a template excepting the footer, I'll separate it in further tasks! thanks again!<3
0 - Mariah Wear• 20
@mariahw4
Submitted
I'm excited with how this project went - I feel like I was able to create simple and functional code/css.
I am unsure/questioning if there's a better way to center my box containing the image/content vertically within the page. Until I added some padding this container seemed to just fill the vertical space of the whole browser window.
Thank you!
Islam abozeed• 240@islamabozeed
Posted
woooooooow, it's awesome, nicely done
0 - Randal• 120
@rcroper
Submitted
I used Flexbox for most of the layout for this project. Is there an advantage to using grid in this situation? How would you make this accessible?
Islam abozeed• 240@islamabozeed
Posted
Amazing and wonderful design as first design hope I see great tasks as this one in the future
0 - Mariah Wear• 20
@mariahw4
Submitted
This is my first submission of a project on this platform and I would welcome feedback!
I found it difficult to adjust the size and placement of the circle element and would appreciate input or suggestions on how to do it more smoothly.
I feel like I needed to create too many classes to properly get all the text sizes and colors and formatting done properly, so again would love feedback on ways to slim down my code.
Thank you!!
Islam abozeed• 240@islamabozeed
Posted
Nicely done Mariah, as a first project, hope you do better next projects will wait for this, the only commit is that the body not centered in page there're many ways to center the div in page ex: body{ min-height: 100vh; display: flex; align-items: center; justify-content: center; } also you can search for more which is proper for your design
0 - Manojmsp1180• 10
@Manojmsp1180
Submitted
Islam abozeed• 240@islamabozeed
Posted
Nicely Done and great effort. hope the challenge was helpful and learned much from it here is only one commit would make the design more similar as the original body{ background-color:#D6E1F0; } this will cover and change all the background to the exact color in original design
0 - Islam abozeed• 240
@islamabozeed
Submitted
Islam abozeed• 240@islamabozeed
Posted
thanks a bunch for commits it's really helpful I made it in nft project and going well, thanks again!
1 - Islam abozeed• 240
@islamabozeed
Submitted
I didn't add any of main landmarks as the document is kinda a component not a full landing page
Islam abozeed• 240@islamabozeed
Posted
noted, I centered it by only margin, that's why I seems without using position thanks for your comments and I'll improve in further designs.
0 - Islam abozeed• 240
@islamabozeed
Submitted
Islam abozeed• 240@islamabozeed
Posted
thanks for your comments, it's really helpful hope if you can send me a code example so I can fully understand your points thanks again
0 - Islam abozeed• 240
@islamabozeed
Submitted
it's a card for any product and easy to use in any landing page or e-commerce web sites
Islam abozeed• 240@islamabozeed
Posted
I wrote those tips up for further tasks, and surely it was helpful thanks a bunch!
0
