<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Wrong text size when using HFONT with appropriate size"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=94836">94836</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>Wrong text size when using HFONT with appropriate size
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>cairo
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>1.12.14
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>Windows (All)
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>major
          </td>
        </tr>

        <tr>
          <th>Priority</th>
          <td>medium
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>win32 backend
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>cairo-bugs@cairographics.org
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>fredbca21@gmail.com
          </td>
        </tr>

        <tr>
          <th>QA Contact</th>
          <td>cairo-bugs@cairographics.org
          </td>
        </tr></table>
      <p>
        <div>
        <pre>I have noticed a bug in the win32 font backend. When setting the font on a
win32 surface with the code below and using the same size as the original HFONT
for drawing, the resulting font size is extremely small (text is almost
invisible).

// assuming my_hfont was created with -12 for LOGFONT.lfHeight
cairo_font_face_t* font_face=cairo_win32_font_face_create_for_hfont(my_hfont);
cairo_set_font_face(ctx,font_face);
cairo_set_font_size(ctx,12);

The reason is that in _cairo_win32_font_face_scaled_font_create, the font
matrix is checked against -font_face->logfont.lfHeight whereas if you look in
_win32_scaled_font_get_scaled_hfont, the expected height should be multiplied
by WIN32_FONT_LOGICAL_SCALE (scaled HFONT created by cairo uses
logfont.lfHeight = -scaled_font->logical_size)

This means that using anything else than 1 for WIN32_FONT_LOGICAL_SCALE will
result in a downscaled font when using a HFONT that was created with the right
size.

However, if the check mentioned above is fixed, in order to have cairo use the
original HFONT, it should be created with a much larger size
(-WIN32_FONT_LOGICAL_SCALE*size). So this defeats the purpose of being able to
mix native win32 fonts with cairo because of this scaling factor (mixing native
and cairo code would require extra scaling everywhere).

After reading older discussions about the WIN32_FONT_LOGICAL_SCALE factor, it
seems to me that the actual problem lies in the fact that the win32 backend
uses fonts that are expected to be created with the logical scale, instead of
using fonts with the appropriate size and applying the logical scale to the
World Transform only.

See scaled font creation here:
static cairo_status_t
_win32_scaled_font_get_scaled_hfont (cairo_win32_scaled_font_t *scaled_font,
    HFONT *hfont_out)
{
    if (!scaled_font->scaled_hfont) {
LOGFONTW logfont = scaled_font->logfont;
logfont.lfHeight = -scaled_font->logical_size; // using the logical
size to create the font - why not use the actual size and scale in the world
transform?</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>