-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix: Added the Open Food facts logo, changed the button to the orange theme and fixed the broken link. #30
Conversation
…e and corrected the play store broken link.
@teolemon Requesting your review for the PR |
index.html
Outdated
@@ -8,7 +8,7 @@ | |||
<body> | |||
<div class="navbar"> | |||
<div class="navbar-container"> | |||
<a href="#"><img src="img/logo_nav@3x.png" alt="Open Food Facts" width="150px"></a> | |||
<a href="#"><img src="img/open food facts logo.png" alt="Open Food Facts" width="250px"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you normalize the name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure @teolemon
Looks good to me @Aarthy153 👍 can you just normalize the file name? |
@teolemon Can I develop this prototype to match the official theme and colour codes of the Open food facts? So then it will suitable for the deployment right? |
@teolemon I have done the required changes in the file name. |
The current "mobile app" website is this one https://world.openfoodfacts.org/open-food-facts-mobile-app In terms of design we can surely use the new one, but from a technical perspective that's a different thing, |
I'm going to merge anyway, it will fix current issues. Thanks @Aarthy153 |
…e and corrected the play store broken link.
What
Added the Open Food facts logo, changed the button to the orange theme, and fixed the broken link.
Screenshot
Fixes bug(s)
#28