Open
Bug 47544
Opened 24 years ago
Updated 15 years ago
implement Insert | Line Break (turn it on)
Categories
(SeaMonkey :: Composer, defect, P3)
SeaMonkey
Composer
Tracking
(Not tracked)
NEW
Future
People
(Reporter: Brade, Unassigned)
References
Details
Attachments
(1 file)
4.22 KB,
patch
|
Details | Diff | Splinter Review |
We need to implement Insert | Line Break
Reporter | ||
Updated•24 years ago
|
Target Milestone: --- → Future
Updated•24 years ago
|
Status: NEW → ASSIGNED
Comment 1•24 years ago
|
||
Why? When you press Enter you get a line break anyway, *not* a new paragraph
(which is a really, really, *really* stupid feature IMNSHO).
Comment 3•24 years ago
|
||
I had bug 48678, so I guess this is mine.
Assignee: beppe → cmanske
Status: ASSIGNED → NEW
Updated•24 years ago
|
Status: NEW → ASSIGNED
Target Milestone: Future → mozilla0.9
Comment 4•24 years ago
|
||
Comment 5•24 years ago
|
||
Note: Bugs 47544 and 47545 are very similar. Stubs for nsInsertBreak and
nsInsertBreakAll already existed in ComposerCommands.js. Diff shows the
implementation added (not enough context in diff: nsInsertBreak is first in diff.)
Updated•24 years ago
|
Whiteboard: FIX IN HAND
Reporter | ||
Comment 6•24 years ago
|
||
r=brade (though I do *not* understand why we'd want this feature right now since
pressing return inserts a <br>; yes, I know that I filed this bug...)
My preference would be to *not* add this item to the Composer menu until pressing
return/enter does a different function (like inserting a paragraph).
Whiteboard: FIX IN HAND → FIX IN HAND; waiting for sr
Comment 7•24 years ago
|
||
I agree with Kathy, we don't really need this.
I'll checkin the suggested code but comment-out the menuitem for "Line Break"
Setting bug to future so we remember the issue after switching Enter key usage.
Target Milestone: mozilla0.9 → Future
Comment 9•24 years ago
|
||
I checked in the "commented out" XUL code as described.
"nsInsertBreak" command (in ComposerCommands.js) was implemented.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 10•24 years ago
|
||
setting milestone to match when it was checked in.
Target Milestone: Future → mozilla0.9
Comment 11•24 years ago
|
||
I do not see this implemented in 4/4 build on windows.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Updated•24 years ago
|
Summary: implement Insert | Line Break → implement Insert | Line Break (turn it on)
Target Milestone: mozilla0.9 → Future
Reporter | ||
Comment 12•24 years ago
|
||
move this bug to Future; we'll turn it on someday
Comment 14•23 years ago
|
||
removing myself from the cc list
Updated•21 years ago
|
QA Contact: sujay → stolenclover
Updated•20 years ago
|
Product: Browser → Seamonkey
Comment 15•17 years ago
|
||
No comment in six years (and a "removing myself" at that... the previous one was six months before). Is this still "Future" or shall we move it to Never i.e. WONTFIX? Or, on the contrary, target it for Sm2?
Comment 16•16 years ago
|
||
Resetting A+QA after long inactivity
Assignee: cmanske → nobody
Status: REOPENED → NEW
QA Contact: danielwang → composer
You need to log in
before you can comment on or make changes to this bug.
Description
•