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

Project Movie site using dynamic routers #76

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

Sherrydev11
Copy link

Project Movie site using dynamic routers
Collab with https://github.com/smily342

https://lemaisoncinemaa.netlify.app/

@dzc1
Copy link
Collaborator

dzc1 commented Dec 5, 2024

Hi Sherry,

Fantastic work on your project! 🎬 I love how you've brought a unique style to your app—your creative approach really stands out, and the name "Le Maison Cinema" is brilliant. It gives your project a classy, cinematic vibe—great choice! 👏

You’ve implemented the API correctly and made excellent use of it on the individual movie pages. The styling there is sharp, and the images are crystal clear, which adds a lot to the user experience.

On the homepage, I noticed some of the images appear pixelated. Perhaps adjusting the image resolution by using larger sizes (like w500 or w780 from the TMDb API) could fix this and bring the homepage visuals up to the same level as your individual movie pages. Small tweaks like this can create a more cohesive experience across the app.

Overall, this is a fantastic submission—keep up the great work! With those minor adjustments, your project will truly shine. 🌟

Best,
Diego

@dzc1 dzc1 closed this Dec 5, 2024
@JennieDalgren JennieDalgren reopened this Jan 25, 2025
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.

3 participants