Revert nodes touch workaround
Summary: Since we now have the correct fix in place, this patch is to revert the hack we put in place to not be launch blocking. Reviewed By: sriramramani Differential Revision: D3019883
This commit is contained in:
parent
89cf5c49b9
commit
aa609a46ad
|
@ -59,13 +59,6 @@ import com.facebook.react.uimanager.annotations.ReactPropGroup;
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void setBackgroundColor(int backgroundColor) {
|
public void setBackgroundColor(int backgroundColor) {
|
||||||
// TODO t10316772 this if statement is a hack - we are mounting any transparent RCTView to an
|
|
||||||
// Android view, which is not necessary (and is inefficent). We're doing this to work around a
|
|
||||||
// bug where a non-Android view won't get touch because a View that is deeper in the hierarchy
|
|
||||||
// (i.e. will get onTouchEvent later) will handle it instead of the higher up view.
|
|
||||||
if (backgroundColor == 0) {
|
|
||||||
forceMountToView();
|
|
||||||
}
|
|
||||||
getMutableBorder().setBackgroundColor(backgroundColor);
|
getMutableBorder().setBackgroundColor(backgroundColor);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue