Description: When saving document in RTF format in code are swapped \clpadl and \clpadt (table cell padding). Top padding is writing as \clpadl and left padding is writing as \clpadt. Actual Results: - In LibreOffice Writer set top padding for cell in table to 2mm and left padding to 5mm - Save document in RTF format - Check code of document Expected Results: In code are swapped \clpadl and \clpadt values. Reproducible: Always User Profile Reset: No Additional Info:
Miklos: looking at https://opengrok.libreoffice.org/xref/core/writerfilter/source/rtftok/rtfdispatchvalue.cxx?r=6d38fc3d#1381 1392 case RTF_CLPADB: 1393 nSprm = NS_ooxml::LN_CT_TcMar_bottom; 1394 break; 1395 case RTF_CLPADL: 1396 nSprm = NS_ooxml::LN_CT_TcMar_top; 1397 break; 1398 case RTF_CLPADR: 1399 nSprm = NS_ooxml::LN_CT_TcMar_right; 1400 break; 1401 case RTF_CLPADT: 1402 nSprm = NS_ooxml::LN_CT_TcMar_left; 1403 break; Shouldn't it be: 1395 case RTF_CLPADT: 1396 nSprm = NS_ooxml::LN_CT_TcMar_top; 1397 break; 1401 case RTF_CLPADL: 1402 nSprm = NS_ooxml::LN_CT_TcMar_left; 1403 break; ?
> // Top and left is swapped, that's what Word does. Is the comment above the block, did you consider that?
(In reply to Miklos Vajna from comment #2) > > // Top and left is swapped, that's what Word does. > > Is the comment above the block, did you consider that? Oups, I should open my eyes more widely! :-)
(In reply to Miklos Vajna from comment #2) > > // Top and left is swapped, that's what Word does. > > Is the comment above the block, did you consider that? Miklos, should this bug be closed as Wontfix? Thanks
I think so, yes. The Writer RTF import follows the RTF spec and also Word's behavior. But in case of conflict, we follow Word's behavior, not what's in the spec. Based on this guideline, I think it makes no sense to change behavior here, sorry.