<html>
    <head>
      <base href="https://bugs.documentfoundation.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Text Body default line spacing too big (proportional 115% instead of Single )"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=94464#c17">Comment # 17</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Text Body default line spacing too big (proportional 115% instead of Single )"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=94464">bug 94464</a>
              from <span class="vcard"><a class="email" href="mailto:vstuart.foote@utsa.edu" title="V Stuart Foote <vstuart.foote@utsa.edu>"> <span class="fn">V Stuart Foote</span></a>
</span></b>
        <pre>(In reply to RGB from <a href="show_bug.cgi?id=94464#c16">comment #16</a>)

<span class="quote">> Can you please reread what I wrote? Typefaces come BY DESIGN with line
> spacings that are between 120 and 140% of the POINT size: Libertinux is
> around the 120% and, as I commented on the description, Sukhumala is at
> 130%. Setting another 120% on top of an existing 130% gives you a frikin
> 156%!!! That cannot be right.
> </span >

Yes this is correct

<span class="quote">> Please, don't trust too much the details on the quoted page, the phrase
> "Word uses pe­cu­liar line-spac­ing math" shows that the author do not
> understand how "most word processors" work. The math is not peculiar, in
> fact there is no math at all: single line spacing is defined by the typeface
> itself! Nowadays only LaTeX uses the point size to define line spacing and
> THERE you'll find some "peculiar line-spacing math".</span >

Also, correct. As we now consistently use either the hhea or OS/2 Typo metric,
we do not need to control the line spacing against "effective" point size--and
in reality we don't. We simply apply a multiplier against the font metric
nHeight to calculate line spacing.

The way we handle paragraph line spacing property is with a SetPropLineSpace(),
default "Single" space is "100" (i.e. as pulled from the font metrics or using
the calculated nHeight of the font.  Proportional is also known as User
setting--and Ahmad's 2013 commit to the default template simply prepopulates
the value for the "Text Body" style to be Proportional at "120", e.g. 120%. 

Unfortunately, because it is a default template--users now must modify the
style to their preference either Single, a user "Proportional" value (as 120%
is too large), the "fixed" 1.15 value--and save as their new default template.

At the least we should reduce the DocumentStylePoolManager.cxx template default
"Proportional" from 120 -> ~105-108; or probably better follow the Default
style's "Single" spacing at 100.  Either would allow the font's height to
correctly apply--and makes the "Text Body" style more efficient to use.

An alternative might be to define User/Proportional but to include "leading"
(e.g. nInterlineSpace) by default--but that could cause problems if "Text Body"
becomes the more frequently used default "Default".

For what it is worth, current Microsoft Office normal.dotm sets spacing for
multi-line paragraph to 1.08, not 1.15

=-ref-=
<a href="http://opengrok.libreoffice.org/xref/core/editeng/source/items/paraitem.cxx#78">http://opengrok.libreoffice.org/xref/core/editeng/source/items/paraitem.cxx#78</a>

<a href="http://opengrok.libreoffice.org/xref/core/sw/source/core/doc/DocumentStylePoolManager.cxx#372">http://opengrok.libreoffice.org/xref/core/sw/source/core/doc/DocumentStylePoolManager.cxx#372</a>

<a href="http://opengrok.libreoffice.org/xref/core/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx#112">http://opengrok.libreoffice.org/xref/core/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx#112</a></pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>