Closed
Bug 17192
Opened 25 years ago
Closed 25 years ago
s in apostrophe s ('s) is dropped on to next line; should never be split
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
FIXED
M15
People
(Reporter: ekrock, Assigned: ftang)
References
()
Details
(Whiteboard: have fix in local tree)
1999102508 mozilla.exe on WinNT 4.0 SP3.
To repro:
1) open http://www.abcnews.go.com/sections/us/DailyNews/modelseggs991023.htm
2) view the paragraph
"Nancy Etcoff, the Cambridge-based author of it’s natural to consider how your
mate’s attractiveness will be reflected in your children. But she said such
direct marketing of beauty is a false promise."
Expected:
it's will never be line-broken after the '
Actual
it's is broken after the ' and the s appears on the next line
Reporter | ||
Updated•25 years ago
|
Reporter | ||
Comment 1•25 years ago
|
||
Corrected URL to have trailing "l" in ".html". The correct URL is
http://www.abcnews.go.com/sections/us/DailyNews/modelseggs991023.htm
The omission was caused by Location bar URL copy-and-paste bug I'm filing
separately.
Change the status whiteboard to indicate that I need a testcase before
proceeding.
Reporter | ||
Updated•25 years ago
|
Whiteboard: NEEDTESTCASE
Target Milestone: M19 → M14
Reporter | ||
Comment 3•25 years ago
|
||
Not to be difficult, but is a simplified test case truly necessary here?
Reason: in text layout algorithms, characters can be grouped as follows,
loosely speaking:
1) characters that define a word break (e.g. space) at which a line break is OK
2) characters within a word after which a hyphenation for line break is OK
3) characters within a word after which neither a hyphen nor a line break are
allowed
The apostrophe clearly falls into group #3. There should never be a line break
or a hyphen after an apostrophe (unless the apostrophe is followed by a space,
in which case breaking the line is OK, but that's a different case).
If it's possible for a line break to be inserted between the apostrophe and the
next non-space character (and the URL demonstrates that it is) the line layout
algorithm is clearly broken and the handling of the apostrophe needs to be
changed.
Removing the "NEEDTESTCASE" statement from Status Whiteboard as making the
Status Whiteboard field non-empty *removes* the bug from the BugAThon list of
bugs that need test cases. If you're trying to get external creation of a test
case, Status Whiteboard field should be empty and the bug will be on the list.
If a bug needs no test case, it's explicitly removed from the list by setting
the field to [NOTESTCASENEEDED].
Changing from M19 to M14 as this is a fairly basic text line layout correctness
issue.
Don't mess with my priorities. If you want to talk about, talk about it.
Comment 5•25 years ago
|
||
I think this bug is invalid, but the i18n folks would know for sure. The
apostrophe used in the document is an ’ (sic) character entity, which is,
I believe, the fake-windows-equivalent for an ’ (left single quote). I
would think that quotation marks are defined in Unicode as separating
punctuation marks, but I'm not sure. (Either way, the author should have used
an apostrophe for an apostrophe. However, there is a bug with that too: bug
15909.)
Frank - you should investigate this and either close it or fix it - the
implication is that the i18n line breaker is not doing the correct thing for
english characters.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M19 → M12
Assignee | ||
Comment 7•25 years ago
|
||
move to M12. It won't be too diffuclt to fix it if that that is the right thing to do. Need to think hard about this first. Also, need to check with JIS
x4501... If JISx4501 didin't mention these characters, it probably safe to fix it.
Assignee | ||
Updated•25 years ago
|
Hardware: PC → All
Assignee | ||
Updated•25 years ago
|
Target Milestone: M12 → M15
Assignee | ||
Comment 8•25 years ago
|
||
post beta per ftang/bobj chat. Should be easy to fix.
Assignee | ||
Updated•25 years ago
|
Whiteboard: have fix in local tree
Assignee | ||
Comment 9•25 years ago
|
||
easy to fix. Add 0x2019 into contextual analaysis when try to line break. Fix in
local tree.
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•25 years ago
|
||
fix check in
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 11•25 years ago
|
||
Fixed in the Dec 22 build.
You need to log in
before you can comment on or make changes to this bug.
Description
•