Skip to content
OverviewSolutions
14
Comments
6
Adepoju Oluwatobi
@adepoju-oluwatobi

All comments

  • Adepoju Oluwatobi•200
    @adepoju-oluwatobiPosted over 1 year ago

    Nice work! But when I checked the component is not aligned to the center. You can achieve this by using flex for the parent container. Then give it a justification of center and align it to the center as well. Do not forget to include height of 100vh.

    1
  • Adepoju Oluwatobi•200
    @adepoju-oluwatobiPosted over 1 year ago

    To do this more easily, you can create a repository in github first and then clone it using the git clone command. That way all you have to do is just git add . Git commit, and fit push from the terminal.

    1
  • Adepoju Oluwatobi•200
    @adepoju-oluwatobiPosted over 1 year ago

    Nice Job! But I observed that the component still submits a null rating when the submit button is clicked. You can create an auth method to know when the rating is empty and prevent them submitting a null rating. Also, i observed that the container for the component is not fixed, i think it should be fixed to prevent it from overflowing and distorting when viewed on different screen.

    Marked as helpful
    0
  • Bowen•280
    @bowen-weiSubmitted over 1 year ago

    Hello, I'm confused by the concept of responsive design. I tried to use 'rem' everywhere instead of fixed units, but when the screen size changes, the size of the components does not seem to change, so I end up changing the text size of every div by hand. Can anyone recommend a video or article that clearly explains how to think about responsive design? Thanks in advance.

    Product preview card component

    2
    Adepoju Oluwatobi•200
    @adepoju-oluwatobiPosted over 1 year ago

    Rem would work fine, but i also used percentage for the box sizing since it is responsive on its own.

    Marked as helpful
    1
  • Adepoju Oluwatobi•200
    @adepoju-oluwatobiPosted over 1 year ago

    Really Nice Work. You really get the width and height of the preview card. But I observed that the eye-image is not present. I also had a bit of challenge achieving that. But you can you opacity to achieve it.

    0
  • ThanHtike（テンタイ）•110
    @CGM-ThanHtikeSubmitted over 1 year ago

    Thanks for checking this out

    I put a transitions on hover, but I'm not sure this is the right approach.

    "If you have any suggestions or feedback for my code, please feel free to comment." I would greatly appreciate all feedback.

    I put a subtle transitions on hover

    2
    Adepoju Oluwatobi•200
    @adepoju-oluwatobiPosted over 1 year ago

    I love the transition effects... cool!

    1