238022 - Remove "Additional Browser Enhancements" from installer

Status: VERIFIED FIXED

(Firefox :: Installer, defect)

Description

[ChanibaL The Antichrist]

User-Agent:       Mozilla/4.0 (compatible; MSIE 5.0; Windows 98; DigExt)
Build Identifier: 

When I install Firefox 0.8, I cannot sellect the "Additional Browser 
Enthansments" in the "select components" dialog.

The checbox is disabled, the second checkbox (Developer Tools) works fine.

Reproducible: Always
Steps to Reproduce:
1. Run the instaler
2. Select Custom, then the install path
3. Try to select the additionan browser enthasments

Actual Results:  
It just didn't want to be selected, the rest of the install went fine.

Expected Results:  
Install it.

I use Windows 98 SE with complete windows update. 
The install was performed to a new directory.

Comment 1

[Bill Mason]

I believe that's deliberate.  Those options have not yet been added to the
install package.

Comment 2

[Simon Paquet [:sipaq]]

This hasn't been implemented yet, but will be in the future.
-> INVALID

Comment 3

[ChanibaL The Antichrist]

Then mayby comment it somewhere in the instaler, so that others don't think 
that it's a bug, but it's still not implemented.

Comment 4

[Asa Dotzler [:asa]]

Actually, we will need to do something here for the release. This should be
removed or populated. Reopening so we don't forget about this. 

Comment 5

[Peter van der Woude [:Peter6]]

1) Why not ->NEW and future ?

2) Shouldn't there be an easy way to add those enhancements in case you change
your mind after you installed the browser w/o them ?

Comment 6

[Benjamin Smedberg]

This has impact on l10n, see also bug 254175. Not needed for PR-proper, but for
the l10n-freeze. I would like to land 254175 first, if at all possible.

Comment 7

[Ben Goodger (use ben at mozilla dot org for email)]

Attachment: patch

Comment 8

[Brian Ryner (not reading)]

Comment on attachment 157239 [details] [diff] [review]
patch

remove it from */installer.cfg as well, and r=me

Comment 9

[Ben Goodger (use ben at mozilla dot org for email)]

Attachment: patch

remove from installer.cfg too

Comment 10

[Ben Goodger (use ben at mozilla dot org for email)]

Benjamin, land my last patch here once you've landed the localizable installer
on the trunk. 

Comment 11

[Asa Dotzler [:asa]]

this is fixed on the trunk as far as I can tell.