Skip to content
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

quest screen #17

Merged
merged 1 commit into from
Jan 14, 2019
Merged

quest screen #17

merged 1 commit into from
Jan 14, 2019

Conversation

wchristian
Copy link
Owner

Some quest screen translations extracted from #7.

image

@limyz
Copy link

limyz commented Dec 28, 2018

Looks awesome

@wchristian
Copy link
Owner Author

@limyz Thanks. :)

The questions that we'll now have to consider are:

Are there translations for "Confirm missions details" that are smaller so it doesn't squish the text and still accurate or do we leave it like that?

Is there a wording for the top text that'd fit the number in a gap, or do i need to try and add spaces to push the text to the right? If pushing it makes it so long that it crashes, can it be shortened?

@miyagami
Copy link

Are there translations for "Confirm missions details" that are smaller so it doesn't squish the text and still accurate or do we leave it like that?

Confirm Mission works just as well.

Is there a wording for the top text that'd fit the number in a gap, or do i need to try and add spaces to push the text to the right? If pushing it makes it so long that it crashes, can it be shortened?

Is it self centering?
Try pushing it to the right with: "new duties remain."

@wchristian
Copy link
Owner Author

I've changed to Confirm Mission.

  • Gonna try pushing it to the right.

@wchristian
Copy link
Owner Author

wchristian commented Jan 14, 2019

image

@wchristian wchristian merged commit 454c5df into master Jan 14, 2019
@wchristian wchristian deleted the quest_screen branch January 14, 2019 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants