Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add option to lock screen orientation#128

Merged
brarcher merged 3 commits into
masterfrom
lock-orientation
Jul 17, 2017
Merged

Add option to lock screen orientation#128
brarcher merged 3 commits into
masterfrom
lock-orientation

Conversation

@brarcher
Copy link
Copy Markdown
Owner

@brarcher brarcher force-pushed the lock-orientation branch 2 times, most recently from 49277b1 to 0785b5d Compare July 17, 2017 01:19
brarcher added 3 commits July 16, 2017 21:26
Even with passing a bundle with the layout ID sometimes
some users will encounter a situation where the _layout
variable is null. To avoid further issues, creating a
class for each intro slide, so the layout is encoded
into the class itself.
When passing a phone to a clerk to scan the barcode, if the
phone is rotated and the screen reloads it can be bothersome
or confusion. To avoid this situation, a new option is
added to lock the screen.

A menu icon is now added which defaults as unlocked. When
touched the app sets its orientation to the "natural" orientation
of the device. When touched again the sensor dictates the
orientation of the device.
@brarcher brarcher merged commit bf60976 into master Jul 17, 2017
@brarcher brarcher deleted the lock-orientation branch July 17, 2017 01:32
@brarcher
Copy link
Copy Markdown
Owner Author

Hi @Arno-github, @lgasp, @techwebpd, @Clonewayx, @pbeckmann, @airon90 and @PanderMusubi:

This request adds some strings which are used in a feature which locks the screen orientation. This is to help prevent the screen from rotating when handing the phone to a clerk to scan a barcode.

The strings are marked with:

<!-- NEEDS TRANSLATED -->

If any are interested in updating translations, please create a merge request with the changes, or attach an updated file and I'll add it.

Thanks!

@PanderMusubi
Copy link
Copy Markdown

Could you please set up a translation project on Transifex or other translation website? In that way, translations will go much quicker and can be made an reviewed in a more efficient way.

@brarcher
Copy link
Copy Markdown
Owner Author

I've setup a project here: https://www.transifex.com/na-243/loyalty-card-locker/dashboard/

I'm not sure what needs to be done to add someone who can contribute a translation. Do I need to have your email address to add you?

@PanderMusubi
Copy link
Copy Markdown

I just requested to become a translator. Think you just have to approve that. Perhaps it is also possible for you to add translators. Probably best is to notify all translaters to go there and also request access.

@brarcher
Copy link
Copy Markdown
Owner Author

I've accepted your request to join. In the future when new strings are added, is all I need to do is update the English strings on the site, and everyone will be notified automatically? Then, before I make a release just take the most recent translations from the site and commit them?

@PanderMusubi
Copy link
Copy Markdown

Notification depends on how translaters have configured this in Transifex. However, more people can make translations in this way.

Yes, before each release, simple used the latest version from Transifex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants