Skip to content

Force all layouts to be LTR#121

Merged
redwarp merged 1 commit into
developfrom
fix-117-ltr
Sep 20, 2019
Merged

Force all layouts to be LTR#121
redwarp merged 1 commit into
developfrom
fix-117-ltr

Conversation

@cortinico

Copy link
Copy Markdown
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

@redwarp redwarp left a comment

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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