Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(42)

Issue 5498059: fix vertical text (mostly) on Leopard

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 4 months ago by caryclark1
Modified:
12 years, 4 months ago
Reviewers:
reed1
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Leopard and Lion can share most vertical code because the SnowLeopard alone fails to scale the vertical data by the font matrix. Leopard requires a few other tweaks; see the comments in the code. Added another vertical text case to check monospaced fonts and centering.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+118 lines, -12 lines) Patch
A gm/verttext2.cpp View 1 chunk +92 lines, -0 lines 0 comments Download
M gyp/gmslides.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M src/ports/SkFontHost_mac_coretext.cpp View 5 chunks +25 lines, -12 lines 0 comments Download

Messages

Total messages: 2
caryclark1
12 years, 4 months ago (2011-12-21 17:30:33 UTC) #1
reed1
12 years, 4 months ago (2011-12-21 19:43:12 UTC) #2
lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b