Review for "improve import/export of line styles"
Michael Weghorn
m.weghorn at posteo.de
Mon Sep 2 06:18:48 UTC 2019
Hi Regina,
On 01/09/2019 19.44, Regina Henschel wrote:
> please have a look at https://gerrit.libreoffice.org/#/c/78372/
>
> An important question is whether you agree that I have changed the
> import and export of our own line styles so that no prstDash line styles
> are used for them. Instead I export them as custDash elements and
> reconstruct them in the import.
>
> I have not added tests yet. Please tell me, what you think would be good
> to test.
>
> You know I always have a hard time with C++. So I welcome hints on how
> to implement something better.
I can't really say anything about your questions regarding line styles...
>
> And technical question: What do I need to do locally in Git, so that the
> information, that the no longer needed file "lo_preset_dashes.odt" has
> to be deleted, is included in my commit?
You can use 'git rm <SOME_PATH>/lo_preset_dashes.odt' to mark that file
for deletion.
Kind regards,
Michael
More information about the LibreOffice
mailing list