react-native-community / releases

React Native releases
https://github.com/facebook/react-native/
1.5k stars 405 forks source link

0.59.3 Text component on Android getting other side on textAlign #110

Closed tamirrab closed 5 years ago

tamirrab commented 5 years ago

Hi Text: on Android only, textAlign: "right" doing "left" instead of right. and "left" doing "right" instead of left. only on 0.59.3 and only on Android

mikehardy commented 5 years ago

This isn't the main repo, I think @kelset meant the facebook/react-native repo, not react-native-releases

That said, this looks directly related to the RTL changes https://github.com/facebook/react-native/commit/871290f (from https://github.com/facebook/react-native/releases), and I'm guessing it's because whatever device you checked it on had an RTL locale set but this wasn't ever expressed until 0.59.3?

That's almost a breaking change really, if I'm correct, to have text direction change in a patch release...

tamirrab commented 5 years ago

There is a different only on android and only from changing from 0.59.2 to 0.59.3 The same device and same repository

On iOS it’s OK

mikehardy commented 5 years ago

Right, you updated react-native, so you might check the 0.59.2 to 0.59.3 changelog, it would be evident iOS wouldn't change, because there were no changes there. https://github.com/facebook/react-native/compare/v0.59.2...v0.59.3

tamirrab commented 5 years ago

So how can I create an issue to fix that?

mikehardy commented 5 years ago

https://github.com/facebook/react-native/issues

you may be able to work around this with your text properties though. The upstream change was the right thing to do, it is just that your app wasn't expecting RTL to actually work and it is now.

tamirrab commented 5 years ago

I think there is an issue because "right" on LTR need to be on the right side. on iOS is in the right side. on android 0.59.2 is in the right side on android 0.59.3 is in the LEFT side something is not good

reyn-nova commented 5 years ago

@tamirrab I think it's better to close this as you have open another one and already tracked reproducible in the main repo

turnrye commented 5 years ago

Thanks for the link to the main repo -- I'm closing this here since it's not related to the release process.