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)

enhancement

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.
Attached file tinderbox2 tarball gzipped (deleted) —
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.
Some of the files do not have a license. Is it ok if I add the MPL to them?
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.
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.
Move to M20 for now.
Target Milestone: --- → M20
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; #
stealing from slamm
Assignee: slamm → mcafee
Keywords: patch
Summary: new version of Tinderbox → new version of Tinderbox: restructuring patch needs checkin
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
I have check in authorization now. We can close this bug as I checked in my code myself.
Status: RESOLVED → CLOSED
Blocks: 48679
Unsetting target milestone so we can delete the old (inappropriate) Mozilla target milestones from Webtools.
Target Milestone: M20 → ---
Status CLOSED is deprecated as per bug 169885
Status: CLOSED → REOPENED
Resolution: FIXED → ---
Re-resolving
Status: REOPENED → RESOLVED
Closed: 24 years ago21 years ago
Resolution: --- → FIXED
verifying former CLOSED bugs
Status: RESOLVED → VERIFIED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: