Closed
Bug 13442
Opened 25 years ago
Closed 25 years ago
Copying a space followed by a line break yields two spaces
Categories
(Core :: DOM: Serializers, defect, P3)
Core
DOM: Serializers
Tracking
()
VERIFIED
FIXED
M13
People
(Reporter: elig, Assigned: akkzilla)
References
Details
(Whiteboard: [COPY/PASTE])
Attachments
(1 file)
(deleted),
text/html
|
Details |
<broken out of 12551>
* TITLE/SUMMARY
Copying a space followed by a line break yields two spaces
* STEPS TO REPRODUCE
0) Launch Apprunner
1) Using the attached test case, copy the entire "CodeStock Web Developer
Seminars" paragraph.
2) View the Clipboard
* RESULT
- What happened
When you copy text from a web page whose source contains a word, followed by a
space, followed by a line break (not a <BR>; an actual line break), followed by a
word, two spaces appear between the words.
In this case, you'll see "web content with" (two spaces b/w 'content' & 'with'):
The source is:
<--->
web content
with
<--->
(with a space after the word "content")
- What was expected
One space between words.
* REGRESSION
- Occurs On
Mac OS Apprunner (19990808 optimized build)
Win32 Apprunner (19990808 optimized build [NT 4, Service Pack 3])
- Can't Reproduce On
Linux Apprunner (1999082316 [busted] build date; probably this morning's
build.) Copy doesn't function.
* CONFIGURATIONS TESTED
- [Mac] Beige Power Mac G3 (266 MHz PowerPC 750), 96 MB RAM (VM on; 1 MB of VM
used), 1024x768 (Thousands of Colors), Mac OS 8.6
- [Win32] Vectra VL (233 MHz P2), 96 MB RAM, 800x600 (True Color), NT 4.0 SP3.
- [Linux] Vectra VL (266 MHz P2), 96 MB RAM. Red Hat Linux 6.0 (GNOME).
Reporter | ||
Updated•25 years ago
|
QA Contact: sujay → elig
Reporter | ||
Comment 1•25 years ago
|
||
[QA Assigning to self.]
Reporter | ||
Comment 2•25 years ago
|
||
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M15
Assignee | ||
Comment 3•25 years ago
|
||
This is actually happening because the parser is calling us multiple times for
adjacent whitespace nodes. I'm going to try to work around it in the sink
stream, but cc'ing parser people since it doesn't seem like it ought to be doing
this.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M15 → M12
Assignee | ||
Comment 4•25 years ago
|
||
M12. Copy/paste of basic plaintext is important for beta.
Assignee | ||
Comment 5•25 years ago
|
||
Changing Component to Output.
Updated•25 years ago
|
Summary: Copying a space followed by a line break yields two spaces → [BETA] Copying a space followed by a line break yields two spaces
Whiteboard: [COPY/PASTE]
Target Milestone: M12 → M13
Comment 6•25 years ago
|
||
I'm moving this to M13, it's a must have for beta
Updated•25 years ago
|
Summary: [BETA] Copying a space followed by a line break yields two spaces → Copying a space followed by a line break yields two spaces
Whiteboard: [COPY/PASTE] → [COPY/PASTE FEATURE]
Comment 7•25 years ago
|
||
updating summary fields
Summary: Copying a space followed by a line break yields two spaces → [FEATURE]Copying a space followed by a line break yields two spaces
Updated•25 years ago
|
Summary: [FEATURE]Copying a space followed by a line break yields two spaces → Copying a space followed by a line break yields two spaces
Whiteboard: [COPY/PASTE FEATURE] → [COPY/PASTE]
Comment 9•25 years ago
|
||
the information in the whiteboard summary is for me to track what folks are
working on, just to keep things simple, I won't put the word feature in anymore
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•25 years ago
|
||
Daniel Brattell's changes should have fixed this, and have as far as I can tell.
Marking fixed ... please reopen with a new test case if you're still seeing it.
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 11•25 years ago
|
||
Verified fixed using 2000011208 Mac OS smoke-tested build, as well as the
2000011108 Win32/Linux smoke-tested builds.
Test case now functions, as did a variant checked on Mac OS of placing the the
line break before the space (and in between two spaces).
Comment 12•25 years ago
|
||
Bulk move of all "Output" component bugs to new "DOM to Test Conversion"
component. Output will be deleted as a component.
You need to log in
before you can comment on or make changes to this bug.
Description
•