-
Notifications
You must be signed in to change notification settings - Fork 1
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
(issue)Redesigned store #177
Conversation
Warning: this is incomplete This is for issue #71. - Updated the store - Removed ads.js - Reordered battlegrounds in live.js
Previously the store would only work for landscape, but now it works on both.
- Removed all `main>` from `main>section` - Added empty lines for prettiness - Added blank upgrades button container - Added a comment for scripts
When you turn it to landscape it gets crushed |
@VishwamR, can you send me a screenshot? |
Wait never mind. I understand what you mean. That is because you are viewing in a browser on a mobile device. This doesn't happen on a computer and users on mobile device will most likely have the app because that is either how they found out or they actually want to play so they download it to remove the browser bars. This should be fine. |
- Removed wood background - Removed merchant - Removed shuffle icon
This resolves issue #157. - Added title attributes to applicable elements on the store page - Cleaned up css - Cleaned up js - Added character upgrades - Added GA events - Added comments to uncommented code
This is the new store.
Changes:
Please check it out to see if there should be any improvements or changes before publishing, @ethanchoo, @VishwamR, @datdopegamer
I still have to add before merging: