Closed
Bug 333383
Opened 19 years ago
Closed 19 years ago
move pref window UI to suite/
Categories
(SeaMonkey :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0a1
People
(Reporter: kairo, Assigned: kairo)
References
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•19 years ago
|
||
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)
Assignee | ||
Comment 2•19 years ago
|
||
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)
Assignee | ||
Comment 3•19 years ago
|
||
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 4•19 years ago
|
||
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+
Comment 5•19 years ago
|
||
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+
Comment 6•19 years ago
|
||
Do you want to move stuff like pref-themes.xul that suiterunner can't use?
Assignee | ||
Comment 7•19 years ago
|
||
(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.
Assignee | ||
Comment 8•19 years ago
|
||
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.
Assignee | ||
Comment 9•19 years ago
|
||
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.
Description
•