Bug 45606

Summary:

Neglect unlikely hyphenation opportunities

Product:

WebKit

Reporter:

mitz

Component:

Text

Assignee:

mitz

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	none
Patch	adele: review+

mitz

Reported 2010-09-11 18:45:09 PDT

Neglect unlikely hyphenation opportunities

Attachments
Patch (2.02 KB, patch) 2010-09-11 18:57 PDT, mitz	no flags	Details Formatted Diff Diff
Patch (2.02 KB, patch) 2010-09-11 19:01 PDT, mitz	adele: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

mitz

Comment 1 2010-09-11 18:57:57 PDT

Created attachment 67324 [details] Patch

mitz

Comment 2 2010-09-11 19:01:13 PDT

Created attachment 67325 [details] Patch

Adele Peterson

Comment 3 2010-09-12 15:12:44 PDT

Comment on attachment 67325 [details] Patch > + // If the maximum width available for the prefix before the hyphen is small, then it is very unlikely > + // that an hyphenation opportunity exists, so do bother to look for it. Did you mean "don't bother"?

mitz

Comment 4 2010-09-12 15:53:41 PDT

(In reply to comment #3) > (From update of attachment 67325 [details]) > > + // If the maximum width available for the prefix before the hyphen is small, then it is very unlikely > > + // that an hyphenation opportunity exists, so do bother to look for it. > Did you mean "don't bother"? Yes :)

mitz

Comment 5 2010-09-12 16:04:59 PDT

Committed <http://trac.webkit.org/changeset/67350>.

Note You need to log in before you can comment on or make changes to this bug.