Closed
Bug 137253
Opened 23 years ago
Closed 22 years ago
Rewrap: Don't touch long words
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
VERIFIED
FIXED
mozilla1.1final
People
(Reporter: 3.14, Assigned: akkzilla)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Brade
:
review+
kinmoz
:
superreview+
|
Details | Diff | Splinter Review |
This is not limited to a particular Mozilla version.
Compare bug 124099 (Akana wants this one -> Assigning)
Rewrap (plain text editor) breaks long words like URLs.
Expected: Long words are not touched.
Rationale: It is not possible for the receiver to unwrap broken URLs or other
long words to recreate paragraphs.
Solution: Do line breaks only at white space.
pi
Assignee | ||
Comment 1•23 years ago
|
||
This should be easy to do, and I agree it's evil that we break urls when we rewrap.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0.1
Assignee | ||
Comment 2•23 years ago
|
||
Here's a fix. Sorry this took so long ... I kept getting ideas about how I
could leverage the wrapping code in the serializers and avoid us having to have
two separate implementations of this logic, but none of the rewrites I tried
ever worked out.
This patch is a simple fix that addresses only the problem at hand. It should
be safe, but to be sure we should test the cases in bug 121046 and bug 112139,
which touched code near this change in order to cure two different hangs.
Unfortunately the mail compose window doesn't come up in today's build, so that
test will have to wait. But if someone could review it in the meantime, that
would get us started ...
Comment 3•23 years ago
|
||
Comment on attachment 89999 [details] [diff] [review]
A fix
r=brade (assuming the bugs mentioned above have been tested with satisfactory
results)
Attachment #89999 -
Flags: review+
Assignee | ||
Comment 4•23 years ago
|
||
I tested the attachments in those two bugs; no regression of the problems
reported there. I think we're good to go.
Kin, could you sr this patch, please?
Attachment #89999 -
Flags: superreview+
Reporter | ||
Comment 6•22 years ago
|
||
Did you already check the patch in? If so when?
pi
Assignee | ||
Comment 7•22 years ago
|
||
The tree has been closed since the patch was sr'ed. I have requested driver
approval for 1.1.
Target Milestone: mozilla1.0.1 → mozilla1.1final
Assignee | ||
Comment 8•22 years ago
|
||
Fixed in the trunk. I haven't gotten a reply about the 1.1 branch; pester
drivers@mozilla.org if you want it in.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 9•22 years ago
|
||
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1b) Gecko/2002080820
Fix WFM.
pi
Comment 10•22 years ago
|
||
vrfy'd fixed with 2003.04.10 comm bits on linux rh8.0.
Status: RESOLVED → VERIFIED
Comment 11•22 years ago
|
||
I'm not sure whether the fix for this bug was what now makes Mozilla so
inconvenient when typing text with long URLs, but if it does, then this is
actually a regression.
For example, pasting the URL below in the "Additional Comments" field (where I'm
writing this comment), causes Mozilla to display a horizontal scroll bar, while
under IE such long URLs flow to the next line.
http://bugzilla.mozilla.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED
In my opinion, using word wrap is easier for users to handle.
Prog.
You need to log in
before you can comment on or make changes to this bug.
Description
•