Skip to content

Conversation

@Hari-krishna-tech
Copy link

@Hari-krishna-tech Hari-krishna-tech commented May 18, 2025

PR to fix #9

Changes:

  • Made UI responsive in landing page, better user experience in smaller devices.

before:
image

After:
image

  • Fixed UI in both result and image page.

PS: I was unable to run the query engine server because I don't have any data in the database. This also prevented me from taking before and after images. Adding an in-memory database for development could rectify this.

@IonelPopJara
Copy link
Owner

Hello @Hari-krishna-tech! Thank you for submitting this PR and try to help make Moogle better. I completely disregarded responsive design when designing the UI.

However, after testing it in my Machine I saw a couple of issues.
image
image
image

This might've been my fault because currently the Dockerfile has a bug that I forgot to fix. To properly compile and run the project with docker you have to do this.

npm install
npm run build
docker compose build
docker compose up

This is because I'm wrongly copying the build folder in the Dockerfile. So most likely when you tested the changes it was not compiled properly. Let me know what you think.

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.

Mobile Version UI is broken

2 participants