<div dir="ltr">For what it's worth as I am no sw expert, I'd say do the commit. <div>It isn't like there is a real chance of an interoperability  problem, AFAIK only other implementation is AOO. </div><div>so... What could possibly go wrong?</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Sep 29, 2018 at 4:51 PM Lionel Elie Mamane <<a href="mailto:lionel@mamane.lu" target="_blank">lionel@mamane.lu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In the absence of feedback from sw people, we just commit the change?<br>
<br>
On Thu, Jul 26, 2018 at 03:10:37PM +0200, Lionel Elie Mamane wrote:<br>
> Anybody?<br>
> <br>
> On Sun, Jun 17, 2018 at 11:48:49AM +0200, Lionel Elie Mamane wrote:<br>
> > Hi,<br>
> > <br>
> > The Base report builder produces Writer (odt) documents directly (it<br>
> > spits out XML, not using Writer to produce it), and then opens them in<br>
> > Writer. It uses tables heavily.<br>
> > <br>
> > In the context of<br>
> > <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=45789" rel="noreferrer" target="_blank">https://bugs.documentfoundation.org/show_bug.cgi?id=45789</a><br>
> > <a href="https://gerrit.libreoffice.org/53977" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/53977</a><br>
> > Ilhan prepared a patch to make Report Builder mark some rows of some<br>
> > tables as "fit to size" in row height. By reading the OpenDocument<br>
> > specification, I thought this would happen by setting the<br>
> > style:use-optimal-row-height attribute on the row. However, it turns<br>
> > out that Ilhan had to change some sw code for that to work, and that<br>
> > made me suspicious whether that was the right thing to do.<br>
> > <br>
> > I looked into the XML generated by Writer with "Fit to Size" enabled<br>
> > and disabled, and apparently that checkbox in writer replaces<br>
> > "style:row-height" by "style:min-row-height, and<br>
> > style:use-optimal-row-height is nowhere to be seen.<br>
> > <br>
> > Could a Writer expert please look at the sw changes in<br>
> > <a href="https://gerrit.libreoffice.org/53977" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/53977</a><br>
> > and let us know whether these changes are right or wrong with respect<br>
> > to OpenDocument? Is style:use-optimal-row-height supposed to override<br>
> > style:row-height or vice-versa?<br>
> > <br>
> > Thanks in advance,<br>
> > <br>
> _______________________________________________<br>
> LibreOffice mailing list<br>
> <a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</blockquote></div>