google-code-export / gwt-platform

Automatically exported from code.google.com/p/gwt-platform
1 stars 0 forks source link

Close handler for Popup PresenterWidget is set to null #416

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.

Start gwtp app. Go to presenter that have popup presenterwidget but don't start 
action yet for popup to be shown. Now navigate to some other presenter and 
return back to presenter with potential popup presenterwidget.

2.

Initiate action that will show popup presenterwidget. Click forward browser 
button so that app navigate to other presenter. 

3.

Click back browser button.

What is the expected output? 

App should return to original presenter but popup presenterwidget should not be 
revealed.

What do you see instead?

App is returned to original presenter and popup presenterwidget is revealed.

What version of the product are you using? On what operating system?

Gwtp 0.7, Debian 6 (Squeeze)

Please provide any additional information below.

PresenterWidget.internalHide() statement 
popupPresenter.getView().setCloseHandler(null) removes close handler before it 
is executed for popup view hide/close 'event' and because of that presenter 
still holds reference to popup presenterwidget in his popupChildren list.

Original issue reported on code.google.com by matija.j...@gmail.com on 3 May 2012 at 11:47

GoogleCodeExporter commented 9 years ago
You should close the popup presenter in the onHide() of the parent.

This is the expected behavior as the presenter should "remember" it's last 
state (and the popup).

Original comment by goudreau...@gmail.com on 23 May 2012 at 2:29