Align text using advance width, not glyph width #253

Activity

1. pelson commented on Jun 18, 2012

   

   pelson

   on Jun 18, 2012

   Member

   More actions

   @mdboom: I know you've had font dealings lately. Do you think this is this still an issue?
2. efiring commented on Jul 22, 2013

   

   efiring

   on Jul 22, 2013

   Member

   More actions

   Stan West, the OP, provided the image and test code below:

   

   import matplotlib.pyplot as plt
   
   fontdictvert = {'family': 'Georgia', 'size': 18.0}
       # or any font with old-style/lowercase/non-aligning figures (numerals)
   fontdicthorz = {'family': 'Arial', 'size': 18.0}
       # or any font with tabular (not proportional) figures and a narrow
       # glyph for digit one
   
   plt.figure()
   plt.subplot(1, 1, 1)
   plt.xlim(0, 1)
   plt.ylim(-0.301, 0.301)
   plt.xticks(**fontdictvert)
   plt.yticks(**fontdicthorz)
   plt.text(0.2, 0, 'left-aligned\n90123\n01234\n12345\n23456\n34567',
            ha='left', va='center', **fontdicthorz)
   plt.text(0.8, 0, 'right-aligned\n90123\n89012\n78901\n67890\n56789',
            ha='right', va='center', **fontdicthorz)
   
   plt.show()
3. efiring commented on Jul 22, 2013

   

   efiring

   on Jul 22, 2013

   Member

   More actions

   @pelson, @mdboom, Stan reports that the problem remains: "The behavior in my installed matplotlib, v. 1.2.1, is still as I described in the issue. I can add that there are some differences across backends for the left-aligned multi-line text. With the Agg backend (and the PNG file), the numerals in the "12345" line are left-aligned by their glyphs or ink, so that they're slightly misaligned relative to the columns of the other numerals. In contrast, with the ps, pdf, and svg backends, the "12345" line is aligned with the others. For those backends, perhaps the glyph-based alignment occurs only for non-left alignments."
4. mdboom commented on Jul 23, 2013

   

   mdboom

   on Jul 23, 2013

   Member

   More actions

   First of all -- sorry this flew under the radar for so long.

   I think it will be easiest to address this as part of my ongoing MEP14 work to overhaul a lot of the text processing. Not because it couldn't be done now, but because we might as well fix these sort of subtle things while fixing a bunch of others, and also to avoid implementing this fix twice. It's sat here for 5 years -- it can sit for a few more months ;)
5. tacaswell

   added this to the v1.5.xThis milestone has been deleted milestone on Aug 18, 2014
6. tacaswell

   added

   MEP14

   on Aug 18, 2014
7. sgmentzer commented on Jan 22, 2015

   

   sgmentzer

   on Jan 22, 2015

   More actions

   This issue is still present in 1.5.dev1 [edit: under Windows 7 SP1 x64]. I also upgraded FreeType to 2.5.5 with no change.

   Here's some test code. It outputs to PDF the same graph with the six common Windows & Mac TrueType fonts, plus the Abel font (source URL below). SVG exhibits different oddities in the same glyphs.

   The problem is noticeable with Arial and Trebuchet and the '1' glyph, but glaring with Abel and the '4' glyph:

   
   Here's the same data aligned in MS Word:

   
   As suggested above, the cause looks to be somewhere in the FT2Font code.

   from matplotlib import rcParams
   from matplotlib.backends.backend_pdf import FigureCanvasPdf as FigureCanvas
   #from matplotlib.backends.backend_svg import FigureCanvasSVG as FigureCanvas
   from matplotlib.figure import Figure
   from matplotlib import lines
   
   data = [ { 'q1': 37.4, 'q2': 4.0, 't': 'Average carton' },
            { 'q1': 31.3, 'q2': 0.2, 't': 'Special order'},
            { 'q1': 7.2, 'q2': 1.1, 't': 'Standard price'},
            { 'q1': 47.1, 'q2': 0.4, 't': 'Double final limit'},
            { 'q1': 1.1, 'q2': 1.4, 't': 'Lorem ipsum dolor sit amet'}
           ]
   
   col1 = 0.3
   col2 = 0.375
   col3 = 0.4
   top = 0.8
   spacing = 0.04
   fonts = [ 'Arial', 'Times New Roman', 'Georgia', 'Tahoma', 'Trebuchet MS', 'Verdana', 'Abel' ] # Abel source: http://www.google.com/fonts/specimen/Abel
   rcParams["font.size"] = 12
   
   for i in range(len(fonts)):
       rcParams["font.family"] = fonts[i]
   
       fig = Figure()
       canvas = FigureCanvas(fig)
   
       ax = fig.add_subplot(111)
       ax.set_yticks([x / 10.0 for x in range(0, 11)])
       ax.set_xticks(ax.get_yticks())
   
       ax.text(0.5, 0.9, fonts[i], horizontalalignment='center')
       for j in range(len(data)):
           r = data[j]
           ax.text(col1, top-j*spacing, r['q1'], horizontalalignment='right')
           ax.text(col2, top-j*spacing, r['q2'], horizontalalignment='right')
           ax.text(col3, top-j*spacing, r['t'])
   
       ax.add_line(lines.Line2D((col1, col1), (0.85, 0.60), linestyle='-', linewidth=0.1, color='k'))
       ax.add_line(lines.Line2D((col2, col2), (0.85, 0.60), linestyle='-', linewidth=0.1, color='k'))
       ax.add_line(lines.Line2D((col3, col3), (0.85, 0.60), linestyle='-', linewidth=0.1, color='k'))
   
       fig.savefig('mpl-' + fonts[i] + '.pdf')
   
8. mdboom

   self-assigned this

   on Jan 23, 2015
9. mdboom

   added a commit that references this issue on Jan 23, 2015

   Fix matplotlib#253. Use advance width to calculate bounding box.

   ...

   This issue will close once commit abd9b8a is merged into the 'master' branch.

   abd9b8a
10. mdboom

    added

    status: needs review

    on Jan 23, 2015
11. mdboom

    added a commit that references this issue on Jan 27, 2015

    Fix matplotlib#253: Use advance width to calculate text bounding box

    This issue will close once commit c6d0183 is merged into the 'master' branch.

    c6d0183
12. tacaswell

    closed this as completedin #4031on Feb 19, 2015
13. tacaswell

    removed

    status: needs review

    on Feb 19, 2015