Closed Bug 52739 Opened 24 years ago Closed 24 years ago

Change setup options' text in Linux installer

Categories

(SeaMonkey :: Installer, defect, P2)

Product:
SeaMonkey
Component:
Installer
Platform:
x86
Linux
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: tpringle, Assigned: samir_bugzilla)

References

Details

(Whiteboard: [nsbeta3++][pdtp2][rtm+])

Attachments

(3 files)

Patch
2.13 KB, patch
Details | Diff | Splinter Review
patch2: To make the dialog test legible again.
1.37 KB, patch
Details | Diff | Splinter Review
With patch2, the dialog looks legibel again as in the attached gif.
10.32 KB, image/gif
Details
Change text to: Recommended - Installs the most common options. Quickest to download; recommended for most users. Java not included. Full - Installs a range of options, including Java and several international language packages. Custom - Recommended for advanced users or users with specific international language package requirements.
Todd, I presume you want this for nsbeta3, hence nominating. PDT,management: Low impact trivial fix.
Status: NEW → ASSIGNED
Keywords: nsbeta3
QA Contact: gemal → gbush
Summary: Change download menu text on Linux → Change setup options' text in Linux installer
Target Milestone: --- → M18
CC'ing Vera for her input.
This is fine.
This is a trivial change, and important for reliability.
Priority: P3 → P2
Whiteboard: [nsbeta3+]
The windows version of this was in today's build. Is this done yet? If not you need to go through international as the UI freeze of 9/14 has passed.
ccing Jaime Rodriguez. I'm guessing you are okay with these changes, as they're beneficial for international users (i.e. steering them to the custom download option)? Plus this was done 9/14 for Windows.
Todd, Until I hear back explicitly from you that l10n is OK with this, I'll hold off on this bug and bug 52740. I'll assume your confirmation will cover both this bug and bug 52740 (Mac manifestation of this bug).
Whiteboard: [nsbeta3+] → [nsbeta3+] Awaiting OK from l10n
Adding Montse and Daniel to cc: list. Daniel/Montse - Please review these changes. My assumption is that they are OK and translatable (Note: These are the same text chnages that we made to Windows last week).
Assignee: sgehani → msanz
Status: ASSIGNED → NEW
UI change approved, go ahead and fix it. reassigning to samir
Assignee: msanz → sgehani
L10N is ok with this change.
pdt agrees p2.
Whiteboard: [nsbeta3+] Awaiting OK from l10n → [nsbeta3+][pdtp2]Awaiting OK from l10n
Attached patch Patch — — Splinter Review
Status: NEW → ASSIGNED
Whiteboard: [nsbeta3+][pdtp2]Awaiting OK from l10n → [nsbeta3+][pdtp2]
Whiteboard: [nsbeta3+][pdtp2] → [nsbeta3+][pdtp2] Fix in hand
Whiteboard: [nsbeta3+][pdtp2] Fix in hand → [nsbeta3+][pdtp2] Super reviewed fix in hand
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified in build 2000092206M18
Status: RESOLVED → VERIFIED
*** Bug 53726 has been marked as a duplicate of this bug. ***
Per bug 53726, this very verbose text that is displacing UI widgets should be trimmed. We'll remove the bit about "Navigator should be installed by default" in the Custom setup type description.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Whiteboard: [nsbeta3+][pdtp2] Super reviewed fix in hand → [nsbeta3+][pdtp2]
Status: REOPENED → ASSIGNED
*** Bug 53843 has been marked as a duplicate of this bug. ***
Too late to make localization changes for PR3 so adding rtm keyword. On the other hand this is the first thing people will see of our product -- can we really afford not to fix this? See screen shots attached to duplicate bugs (first is worse): http://bugzilla.mozilla.org/showattachment.cgi?attach_id=15359 http://bugzilla.mozilla.org/showattachment.cgi?attach_id=15316 Note that the attached patch is not the fix for the above problems, but rather the fix for the original problem. This bug was reopened due to the clipping problems with the results.
Keywords: rtm
Whiteboard: [nsbeta3+][pdtp2] → [nsbeta3+][pdtp2][rtm+]
Samir, is there a simple and safe change to force this window to be a little larger? Perhaps that would mostly solve the problem without requiring a localization change.
ssu, Please review patch2. Thanks.
looks like a simple size change. r=ssu
We wouldn't hold PR3 for this, but it's good to see what you're going to install, so marking nsbeta3++ with a second opinion from jar. If this bug hangs around without getting fixed, we'll push it past PR3.
Whiteboard: [nsbeta3+][pdtp2][rtm+] → [nsbeta3++][pdtp2][rtm+]
sr=a=vishy. go for it.
Fix landed on trunk and pr3 branch.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
build 2000092810MN6 looks good
Status: RESOLVED → VERIFIED
Keywords: vtrunk
Reopening to reresolve. Bugs stay resolved until they have been verified on both branch and trunk.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
reresolving as Fixed. vtrunk keyword in place requesting trunk verification. When this has been verified on teh trunk please set the bug to Verified and remove the vtrunk keyword. Thanks and sorry for the spam.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
verified on trunk build 2000111706
Status: RESOLVED → VERIFIED
Keywords: vtrunk
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: