Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 368638 - Layout error when display MultiEditorBehaviorDemo in IE 8
Summary: Layout error when display MultiEditorBehaviorDemo in IE 8
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: lu lu CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-16 01:55 EST by Huo Zhen Zhong CLA
Modified: 2017-02-23 14:14 EST (History)
4 users (show)

See Also:

Attachments
test project (41.58 KB, application/x-zip-compressed)
2012-01-16 01:55 EST, Huo Zhen Zhong CLA 		no flags Details
screen shot (10.19 KB, image/jpeg)
2012-01-16 01:56 EST, Huo Zhen Zhong CLA 		no flags Details
sample (283.16 KB, application/x-zip-compressed)
2012-02-01 04:07 EST, Huo Zhen Zhong CLA 		no flags Details
patch for 368638 (946 bytes, patch)
2012-02-01 08:11 EST, lu lu CLA 		no flags Details | Diff
Using CSS to control combo display (729 bytes, patch)
2012-02-06 04:12 EST, lu lu CLA 		no flags Details | Diff
Using IE8 hack to control (659 bytes, patch)
2012-02-06 21:47 EST, lu lu CLA 		hjiyong: iplog+
hjiyong: review+
Details | Diff
Show Obsolete (2) 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 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