Flex box to center the card as well as the image.

Community feedback

• 

  Kinga Miśkiewicz•160

  @codingbearyPosted over 1 year ago

  Hi! :) The solution looks great but I would recommend improving a few things.

  1. Accessibility - you should add <main> section and <footer> so browsers can read your code better. Read about semantics in HTML :)
  2. To center the app perfectly you can wrap it into container and then from CSS add .container { display: flex; justify-content: center; align-items: center; min-height: 100vh; }
  3. Best practice is to write your HTML and CSS in separate files so keep that in mind :)
  4. In terms of responsiveness there is nothing to work with in this project because it's small by default
  5. I think you're using different font so remember to import fonts you want to use into your HTML :)

  I hope those tips were helpful :)

  Marked as helpful

  1

  ---

  

  Alex Mercer•20

  @AlexMercer4Posted over 1 year ago

  Thank you for the advice. I've started learning web developement fairly recently so its definitely helpful to get advice from someone else. I'm looking forward to more advice/tips in the future.

  0