Feature/main page css - Githubissues

em2396 / rancid-tomatillos

1 stars 1 forks source link

Feature/main page css #15

Closed em2396 closed 9 months ago

em2396 commented 9 months ago

What I did:

Added left and right arrows to move between movies
changed the background color to black
adjusted the height and width of movie displays
removed titles from rendering

Did this break anything?

[x] No
[ ] Yes

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] Styling
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Refactor(DRY-ing up/ reorganizing code, etc.)
[x] Super small fix (Corrected a typo, removed a comment, etc.)
[ ] Skip all the other stuff and briefly explain the fix.

Checklist:

[x] If this code needs to be tested, all tests are passing
[x] The code runs locally
[ ] There are comments where clarification/ organization is needed
[x] The code is DRY. If it's not, I tried my best
[x] I reviewed my code before pushing

✨ What's next? Continue styling, add video API to singleMovie, begin iteration 4 with cypress

em2396 commented 9 months ago

I got the heart button to work. I originally put the function inside of App.js but this caused the state to update all the heart buttons so the function to toggle between heart .svg is in movie and so is the useState. The only issue is, if you arrow over to the point where the movie is hidden, the heart resets. I will come back to this issue later.