Skip to content

Commit

Permalink
Black text not visible #5291
Browse files Browse the repository at this point in the history
Issue #5291 black text not visible

Optimized code
  • Loading branch information
thejdeep committed Oct 14, 2014
1 parent 8445ff1 commit 6acc9c0
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/display/canvas.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@

// Minimal font size that would be used during canvas fillText operations.
var MIN_FONT_SIZE = 16;
// Maximum font size that would be used during canvas fillText operations.
var MAX_FONT_SIZE = 50;
var MAX_GROUP_SIZE = 4096;

var COMPILE_TYPE3_GLYPHS = true;
Expand Down Expand Up @@ -1226,8 +1228,9 @@ var CanvasGraphics = (function CanvasGraphicsClosure() {
// Keeping the font at minimal size and using the fontSizeScale to change
// the current transformation matrix before the fillText/strokeText.
// See https://bugzilla.mozilla.org/show_bug.cgi?id=726227
var browserFontSize = size >= MIN_FONT_SIZE ? size : MIN_FONT_SIZE;
this.current.fontSizeScale = browserFontSize !== MIN_FONT_SIZE ? 1.0 :
var browserFontSize = size < MIN_FONT_SIZE ? MIN_FONT_SIZE :
size > MAX_FONT_SIZE ? MAX_FONT_SIZE : size;
this.current.fontSizeScale = browserFontSize === size ? 1.0 :
size / MIN_FONT_SIZE;

var rule = italic + ' ' + bold + ' ' + browserFontSize + 'px ' + typeface;
Expand Down

0 comments on commit 6acc9c0

Please sign in to comment.