Skip to content
Submitted about 4 years ago

Desktop First using HTML5, Vanilla CSS3 and a little bit of JavaScript

imjackfrost1997•145
@imjackfrost1997
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Easybank landing page
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

Is it safe to set the background image(I used SVG image) on CSS background property? Feedback is appreciated thanks :D

Community feedback

  • Karim•810
    @Karimsamir112Posted about 4 years ago

    This is very good , but I have 1 suggestion:

    .you can use **margin-left :”” px ** on icons to give spaces for them (facebook ,instgram ,...etc) , and margin-top :””px for links in footer (about us , contact , ....etc)

    Happy coding and keep going .

    0
    imjackfrost1997•145
    @imjackfrost1997Posted about 4 years ago

    @Karimsamir112 Thankyou for the feedback I will definitely use that on my next challenge. Happy coding too ;)

    0
  • Raymart Pamplona•16,060
    @pikapikamartPosted about 4 years ago

    yep, nothing wrong with as it supported in browsers, also it is good since it has very low file size

    0
    imjackfrost1997•145
    @imjackfrost1997Posted about 4 years ago

    @pikamart oh thanks to you this solve my big question on background property :D

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord