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

Force all layouts to be LTR #121

Merged
merged 1 commit into from
Sep 20, 2019
Merged

Force all layouts to be LTR #121

merged 1 commit into from
Sep 20, 2019

Conversation

cortinico
Copy link
Member

Developing full support for RTL would require to adapt all
the ViewPagers/TabLayouts to work correctly. I'm currently
fixing all the layouts to LTR and we'll wait for ViewPager2
to provide full RTL support.

List Transaction
Screenshot_1568907113 Screenshot_1568907116

Fixes #117

Developing full support for RTL would require to adapt all
the ViewPagers/TabLayouts to work correctly. I'm currently
fixing all the layouts to LTR and we'll wait for ViewPager2
to provide full RTL support.

Fixes #117
@cortinico cortinico added the bug Something isn't working label Sep 19, 2019
@cortinico cortinico added this to the 3.1.0 milestone Sep 19, 2019
@cortinico cortinico requested a review from redwarp September 19, 2019 15:32
@cortinico cortinico self-assigned this Sep 19, 2019
Copy link
Collaborator

@redwarp redwarp left a comment

Choose a reason for hiding this comment

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

Have you tried writing in the manifest android:supportRtl="false"?
I think it might work. https://android-developers.googleblog.com/2013/03/native-rtl-support-in-android-42.html

@cortinico
Copy link
Member Author

Have you tried writing in the manifest android:supportRtl="false"?

Yes. This will break the build because of Manifest merging if the user of the library applies android:supportRtl="true"

@redwarp redwarp merged commit 3d25aef into develop Sep 20, 2019
@redwarp redwarp deleted the fix-117-ltr branch September 20, 2019 10:44
@cortinico cortinico mentioned this pull request Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better RTL support
2 participants