<div dir="ltr"><div dir="ltr"></div>Németh László <<a href="mailto:nemeth@numbertext.org">nemeth@numbertext.org</a>> ezt írta (időpont: 2019. szept. 2., H, 8:19):<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi Regina,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Regina Henschel <<a href="mailto:rb.henschel@t-online.de" target="_blank">rb.henschel@t-online.de</a>> ezt írta (időpont: 2019. szept. 1., V, 19:44):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
please have a look at <a href="https://gerrit.libreoffice.org/#/c/78372/" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/#/c/78372/</a><br>
<br>
An important question is whether you agree that I have changed the <br>
import and export of our own line styles so that no prstDash line styles <br>
are used for them. Instead I export them as custDash elements and <br>
reconstruct them in the import.<br></blockquote><div><br></div><div>The original idea with binding preset styles of LO and MSO was to support some kind of interoperability: using the same or similar UI to check and modify the line styles.</div><div>I can imagine easily, that StarOffice/OpenOffice.org copied the preset styles of the old MSO originally. So maybe modifying/extending the preset styles of LibreOffice would be the final step to create a better interoperability again, and your nice improvement is the first step in this direction.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I have not added tests yet. Please tell me, what you think would be good <br>
to test.<br></blockquote><div><br></div><div>I believe, testing import/export of preset styles of LO and MSO are the most important,</div><div>but only unit testing can guarantee keeping the improvements, so maybe it's worth to extend the list.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
You know I always have a hard time with C++. So I welcome hints on how <br>
to implement something better.<br>
<br>
And technical question: What do I need to do locally in Git, so that the <br>
information, that the no longer needed file "lo_preset_dashes.odt" has <br>
to be deleted, is included in my commit?<br></blockquote><div><br></div><div>If you deleted the file accidentally,</div><div><br></div><div>git checkout sw/qa/extras/ooxmlexport/data/lo_preset_dashes.odt</div><div><br></div><div>and</div><div><br></div><div>git rm sw/qa/extras/ooxmlexport/data/lo_preset_dashes.odt</div><div><br></div><div>and add deletion to the actual commit:</div><div><br></div><div>git commit --amend</div><div><br></div><div>Best regards,</div><div>László<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Kind regards<br>
Regina<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></blockquote></div></div>
</blockquote></div></div>