Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 359164 - The external type widgets added into a box causing VE error
Summary: The external type widgets added into a box causing VE error
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Huang Ji Yong CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-27 22:19 EDT by fahua jin CLA
Modified: 2017-02-23 14:15 EST (History)
4 users (show)

See Also:

Attachments
The sample project. (44.19 KB, multipart/x-zip)
2011-09-27 22:20 EDT, fahua jin CLA 		no flags Details
screen shot (77.54 KB, image/jpeg)
2011-10-13 03:03 EDT, Huo Zhen Zhong CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description fahua jin CLA 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 CLA 2011-09-27 22:20:59 EDT 
Created attachment 204134 [details]
The sample project.
Comment 2 Huo Zhen Zhong CLA 2011-10-13 03:03:43 EDT 
Created attachment 205090 [details]
screen shot
Comment 3 Huo Zhen Zhong CLA 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 CLA 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 CLA 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 CLA 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 CLA 2012-01-16 21:13:11 EST 
Verified in 0.8.0.v201201150902-1CT-Fiv5_M5_LOPfUa4.