My favorites | Sign in
Google
Project hosting will be READ-ONLY Wednesday at 8am PST due to brief network maintenance.
                
New issue | Search
for
| Advanced search | Search tips
Issue 1120: Calling UIObject.setVisible(boolean) on PopupPanel causes error when PopupPanel is not showing
1 person starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  rdayal+public@google.com
Closed:  Aug 2007
Type-Defect
Priority-High
Category-UI
Milestone-1_4_RC2
ReviewBy-jgw


Sign in to add a comment
 
Reported by hoosiemama, May 30, 2007
Found in GWT Release: 1.4 RC

Detailed description:
I was using a trick to cause PopupPanels to fade in by setting their
visibility to false, calling show, and then using scriptaculous to fade
them in.  In 1.4 RC this now results in an error.

Workaround if you have one:
None as of yet, still investigating

Links to the relevant GWT Developer Forum posts:
http://groups.google.com/group/Google-Web-Toolkit/browse_thread/thread/ea863e390cdff405

MyApplication.java
1.3 KB   Download
Comment 1 by hoosiemama, May 30, 2007
Error reported in Hosted Mode:

[ERROR] Uncaught exception escaped
com.google.gwt.core.client.JavaScriptException: JavaScript TypeError exception:
'__frame.style' is null or not an object
	at com.google.gwt.dev.shell.ModuleSpace.invokeNative(ModuleSpace.java:481)
	at com.google.gwt.dev.shell.ModuleSpace.invokeNativeVoid(ModuleSpace.java:270)
	at com.google.gwt.dev.shell.JavaScriptHost.invokeNativeVoid(JavaScriptHost.java:137)
	at com.google.gwt.user.client.ui.impl.PopupImplIE6.setVisible(PopupImplIE6.java:68)
	at com.google.gwt.user.client.ui.PopupPanel.setVisible(PopupPanel.java:314)
	at com.mycompany.client.MyApplication.onClick(MyApplication.java:39)
	at
com.google.gwt.user.client.ui.ClickListenerCollection.fireClick(ClickListenerCollection.java:36)
	at com.google.gwt.user.client.ui.FocusWidget.onBrowserEvent(FocusWidget.java:98)
	at com.google.gwt.user.client.DOM.dispatchEventImpl(DOM.java:1211)
	at com.google.gwt.user.client.DOM.dispatchEventAndCatch(DOM.java:1194)
	at com.google.gwt.user.client.DOM.dispatchEvent(DOM.java:1161)
	at com.google.gwt.dev.shell.ie.IDispatchImpl.callMethod(IDispatchImpl.java:126)
Comment 2 by skaflotten, Jun 05, 2007
I get the same error when simply trying to show a popup panel.

In FireFox, it works fine. In IE6, I get "Error: '__frame.style' is null or not an 
object.

This worked fine in 1.3.
Comment 3 by skaflotten, Jun 05, 2007
Clarification: It fails when I call:

    popup.setVisible(true);
    popup.show();

Not when I just call popup.show();
Comment 4 by gwt.team.morrildl, Jun 06, 2007
(No comment was entered for this change.)
Owner: gwt.team.jgw
Labels: -Priority-Medium Priority-High Category-UI Milestone-1_4_RC2
Comment 5 by gwt.team.jgw, Jun 12, 2007
(No comment was entered for this change.)
Owner: gwt.team.rdayal
Comment 6 by gwt.team.rdayal, Jun 12, 2007
This issue is also related to the problem of having to call show() before center()
when  centering a Popup. The result is a "jumpiness" effect.

Another problem has to do with the initial Popup position causing a browser window's
scrollbars to flash whenever a Popup is initially shown.

Here is a link to the thread describing these issues:

http://groups.google.com/group/Google-Web-Toolkit-Contributors/browse_thread/thread/23ee5bd6237dd6b/60c286e78ee8d212

We'll take care of these additional problems as part of this defect.
Comment 7 by gwt.team.jgw, Jun 15, 2007
(No comment was entered for this change.)
Status: Accepted
Comment 8 by gwt.team.rdayal, Jun 22, 2007
(No comment was entered for this change.)
Status: Started
Comment 9 by gwt.team.rdayal, Jun 22, 2007
(No comment was entered for this change.)
Status: ReviewPending
Labels: ReviewBy-jgw
Comment 10 by gwt.team.jgw, Jun 25, 2007
Fix committed as r1201.
Comment 11 by gwt.team.jgw, Jun 25, 2007
(No comment was entered for this change.)
Status: FixedNotReleased
Comment 12 by gwt.team.scottb, Aug 20, 2007
1.4 RC2 now released.
Status: Fixed
Comment 13 by sumitcha...@google.com, May 01, 2008
 
Owner: rdayal
Sign in to add a comment