Fix: Enhancement in scenarios UI

anitab-org / powerup-android

PowerUp is an educational choose-your-own-adventure game that utilizes a users uploaded curriculum to empower pre-adolescents to take charge of their reproductive health. This is the Android version of the game.

GNU General Public License v2.0

116 stars 430 forks source link

Fix: Enhancement in scenarios UI #1427

Open aditmehta9 opened 4 years ago

aditmehta9 commented 4 years ago

Description

Enhance the UI of Level 1 & 2 scenario of Hospital and Library scenario

Fixes #1406

Type of Change:

Delete irrelevant options.

Code
User Interface

Code/Quality Assurance Only

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

This update in UI has been tested in 5.0 inch mobile to 6.3 inch mobile screen size.

Checklist:

Delete irrelevant options.

[ ] My PR follows the style guidelines of this project
[ ] I have performed a self-review of my own code or materials
[ ] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas

Code/Quality Assurance Only

[ ] My changes generate no new warnings

aditmehta9 commented 4 years ago

@robotjellyzone @sammy1997 I am sorry for doing pull requests multiple times. It won't happen again. I mistakenly had copied the link of AnitaB while cloning that's why this problem was created. Thanks for the guidance and for bearing my mistake. It will never happen again. Now all problems are sorted.

aditmehta9 commented 4 years ago

As you said earlier I have even squashed the commits

aditmehta9 commented 4 years ago

Yes, I have updated for level 1 and level 2 scenarios. Sorry, I forgot to mention the issue #1396.

aditmehta9 commented 4 years ago

Please look at mocks. I had included mocks of levels 1 and 2.

aditmehta9 commented 4 years ago

@sammy1997 I have squashed commits please review.

sammy1997 commented 4 years ago

@aditmehta9 I have closed #1396 Please remove it from the PR description

aditmehta9 commented 4 years ago

@aditmehta9 I have closed #1396 Please remove it from the PR description

Yes, Sir updated, please review whenever you are free @sammy1997 .