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

Bug 368638

Summary: Layout error when display MultiEditorBehaviorDemo in IE 8
Product: z_Archived Reporter: Huo Zhen Zhong <huozz>
Component: EDTAssignee: lu lu <lulu>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: hjiyong, jinfahua, mayunf, svihovec
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
test project
none
screen shot
none
sample
none
patch for 368638
none
Using CSS to control combo display
none
Using IE8 hack to control hjiyong: iplog+, hjiyong: review+

Description Huo Zhen Zhong CLA 2012-01-16 01:55:15 EST 
Created attachment 209522 [details]
test project

1. import attached test project
2. open MultiEditorBehaviorDemo in IE 8
3. there is layout error, see attached screen
Comment 1 Huo Zhen Zhong CLA 2012-01-16 01:56:03 EST 
Created attachment 209523 [details]
screen shot
Comment 2 Huo Zhen Zhong CLA 2012-01-16 02:20:32 EST 
find in 0.8.0.v201201152102
Comment 3 lu lu CLA 2012-01-19 05:56:48 EST 
Had taken new comer training on Wed and Thu, and will take vacation from 1.20 to 1.28, so need to defer this defect to 0.8.0 I2. Thanks.
Comment 4 Brian Svihovec CLA 2012-01-19 14:38:57 EST 
The file MultiEditorBehaviorDemo does not seem to appear in the attached project.

I am ok with deferring this to I2.
Comment 5 Huo Zhen Zhong CLA 2012-02-01 04:07:11 EST 
Created attachment 210358 [details]
sample
Comment 6 Huo Zhen Zhong CLA 2012-02-01 04:07:42 EST 
Please use the sample project.
Comment 7 lu lu CLA 2012-02-01 08:11:58 EST 
Created attachment 210369 [details]
patch for 368638

Fix this bug by removing combo class while using IE.
Comment 8 lu lu CLA 2012-02-01 08:13:55 EST 
Hi YunFeng, Can you help to review the patch? Thanks.
Comment 9 lu lu CLA 2012-02-06 04:12:36 EST 
Created attachment 210558 [details]
Using CSS to control combo display

Using CSS to control combo display, remove fixed 'width', then the width of combo will self-adjust with the width of opinions. and test on IE,FF and Chrome. Thanks.
Comment 10 lu lu CLA 2012-02-06 21:47:29 EST 
Created attachment 210627 [details]
Using IE8 hack to control

Using IE8 hack '\0 ' to control 'width' in CSS, has been verified on FF/chrome/IE6/7/8/9.

Hi JiYong, please help to review and commit. :-) Thanks.
Comment 11 Huang Ji Yong CLA 2012-02-06 21:50:45 EST 
Comment on attachment 210627 [details]
Using IE8 hack to control

Looks good and lulu has tested across browsers. Committed.
Comment 12 lu lu CLA 2012-02-06 21:57:44 EST 
Mark this bug as resolved. Thanks.
Comment 13 Huo Zhen Zhong CLA 2012-02-20 00:18:10 EST 
verified in 0.8.0.v201202190901