Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(464)

Issue 1006363004: Introduce sky/framework/theme/typography.dart (Closed)

Created:
5 years, 9 months ago by abarth-chromium
Modified:
5 years, 9 months ago
Reviewers:
eseidel
CC:
mojo-reviews_chromium.org, ojan, qsr+mojo_chromium.org, rafaelw
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Introduce sky/framework/theme/typography.dart This CL adds typography information to the Sky theme. The values are from the Material Design spec. I've also applied these values to the stocks app and the various components. We're not geting precisely the right typography in some cases because of https://github.com/domokit/mojo/issues/65. R=eseidel@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/c9ce964bec1a210479577e89642a267777b88864

Patch Set 1 #

Patch Set 2 : Add comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+79 lines, -28 lines) Patch
M sky/examples/stocks-fn/stockrow.dart View 2 chunks +2 lines, -3 lines 0 comments Download
M sky/examples/stocks-fn/stocksapp.dart View 4 chunks +18 lines, -16 lines 0 comments Download
M sky/framework/components/action_bar.dart View 1 chunk +0 lines, -1 line 0 comments Download
M sky/framework/components/drawer_header.dart View 1 chunk +1 line, -3 lines 0 comments Download
M sky/framework/components/floating_action_button.dart View 1 chunk +0 lines, -1 line 0 comments Download
M sky/framework/components/menu_item.dart View 1 chunk +2 lines, -4 lines 0 comments Download
A sky/framework/theme/typography.dart View 1 1 chunk +56 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
abarth-chromium
5 years, 9 months ago (2015-03-16 22:35:40 UTC) #1
eseidel
lgtm
5 years, 9 months ago (2015-03-16 23:13:25 UTC) #2
abarth-chromium
5 years, 9 months ago (2015-03-17 03:53:30 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
c9ce964bec1a210479577e89642a267777b88864 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698