Last Comment Bug 139335 - QA and Debug menu items should be removed for the 1.0 final release
: QA and Debug menu items should be removed for the 1.0 final release
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: General (show other bugs)
: Trunk
: All All
: -- major (vote)
: ---
Assigned To: Daniel (Leaf) Nunes
: Zach Lipton [:zach]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2002-04-22 17:18 PDT by Daniel (Leaf) Nunes
Modified: 2011-08-05 21:17 PDT (History)
22 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
this is from mozilla/build/package/rpm/SOURCES/mozilla-navigator-overlay-menu.patch, thanks blizzard :) (10.46 KB, patch)
2002-04-22 17:56 PDT, Daniel (Leaf) Nunes
rjesup: review+
hewitt: superreview+
Details | Diff | Review
editor menu surgery (1.54 KB, patch)
2002-04-22 17:58 PDT, Daniel (Leaf) Nunes
rjesup: review+
hewitt: superreview+
Details | Diff | Review
the debug prefs in the preferences ui; this is more optional, but a possibility for the final release (1.15 KB, patch)
2002-04-22 18:02 PDT, Daniel (Leaf) Nunes
rjesup: review+
hewitt: superreview+
chofmann: approval+
Details | Diff | Review

Description Daniel (Leaf) Nunes 2002-04-22 17:18:02 PDT
The QA and Debug menu items ought to be removed for the 1.0 final release;
assigning to User Interface Design because this is pretty visible UI, but the
patch isn't rocket science.
Comment 1 Daniel (Leaf) Nunes 2002-04-22 17:19:56 PDT
taking.
Comment 2 Daniel (Leaf) Nunes 2002-04-22 17:56:02 PDT
Created attachment 80499 [details] [diff] [review]
this is from mozilla/build/package/rpm/SOURCES/mozilla-navigator-overlay-menu.patch, thanks blizzard :)

this removes debug and qa from navigator; i'll be stealing blizzard's other
patches for editor and mailnews presently.
Comment 3 Daniel (Leaf) Nunes 2002-04-22 17:58:36 PDT
Created attachment 80500 [details] [diff] [review]
editor menu surgery
Comment 4 Daniel (Leaf) Nunes 2002-04-22 18:02:41 PDT
Created attachment 80501 [details] [diff] [review]
the debug prefs in the preferences ui; this is more optional, but a possibility for the final release
Comment 5 timeless 2002-04-22 18:57:20 PDT
asa??
Comment 6 Zach Lipton [:zach] 2002-04-22 19:30:28 PDT
cc'ing mpt on this to keep him in the loop (sorry to comment on a cc)
Comment 7 Dawn Endico 2002-04-30 18:42:26 PDT
The qa and debug menus should be removed from the mozilla1.0 release branch.

The 1.0 release branch is supposed to something from which people can pull
to make a releasable product and don't want others to include our debug or
qa menus. I don't want the menus even in our own mozilla 1.0 builds.

I'm not suggesting that these menus be removed from daily builds or
from regular milestones, only from the actual blessed 1.0 builds. These
builds are going to get a LOT of press. RC1 already had an article in
Time magazine. We want these people to use mozilla for the talkback data
but only a tiny fraction of them would find anything in the debug or qa
menus useful. Let them get a nightly build.

Out of curiosity a lot of time magazine readers will hit the 'bugs
filed today' and 'cvs checkins today' menu items. We cannot handle the server
load of a million curious users. The start page is the proper place to 
evangelize new recruits and people who really do want to debug should be
using other builds.
Comment 8 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2002-04-30 18:48:46 PDT
Does the commercial tree remove these by overlay?  If so, I'm assuming leaf will
make the necessary commercial changes (if any -- I'm not sure if any really are).
Comment 9 [:jesup] on pto until 2016/7/5 Randell Jesup 2002-04-30 19:01:00 PDT
We need r/sr for these; please beat up the right people for r/sr.  We need to
disable/remove them from 1.0 official builds.  Adding dependency on RC2 Not Suck
Comment 10 Asa Dotzler [:asa] 2002-04-30 23:43:57 PDT
This makes me sad. I can see the problem with the bugzilla and bonsai queries in
the QA menu leading to performance problems in our tool suite but the rest of
menus serve a good purpose of providing easy access to the tools for testers and
don't cost us. Our releases/index.html says "We make binary versions of Mozilla
available for testing purposes only!" and removing access points for testing our
test binaries seems wrong. 
Comment 11 Dawn Endico 2002-05-01 00:11:00 PDT
Yes, but that doesn't mean that everyone has to do each test.
With a mega-release like 1.0 we should be happy to get lots of
talkback data and not annoy people with a lot of extra crap
which makes them send whiney or desperate or desperate (xul
boxes) mail to webmaster.

The people who aren't annoyed by the 'extra crap' are already
using regular builds or can be recruited by the start page.
Remember, this is only for 1.0.
Comment 12 Ben Bucksch (:BenB) 2002-05-01 22:48:07 PDT
If you remove the stuff, commenting it out should be enough.
That's what's usually been done on the Netscape branches, e.h., the
point-release branches. (Although it annoys me, as I always comment the stuff
out, too, and I get merge conflicts all the time.)
Comment 13 timeless 2002-05-01 23:53:54 PDT
i think a dynamic overlay would be more correct. then releases could not make 
the reference for it. although there's a minor perf hit..
Comment 14 Ben Goodger (use ben at mozilla dot org for email) 2002-05-02 03:33:33 PDT
r=ben on these patches. Commercial hides these with a stylesheet, but it won't
be negatively affected as the removal of the items will cause the rules in the
stylesheets not to match, which has no effect. 
Comment 15 Asa Dotzler [:asa] 2002-05-02 08:47:44 PDT
who can sr= here? Time is short. We need quick turnaoround. Blake? Hewitt?
Comment 16 Joe Hewitt (gone) 2002-05-02 13:19:28 PDT
Comment on attachment 80501 [details] [diff] [review]
the debug prefs in the preferences ui; this is more optional, but a possibility for the final release

sr=hewitt
Comment 17 Joe Hewitt (gone) 2002-05-02 13:20:08 PDT
Comment on attachment 80500 [details] [diff] [review]
editor menu surgery

sr=hewitt
Comment 18 Joe Hewitt (gone) 2002-05-02 13:24:42 PDT
Comment on attachment 80499 [details] [diff] [review]
this is from mozilla/build/package/rpm/SOURCES/mozilla-navigator-overlay-menu.patch, thanks blizzard :)

