Skip to content
OverviewSolutions
21
Comments
9
Islam abozeed
@islamabozeed

All comments

  • Islam abozeed•240
    @islamabozeedPosted over 1 year ago

    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
    @mariahw4Submitted over 1 year ago

    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!

    QR code display using CSS and HTML

    2
    Islam abozeed•240
    @islamabozeedPosted over 1 year ago

    woooooooow, it's awesome, nicely done

    0
  • Randal•120
    @rcroperSubmitted over 1 year ago

    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?

    Order Summary component

    2
    Islam abozeed•240
    @islamabozeedPosted over 1 year ago

    Amazing and wonderful design as first design hope I see great tasks as this one in the future

    0
  • Mariah Wear•20
    @mariahw4Submitted over 1 year ago

    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!!

    Responsive Result Summary Page using HTML and CSS

    3
    Islam abozeed•240
    @islamabozeedPosted over 1 year ago

    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
  • Islam abozeed•240
    @islamabozeedPosted over 1 year ago

    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
    @islamabozeedPosted over 1 year ago

    thanks a bunch for commits it's really helpful I made it in nft project and going well, thanks again!

    1
  • Islam abozeed•240
    @islamabozeedPosted over 1 year ago

    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
    @islamabozeedPosted over 1 year ago

    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
    @islamabozeedPosted over 1 year ago

    I wrote those tips up for further tasks, and surely it was helpful thanks a bunch!

    0