Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 358379 - XULRunner setting should be disabled in linux
Summary: XULRunner setting should be disabled in linux
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: Yun Feng Ma CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-21 05:15 EDT by Xin Wu CLA
Modified: 2017-02-23 14:15 EST (History)
4 users (show)

See Also:

Attachments
XULRunner setting (1.32 MB, image/bmp)
2011-09-21 05:16 EDT, Xin Wu CLA 		no flags Details
Patch (2.85 KB, patch)
2011-09-29 05:12 EDT, Yun Feng Ma CLA 		lasher: iplog+
Details | Diff
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 Xin Wu CLA 2011-09-21 05:15:13 EDT 
Build Identifier: 20110920 nightly build

XULRunner setting should be disabled in linux. Now it is enabled, and MS IE is disabled.

Reproducible: Always

Steps to Reproduce:
1. Open EDT in linux
2. Open Window --> Preferences --> EGL --> Rich UI --> Appearance
3. "Mozilla XULRunner" option should be disabled but now it's enabled
4. "Microsoft Internet Explorer" option should be enabled but now it's disabled
Comment 1 Xin Wu CLA 2011-09-21 05:16:56 EDT 
Created attachment 203742 [details]
XULRunner setting
Comment 2 Justin Spadea CLA 2011-09-21 09:14:12 EDT 
I'm confused by your expectations. On Linux your only choice is XULRunner; there is no Internet Explorer for this platform. So I think the UI is correct, with the exception that the XULRunner radio button should be selected instead of IE.

(X) Mozilla XULRunner
( ) Microsoft Internet Explorer <disabled since not valid on Linux - or, better yet, hidden entirely>
Comment 3 Xin Wu CLA 2011-09-21 21:48:53 EDT 
Hi Justin,
Thanks for your comments. For the disable/enable setting, I found that the test scenario got some mistake. I'll correct it. 
And the radio button should be set to right option.
Comment 4 Yun Feng Ma CLA 2011-09-29 05:12:55 EDT 
Created attachment 204270 [details]
Patch

Here is a patch.
Comment 5 Yun Feng Ma CLA 2011-09-29 05:35:20 EDT 
Fixed. Thanks.
Comment 6 Xin Wu CLA 2011-09-29 21:51:18 EDT 
Verified on build 20110929
Comment 7 Lisa Lasher CLA 2011-10-11 16:22:03 EDT 
Closing this defect.