Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 359164

Summary:

The external type widgets added into a box causing VE error

Product:

z_Archived

Reporter:

fahua jin <jinfahua>

Component:

EDT

Assignee:

Huang Ji Yong <hjiyong>

Status:

CLOSED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

chenzhh, hjiyong, huozz, svihovec

Version:

unspecified

Target Milestone:

---

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
The sample project.	none
screen shot	none

Description fahua jin

2011-09-27 22:19:43 EDT

Build Identifier: 0.7.0.v201109192102

1) Open H6.egl in attached project with RUI editor, and you can see VE works fine.
2) Remove the comments from line 30 to 32.
3) Switch back to the Design pane, the VE is a blank page.

If do not add the external type widgets into the box, the VE works. This is also maybe a problem of JS generator.

Reproducible: Always

Comment 1 fahua jin

2011-09-27 22:20:59 EDT

Created attachment 204134 [details]
The sample project.

Comment 2 Huo Zhen Zhong

2011-10-13 03:03:43 EDT

Created attachment 205090 [details]
screen shot

Comment 3 Huo Zhen Zhong

2011-10-13 03:05:47 EDT

If open the handler in extra browser shown in screen shot, you will fond that it is a JS Gen defect, so change to owner to Scott.

Comment 4 Brian Svihovec

2011-10-26 14:15:01 EDT

This may have been fixed indirectly, but at this point it looks like the remaining issues are with the test case.  

The .js file for SimpleDiv should have the following content at the top:

egl.defineWidget(
	'org.eclipse.samplewidget', 'SimpleDiv',  		// this class
	'eglx.ui.rui', 'Widget',  	// the super class
	'div',						// dom element type name
{
	
	"constructor" : function() {
		this.eze$$DOMElement.style.backgroundColor="yellow";
		this.eze$$DOMElement.innerHTML = "My default text values";		
	},

note that defineClass has become defineWidget, a super class has been added, a widget type (div) has been added, and the document.createElement line was removed from the constructor.

Comment 5 Lisa Lasher

2011-11-22 11:06:54 EST

defer for now, but see the comment above that this is likely a test case problem.  Need to correct the testcase & rerun, and hopefully close this defect.

Comment 6 Huang Ji Yong

2011-12-14 04:06:03 EST

The sample should change according Brian in comment 4.
After making the change, the problem is gone.

Comment 7 fahua jin

2012-01-16 21:13:11 EST

Verified in 0.8.0.v201201150902-1CT-Fiv5_M5_LOPfUa4.