Skip to content
Submitted over 3 years ago

Responsive Landing Page (Laptop,Tablet,Mobile) Using Grid

SHAHAB MALIK•345
@Shahab-Malikk
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Manage landing page
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

Question 1 : One of the main thing is that i was having problem with background image is it beside body or it's background of header ?

Question 2 : I tried my best to change the color of circles under the testimonial section but i was unable to change it how can we change it?

Last thing is please rate my work out of 10 since it's my 3rd project of web development

Community feedback

  • Chamu•13,130
    @ChamuMutezvaPosted over 3 years ago

    Fix errors mentioned in the report , which might include some of the following:

    • image have no alt text , for decorative images must have an empty alt value.
    • the following are supposed to be buttons, using a div in place of a button does not assist assistive tech users . Apply the correct wai-aria attributes if you insist on using a div
    <div class="menue-toggle open ">
                <i class="fas fa-bars "></i>
            </div>
            <div class="menue-toggle close ">
                <i class="fas fa-times "></i>
            </div>
    • the swiper is working well , the responsiveness still needs attention - especially the footer (530px) and on medium devices there is a large white space

    Good lucky

    Marked as helpful
    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