Closed Bug 540788 Opened 15 years ago Closed 15 years ago

disable text hinting on mobile (windows mobile + qt)

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
ARM
Windows Mobile 6 Professional
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: blassey, Assigned: dougt)

References

Details

Attachments

(3 files, 1 obsolete file)

patch v.1
1.04 KB, patch
Details | Diff | Splinter Review
patch v.2
1.19 KB, patch
dougt
: review-
Details | Diff | Splinter Review
finishing move
1.13 KB, patch
vlad
: review+
Details | Diff | Splinter Review
+++ This bug was initially created as a clone of Bug #470440 +++ Load any page into fennec that uses standard variable width fonts and note the kerning (space between the letters). You'll see that letters are oddly spaced. For example, on bugzilla.mozilla.org, you can note that the a's have too much space after them and the t's have too little (check out t's at the ends of words). You can see the same effect on cnn.com. If you zoom in, the kerning is repaired and looks normal. But at the default zoom level, this makes pages hard to read. 1. Go to site (bugzilla.mozilla.org, cnn.com, etc) == Actual == Kerning is messed up. == Expected == Kerning is displayed properly. Note that if you zoom in, the kerning is displayed properly, but returning to the default zoom level, the incorrect spacing returns. I'll try to get some screen shots. Marking blocking ? because if you can't easily read sites, fennec isn't useful.
Attached patch patch (obsolete) — — Splinter Review
Assignee: nobody → bugmail
Attachment #422490 - Flags: review?(pavlov)
Attached patch patch v.1 — — Splinter Review
I think you want to do this sort of thing outside of cairo.
Assignee: bugmail → mozbugz
Attachment #422490 - Attachment is obsolete: true
Attachment #425195 - Flags: review?
Attachment #422490 - Flags: review?(pavlov)
clearing status1.9.2 flag.
status1.9.2: final-fixed → ---
OS: Windows Mobile 6 Professional → All
Summary: text spacing is bad in Fennec at non-1:1 zoom levels on Windows Mobile → disable text hinting on mobile (windows mobile + qt)
Attachment #425195 - Flags: review? → review+
Comment on attachment 425195 [details] [diff] [review] patch v.1 still seeing some issues. holding on this one for a sec.
Attachment #425195 - Flags: review+
Attached patch patch v.2 — — Splinter Review
doug, try this patch in addition to the first. Either one alone seems to do the trick for me on winmo, but perhaps qt will need both.
Attachment #425207 - Flags: review?(mozbugz)
Attachment #425207 - Attachment is patch: true
Attachment #425207 - Attachment mime type: application/octet-stream → text/plain
Attachment #425207 - Flags: review?(mozbugz) → review-
tracking-fennec: 1.0+ → ---
tracking-fennec: --- → ?
Attached patch finishing move — — Splinter Review
with our patches combined, we can defeat this bug once and for all.
Attachment #440671 - Flags: review?
Attachment #440671 - Flags: review? → review?(dougt)
http://hg.mozilla.org/mozilla-central/rev/b218850ce72e
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Component: Windows Mobile → General
OS: All → Windows Mobile 6 Professional
QA Contact: mobile-windows → general
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: