Closed GoogleCodeExporter closed 9 years ago
Are you seeing the same problem with mgwt 2.0-m2-alpha2?
Original comment by kurka.da...@gmail.com
on 13 Jun 2014 at 9:43
Yes. Both 1.2 and 2.0.
Original comment by c...@rahmstrom.com
on 13 Jun 2014 at 9:45
Can you post a simple reproducing example?
Original comment by kurka.da...@gmail.com
on 5 Aug 2014 at 8:26
Any simple example seems to work fine. I can only reproduce with our rather
complex code. And it only happens occasionally.
I suggest this simple fix:
refreshHandler = scrollPanel.addScrollRefreshHandler(new
ScrollRefreshEvent.Handler() {
@Override
public void onScrollRefresh(ScrollRefreshEvent event) {
if (refreshHandler != null) { <-------------------------------------- New null check
refreshHandler.removeHandler(); <------------------------------- Avoids any NPE here
}
refreshHandler = null;
scrollPanel.scrollToPage(currentPage, 0, 0);
}
});
Original comment by c...@schoolity.com
on 5 Aug 2014 at 9:08
Well if this order is broken for any reason there is obviously something fishy
going on.
Can you post a more complex example that reproduces this?
btw: moving this to the new issue tracker:
https://github.com/mgwt/mgwt/issues/122
Original comment by kurka.da...@gmail.com
on 9 Aug 2014 at 9:08
Original issue reported on code.google.com by
c...@rahmstrom.com
on 4 Jun 2014 at 12:03