Closed
Bug 38305
Opened 25 years ago
Closed 21 years ago
new version of Tinderbox: restructuring patch needs checkin
Categories
(Webtools Graveyard :: Tinderbox, enhancement, P3)
Webtools Graveyard
Tinderbox
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: kestes, Assigned: mcafee)
References
Details
Attachments
(5 files)
I am going to attach a tarball with a new version of tinderbox.
Please check it into the tree (tinderbox2?) and lets discuss what
to do with it.
Comment 2•25 years ago
|
||
I asked staff@mozilla.org if it would be ok to plop this down in a new
directory. Hopefully I can check thin in somewhere today.
Comment 3•25 years ago
|
||
Some of the files do not have a license. Is it ok if I add the MPL to them?
Comment 5•25 years ago
|
||
The best way to get feedback is to have the main tinderbox page link to a beta
of tinderbox 2. This way everyone will see the link and be curious. If you put
this bug number on the Tinderbox 2 page, then people will easily be able to
provide feedback through bugzilla.
Comment 6•25 years ago
|
||
I checked in the last tarball to
mozilla/webtools/tinderbox2
I considered putting on a branch tag under mozilla/webtools/tinderbox, but
adding a new directory was easier. If we do eventually want to replace the
current tinderbox code, we can move the code there when the time comes.
Index: MajorCoolWindow.pm
===================================================================
RCS file: ./tinderbox2/src/lib/HTMLPopUp/MajorCoolWindow.pm,v
retrieving revision 1.8
diff -c -r1.8 MajorCoolWindow.pm
*** MajorCoolWindow.pm 2000/06/19 18:32:10 1.8
--- MajorCoolWindow.pm 2000/06/19 20:04:21
***************
*** 370,376 ****
# set the defaults
$args{'windowtitle'} = $args{'windowtitle'} ||
! $HTML::DEFAULT_POPUP_TITLE;
# These characters inside the popupwindow will confuse my popup
--- 370,376 ----
# set the defaults
$args{'windowtitle'} = $args{'windowtitle'} ||
! $HTMLPopUp::DEFAULT_POPUP_TITLE;
# These characters inside the popupwindow will confuse my popup
***************
*** 391,400 ****
# determine what a HTML row is.
$args{'windowheight'} = $args{'windowheight'} ||
! $HTML::DEFAULT_POPUP_HEIGHT;
$args{'windowwidth'} = $args{'windowwidth'} ||
! $HTML::DEFAULT_POPUP_WIDTH;
#
--- 391,400 ----
# determine what a HTML row is.
$args{'windowheight'} = $args{'windowheight'} ||
! $HTMLPopUp::DEFAULT_POPUP_HEIGHT;
$args{'windowwidth'} = $args{'windowwidth'} ||
! $HTMLPopUp::DEFAULT_POPUP_WIDTH;
#
Assignee | ||
Updated•24 years ago
|
Keywords: patch
Summary: new version of Tinderbox → new version of Tinderbox: restructuring patch needs checkin
Assignee | ||
Comment 11•24 years ago
|
||
I changed my mind and did as Ken suggested.
Nuked the old tinderbox2 tree, dumped in new stuff,
here we go again.
Fixed. QA: Just verify this, this was a code-thing.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 12•24 years ago
|
||
Reporter | ||
Comment 13•24 years ago
|
||
Reporter | ||
Comment 14•24 years ago
|
||
I have check in authorization now. We can close this bug as I checked
in my code myself.
Status: RESOLVED → CLOSED
Comment 15•24 years ago
|
||
Unsetting target milestone so we can delete the old (inappropriate)
Mozilla target milestones from Webtools.
Target Milestone: M20 → ---
Comment 16•21 years ago
|
||
Status CLOSED is deprecated as per bug 169885
Status: CLOSED → REOPENED
Resolution: FIXED → ---
Comment 17•21 years ago
|
||
Re-resolving
Status: REOPENED → RESOLVED
Closed: 24 years ago → 21 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•