Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maryyy_ux_Movies_project #75

Closed
wants to merge 3 commits into from
Closed

Maryyy_ux_Movies_project #75

wants to merge 3 commits into from

Conversation

Maryyy-ux
Copy link

@dzc1
Copy link
Collaborator

dzc1 commented Dec 5, 2024

Hi Mary,

Great work on your project! 🎉 It's clear you've put effort into using the API effectively, and I can see the proper implementation for fetching and displaying movies—awesome job on that! 👏

I noticed the styling on the individual movie detail page could use a bit more refinement to enhance the user experience. Perhaps adding elements like better spacing, typography adjustments, or visually engaging components like a backdrop or poster image with a gradient overlay could make the details stand out more. Small tweaks like these can elevate the design significantly!

Your homepage is solid, and I love how smoothly the API integration works. Keep pushing those creative design choices to make your app shine even brighter. 💫

Overall, this is a great submission—you're on the right track! A little more styling polish, and it’ll be a stellar project.

Best,
Diego

@dzc1 dzc1 closed this Dec 5, 2024
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