Skip to content
This repository has been archived by the owner on May 5, 2020. It is now read-only.

Dialogue scrolling #121

Merged
merged 7 commits into from
Aug 14, 2017
Merged

Dialogue scrolling #121

merged 7 commits into from
Aug 14, 2017

Conversation

identity2
Copy link
Contributor

@identity2 identity2 commented Aug 1, 2017

  • Updated dialogue images, so they have lighter stripes.
  • Updated the new hospital scenario background image.
  • Use UITableView instead of buttons for the choices in scenarios, so that the player could scroll down for more choices.
  • Karma point will be added after the "Gain Karma" pop up is shown.

(This branch should be merged after #116 )

Copy link
Contributor

@sunjunkie sunjunkie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good use of UITableView. :-)

@identity2
Copy link
Contributor Author

Yeah I really like UITableView, so convenient :D!

Copy link
Contributor

@jenpaff jenpaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The features work perfectly fine, however I noticed that the bottom response in the dialogue images is cut off. Can we change that?

@identity2
Copy link
Contributor Author

@jenpaff, could you send a screenshot of this issue? I'm not quite sure about what you mean.

@jenpaff
Copy link
Contributor

jenpaff commented Aug 12, 2017

screenshot_powerup
(see bottom response option)

Is that also happening your simulator ?

@identity2
Copy link
Contributor Author

@jenpaff, oh, actually I intentionally make it look like that so that I will be the same as the design of Android. You can scroll the dialogue down by dragging.

@jenpaff
Copy link
Contributor

jenpaff commented Aug 14, 2017

Ok, my bad !

@jenpaff jenpaff merged commit 1411b8f into anitab-org:GSoC17 Aug 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants