Closed
Bug 53994
Opened 25 years ago
Closed 25 years ago
Remove/Hide Composer Debug menu for nsbeta3/rtm
Categories
(Core :: DOM: Editor, defect, P2)
Core
DOM: Editor
Tracking
()
VERIFIED
FIXED
People
(Reporter: jrgmorrison, Assigned: Brade)
References
Details
(Whiteboard: [nsbeta3-][rtm++]PATCH ATTACHED)
Attachments
(1 file)
|
528 bytes,
patch
|
Details | Diff | Splinter Review |
I don't know if you have this bug already, but a search of bugzilla and
bugscape did not turn one up.
Composer has a debug menu which should be removed (commented out) in the
branch builds (or however you want to deal with it).
The equivalent Mail debug menu has been removed (bug 45159) and browser Debug
and QA menus are in the process of being removed (bug 52215).
|
Reporter | |
Comment 1•25 years ago
|
||
Adding decorative keywords nsbeta3,rtm (collect them all!).
|
|
||
Comment 2•25 years ago
|
||
yes, the menu item should be removed, would be nice to have a hook in place that
would drop it back in though.
Assignee: beppe → cmanske
Priority: P3 → P2
Whiteboard: [nsbeta3+]
Target Milestone: --- → M19
|
|
||
Comment 3•25 years ago
|
||
I'll just put just put 'collapse="true"' so we could add a pref to have it
show/hide more easily in next version.
If we are concerned about UI bloat, we could completely remove it just before
RTM.
Status: NEW → ASSIGNED
|
|
||
Comment 4•25 years ago
|
||
Reassigning to Kathy as I am going on sabatical
Assignee: cmanske → brade
Status: ASSIGNED → NEW
|
|
||
Comment 5•25 years ago
|
||
Not going to hold PR3 for this, but agree it must be fixed for RTM
Whiteboard: [nsbeta3+] → [nsbeta3-]
|
|
||
Comment 6•25 years ago
|
||
we can't go out with the debug menu visible
Kathy, please include the required information per the rtm checkin rules
Whiteboard: [nsbeta3-] → [nsbeta3-][rtm+ NEED INFO]
|
|
||
Comment 7•25 years ago
|
||
PDT agrees [rtm need info] until patch and code reviews are available.
Whiteboard: [nsbeta3-][rtm+ NEED INFO] → [nsbeta3-][rtm NEED INFO]
|
|
||
Updated•25 years ago
|
Whiteboard: [nsbeta3-][rtm NEED INFO] → [nsbeta3-][rtm NEED INFO]FIX IN HAND
|
Assignee | |
Comment 8•25 years ago
|
||
I am about to attach the patch for this bug.
It was submitted by Ryan Cassin (rcassin@supernova.org).
I have reviewed and tested it on Linux and Macintosh (Ryan has Windows) on the
branch.
Simon--please add sr=
Note: This fix is ONLY for the branch (it should NOT land on the trunk).
Status: NEW → ASSIGNED
Keywords: patch
Whiteboard: [nsbeta3-][rtm NEED INFO]FIX IN HAND → [nsbeta3-][rtm NEED INFO]PATCH ATTACHED
|
|
Assignee | |
Comment 9•25 years ago
|
||
|
|
||
Comment 10•25 years ago
|
||
|
|
Assignee | |
Comment 11•25 years ago
|
||
patch submitted by rcassin@supernova.org
patch tested/reviewed by module owner brade
patch super reviewed by kin
update status whiteboard; send off to pdt
Whiteboard: [nsbeta3-][rtm NEED INFO]PATCH ATTACHED → [nsbeta3-][rtm+]PATCH ATTACHED
|
|
||
Comment 12•25 years ago
|
||
rtm++
Whiteboard: [nsbeta3-][rtm+]PATCH ATTACHED → [nsbeta3-][rtm++]PATCH ATTACHED
|
||
Comment 13•25 years ago
|
||
r=sfraser
|
|
Assignee | |
Comment 14•25 years ago
|
||
fix checked into branch only (not this is a branch only fix!)
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
||
Comment 15•25 years ago
|
||
*** Bug 55741 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•