productbrew / react-native-reanimated-flip

35 stars 12 forks source link

android issue related property zIndex of style #3

Closed andriy-shakh closed 4 years ago

andriy-shakh commented 4 years ago

in your style:

const styles = StyleSheet.create({
  container: {
    justifyContent: "center",
    alignItems: "center",
  },
  side: {
    width: "100%",
    height: "100%",
    position: "absolute",
  },
  front: {
    zIndex: 2,
  },
  back: {
    zIndex: 1,
  },
})

the property "zIndex" affects the client code if the back card has a button. it can't handle the click on the back of the card. if I have a button on the front card, it works fine. iOS works fine. if I delete next code with your package, it works fine for android (it handle click of a button on the back card)

 front: {
    zIndex: 2,
  },
  back: {
    zIndex: 1,
  },

if there a reason to keep these zIndex properties or maybe I can control this style from my client code?

czystyl commented 4 years ago

I think we can remove these two styles because they were used in the first iteration.

PR appreciated for this :)

andriy-shakh commented 4 years ago

should I prepare PR for this?

czystyl commented 4 years ago

@andriy-shakh if you can then would be great 🙌

czystyl commented 4 years ago

@andriy-shakh it should be fixed by this PR #8