Skip to content
OverviewSolutions
2
Comments
2
Xay
@xayrax88

All comments

  • Xay•20
    @xayrax88Posted about 2 months ago

    It looks great and very close to the design. I believe you could also implement media queries in CSS to make it responsive and have a mobile version of the QR component. Other than that great job!

    0
  • Talika Bajaj•650
    @Talika-BajajSubmitted 7 months ago
    What specific areas of your project would you like help with?

    Displayed share options with the help of display: flex property for small screen size and used position: absolute for larger screen size. I hope I have done it right.

    If someone could review my JavaScript code and give me feedback, that would be really helpful

    Article Preview Component

    1
    Xay•20
    @xayrax88Posted 4 months ago

    In one of your styles in CSS within the share options class add this "transition: transform 0.3s ease-in-out;" so that when you click it it has a nice smooth transition from when it disappears and appears again. Great job overall, you have it looking pretty much very closely to the picture. It looks good one a smaller mobile sized screen with media query. Your code is readable friendly, I would love your feedback on mine as well, when you have a moment. Thank you & happy coding.

    1