Ameer Jhan

Results 87 comments of Ameer Jhan
trafficstars

@inductor this is a valid point, will add this for the next release

No im still open for it

@ucarbehlul I'm unable to understand the problem, can you please elaborate

@pulpiks I think this is a duplicate PR for #213

@frankn11 we fixed this in the lates version, also please try the beta version which also has a potential fix

@frankn11 can you provide a codesandbox link which replicates your issue

Hi @simplecommerce this span is necessary to avoid usage of findDom API and ReactRef actually needs a dom element in the jsx, the className in the span tag is provided...

@simplecommerce the findDom was deprecated and it was causing warning in lot of productions that is why we switched to React Ref. I can understand your concern that the consumers...

@simplecommerce for your reference https://github.com/twobin/react-lazyload/issues/303 we ran a beta and then switched to stable but I honestly did not anticipate this

@danielpquinn I'm also starting think on the same line, if more people are affected I will make this into a Major update