oblador / react-native-lightbox

Images etc in Full Screen Lightbox Popovers for React Native
MIT License
2.82k stars 502 forks source link

Lightbox can receive onLongPress function as a prop. #108

Closed TomJamesDuffy closed 4 years ago

EricPKerr commented 5 years ago

@TomJamesDuffy You can simplify this code quite a bit with:

static defaultProps = {
  onLongPress: () => {},
  ...
};

render() {
  return (
    <TouchableHighlight
      ...
      onLongPress={this.props.onLongPress}
    />
   );
}
TomJamesDuffy commented 5 years ago

Thanks for the pointer Eric, that is much simpler!

ChrisEdson commented 5 years ago

Hello! I think this would be a great feature to add in if possible. Should be a simple merge @oblador ?

pzatorski commented 5 years ago

Please merge this feature... @oblador

sqlninja commented 4 years ago

Really need this feature... What is needed to get this merged?