Skip to content
OverviewSolutions
3
Comments
7
Jashwanth
@Jashwanth-k

All comments

  • Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    Everything is perfect great work

    1
  • Adedayo•200
    @pleasantvikSubmitted almost 3 years ago

    My focus is to improve my CSS and Javascript ability. Hence the reason i am practicing with ths challenges.

    Feedback on what to improved on will be highly appreciated

    CSS GRID CSS FLEXBOX VANILLA JAVASCRIPT

    1
    Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    great work, issues i was notices

    1. when we change the lightbox gallery image. main image also changing with it
    2. unable to close cart when clicked on body or image
    0
  • Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    great work, but the lightbox gallery is out of flow and everything is fine

    Marked as helpful
    0
  • Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    awesome design bro,how long does it take to complete

    1
  • Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    Great design - how long does it took complete, -> but you didn't add lightbox gallery to main image -> cart box design is a bit disturbing

    Marked as helpful
    0
  • Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    Thanks for the feedback, -- it took me 2 days to compelete😅 currently i was running out of time so i skipped the media queries that causes the design look smaller for different screen sizes

    1
  • Jashwanth•120
    @Jashwanth-kPosted almost 3 years ago

    You can use rem instead of px in css and add media queries

    1