Skip to content

Jonny Hicks/ JS-Project-movies#22

Open
KidFromCalifornia wants to merge 19 commits intoTechnigo:mainfrom
KidFromCalifornia:main
Open

Jonny Hicks/ JS-Project-movies#22
KidFromCalifornia wants to merge 19 commits intoTechnigo:mainfrom
KidFromCalifornia:main

Conversation

@KidFromCalifornia
Copy link
Copy Markdown

…yling, improve About page content, and optimize MovieDetails layout
…gation layout and implement decade-based movie retrieval
- Introduced a new LoadingIcon component that displays a spinning popcorn SVG as a loading indicator.
- Created a styled loading group and spinner using styled-components for better styling and animation.
- The spinner rotates continuously to indicate loading status, enhancing user experience during data fetching or processing.
Copy link
Copy Markdown
Contributor

@HIPPIEKICK HIPPIEKICK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I was a bit quick on the button there. You're missing the back-button on the details page 👀 Implement this and you're good to go 💪

@KidFromCalifornia
Copy link
Copy Markdown
Author

@HIPPIEKICK I had the option in the navigation bar, but I've made it clearer now, thanks for clarifying

Copy link
Copy Markdown
Contributor

@HIPPIEKICK HIPPIEKICK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aah, sorry about that! Nice work ⭐

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants