Closed Bug 333383 Opened 19 years ago Closed 19 years ago

move pref window UI to suite/

Categories

(SeaMonkey :: Build Config, defect)

Other Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.0a1

People

(Reporter: kairo, Assigned: kairo)

References

Details

Attachments

(2 files, 1 obsolete file)

As part of the consolidation of SeaMonkey-specific code in the suite/ directory, the pref window UI needs to be moved. I'll take this as a start and proof-of-concept for the rest of the needed UI code moves.
Here's the part of this move that can be stated in a real patch, actually just jar.mn changes. I'll attach a cvsmoves file shortly that states which files need to be moved/copied. Basically, this is plain moving/copying of everything in prefwindow into the appropriate suite/ directories. I've tested this not only with the default SeaMonkey build, but also with a --enable-ui-locale=de build where I put German locale files into the appropriate dir in /l10n and that works fine as well :)
Attachment #217796 - Flags: superreview?(neil)
Attachment #217796 - Flags: review?(neil)
Attached patch cvs moves for prefwindow (obsolete) (deleted) — Splinter Review
Those are the files that need to be moved/copied - I'll open a cvs moves bug once I got reviews so I know the new locations of those files are correct.
Attachment #217798 - Flags: superreview?(neil)
Attachment #217798 - Flags: review?(neil)
Attached patch cvs moves for prefwindow, v2 (deleted) — Splinter Review
Thanks Neil for catching the bug in that cvsmoves list!
Attachment #217798 - Attachment is obsolete: true
Attachment #218119 - Flags: superreview?(neil)
Attachment #218119 - Flags: review?(neil)
Attachment #217798 - Flags: superreview?(neil)
Attachment #217798 - Flags: review?(neil)
Comment on attachment 217796 [details] [diff] [review] jar.mn changes for prefwindow move Well these obviously work.
Attachment #217796 - Flags: superreview?(neil)
Attachment #217796 - Flags: superreview+
Attachment #217796 - Flags: review?(neil)
Attachment #217796 - Flags: review+
Depends on: 333845
Comment on attachment 218119 [details] [diff] [review] cvs moves for prefwindow, v2 Hmm, my make-jars.pl no worky, so it's good that I visually reviewed attachment 217798 [details] [diff] [review] too!
Attachment #218119 - Flags: superreview?(neil)
Attachment #218119 - Flags: superreview+
Attachment #218119 - Flags: review?(neil)
Attachment #218119 - Flags: review+
Do you want to move stuff like pref-themes.xul that suiterunner can't use?
(In reply to comment #6) > Do you want to move stuff like pref-themes.xul that suiterunner can't use? suiterunner abandoning that came after my patch, and I really don't want to redo my patches for everything we realize that we may abandon in the future. There's no harm done with copying it now and then cvs removing it when we're really abandoning it.
The jar.mn changes are checked in, once I know that it builds, I'll also cvs remove the old files that aren't used any more now.
Everything looks OK, the old files in xpfe/ are gone now as well.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: