Imran• 150
@xdevimran
Submitted
feedback are always welcome
Adepoju Oluwatobi
@adepoju-oluwatobi
All comments
- Adepoju Oluwatobi• 200
@adepoju-oluwatobi
Posted
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 - Samuel0008• 40
@Samuel0008
Submitted
I had a lot of challenges moving my code locally from VSCode and putting it on my repository on Github. Please suggest a way for me to do so more easily.
Adepoju Oluwatobi• 200@adepoju-oluwatobi
Posted
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 - yetkin karasungur• 560
@yetkinkrsngr
Submitted
Adepoju Oluwatobi• 200@adepoju-oluwatobi
Posted
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-wei
Submitted
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.
Adepoju Oluwatobi• 200@adepoju-oluwatobi
Posted
Rem would work fine, but i also used percentage for the box sizing since it is responsive on its own.
Marked as helpful
1 - Pablo Olvera• 20
@pablohimself
Submitted
I need to figure how to make the eye icon appear while hovering the image.
Adepoju Oluwatobi• 200@adepoju-oluwatobi
Posted
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-ThanHtike
Submitted
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.
Adepoju Oluwatobi• 200@adepoju-oluwatobi
Posted
I love the transition effects... cool!
1
