|
|||||
Issue descriptionIn https://chromiumcodereview.appspot.com/11359020, the print preview was turned to use a webkit and the html file for that uses a hardcode font-family, "sans". At minimum, it has to be 'sans-serif' (a CSS generic font family we use for the UI). Better would be to use IDS_WEB_FONT_FAMILY instead of hard-coding the family. Nov 14 2012, Project MemberThe following revision refers to this bug: http://src.chromium.org/viewvc/chrome?view=rev&revision=167608 ------------------------------------------------------------------------ r167608 | vitalybuka@chromium.org | 2012-11-14T05:32:01.221628Z Changed paths: M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/print_preview/print_preview_page.html?r1=167608&r2=167607&pathrev=167608 Use 'sans-serif' instead of 'sans' to print headers and footers. BUG= 160830 , 152893 Review URL: https://chromiumcodereview.appspot.com/11275302 ------------------------------------------------------------------------ Nov 14 2012,
Nov 17 2012, Project Member
The following revision refers to this bug: http://src.chromium.org/viewvc/chrome?view=rev&revision=168380 ------------------------------------------------------------------------ r168380 | vitalybuka@chromium.org | 2012-11-17T04:45:12.689359Z Changed paths: M http://src.chromium.org/viewvc/chrome/branches/1312/src/chrome/browser/resources/print_preview/print_preview_page.html?r1=168380&r2=168379&pathrev=168380 Merge 167608 - Use 'sans-serif' instead of 'sans' to print headers and footers. BUG= 160830 , 152893 Review URL: https://chromiumcodereview.appspot.com/11275302 TBR=vitalybuka@chromium.org Review URL: https://codereview.chromium.org/11348114 ------------------------------------------------------------------------ Mar 10 2013, Project Member
Mar 20 2013, Project Member
|
|||||
►
Sign in to add a comment |
Comment 1 by vitalyb...@chromium.org, Nov 13 2012