sr=hewitt
Comment 19 Brian McGroarty 2002-05-02 15:40:19 PDT
would rather see this as a command-line option or similar, defaulting to off in
release. fewer functionality changes between debug/release are good.
Comment 20 [:jesup] on pto until 2016/7/5 Randell Jesup 2002-05-02 22:20:54 PDT
Comment on attachment 80499 [details] [diff] [review]
this is from mozilla/build/package/rpm/SOURCES/mozilla-navigator-overlay-menu.patch, thanks blizzard :)

marking r=ben on patches per his comment
Comment 21 Mark Bitterling 2002-05-03 00:28:49 PDT
I have mixed emotions about the removal of the Debug and QA menus, but oh well.

However, I do have a question.

If the decision is to remove the menus, then do the various sample and test
files (ie the samples directory) also get removed?  Is there any reason to
include them when the menus are gone?  I realize it is only about 1MB or so
uncompressed, but if millions of downloads are anticipated...
Comment 22 Christian :Biesinger (don't email me, ping me on IRC) 2002-05-03 07:00:10 PDT
>If you remove the stuff, commenting it out should be enough.

Why not remove it, leaving it in serves no purpose.
And why not use a stylesheet as somebody suggested here?


Though I personally am against this, and so is Asa, apparently. Especially the
QA menu contains useful links, for me at least, and for other people interested
in testing Mozilla.
Comment 23 Andrew Hagen 2002-05-03 07:14:37 PDT
There are going to be a bunch of novice users who download 1.0. They won't be
able to contribute to QA anyway. Thus, I'd say take these menus out for the 1.0
release only.
Comment 24 Pete Boyd 2002-05-04 00:47:45 PDT
is it proposed to remove Debug and QA menus from 1.0.x or just 1.0.0? reading
this bug, it appears to just be 1.0.0. and if its done for 1.0.x, what of 1.1.0
once the precident is set?
Comment 25 Henri Sivonen (:hsivonen) 2002-05-04 02:08:31 PDT
asa, cbiesinger: 
The Debug and QA menus contain bookmark-like items. IMO, those items would fit
better in the default bookmarks. (I think it makes sense to remove the Debug and
QA menus.)
Comment 26 Asa Dotzler [:asa] 2002-05-09 14:07:32 PDT
last minute changes ->RC3.
Comment 27 Håkan Waara 2002-05-12 14:04:51 PDT
Do we have an official a=drivers here, and if so, can someone go ahead and check
the patch into the branch?
Comment 28 Christian :Biesinger (don't email me, ping me on IRC) 2002-05-12 14:08:37 PDT
hwaara, read comment 11... only the exact 1.0 release will have this removed, no
other build.
Comment 29 chris hofmann 2002-05-14 08:12:04 PDT
Comment on attachment 80501 [details] [diff] [review]
the debug prefs in the preferences ui; this is more optional, but a possibility for the final release

a=chofmann, leaf do the deed and lets get this off the list. thx
Comment 30 Daniel (Leaf) Nunes 2002-05-14 17:49:44 PDT
menus whacked. editor menus had been commented out already (i commented out one
last reference that brade didn't comment out).
Comment 31 Asa Dotzler [:asa] 2002-05-16 02:27:17 PDT
housekeeping: adding branch resolution keyword. 

Note You need to log in before you can comment on or make changes to this bug.