D24830: Add support for strikeout text in RTF documents

2019-10-22 Thread Pekka Vuorela
This revision was automatically updated to reflect the committed changes. Closed by commit R8:11b3b7718796: Add support for strikeout text in RTF documents (authored by pvuorela). REPOSITORY R8 Calligra CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24830?vs=68442&id=68516 REVISION

D24830: Add support for strikeout text in RTF documents

2019-10-22 Thread Pekka Vuorela
pvuorela accepted this revision. pvuorela added a comment. This revision is now accepted and ready to land. Looking good to me. REVISION DETAIL https://phabricator.kde.org/D24830 To: akomakhin, pvuorela, #calligra:_3.0 Cc: boemann, Calligra-Devel-list, dcaliste, cochise, vandenoever

D24830: Add support for strikeout text in RTF documents

2019-10-21 Thread Camilla Boemann
boemann added a comment. As far as I'm concerned you may push to rtf importer at your own discretion REVISION DETAIL https://phabricator.kde.org/D24830 To: akomakhin, pvuorela, #calligra:_3.0 Cc: boemann, Calligra-Devel-list, dcaliste, cochise, vandenoever

D24830: Add support for strikeout text in RTF documents

2019-10-21 Thread Aleksandr Komakhin
akomakhin marked an inline comment as done. REVISION DETAIL https://phabricator.kde.org/D24830 To: akomakhin, pvuorela, #calligra:_3.0 Cc: Calligra-Devel-list, dcaliste, cochise, vandenoever

D24830: Add support for strikeout text in RTF documents

2019-10-21 Thread Aleksandr Komakhin
akomakhin updated this revision to Diff 68442. akomakhin added a comment. Fixed set value from "true" to expression " ! hasValue || ( hasValue && value != 0 ) " CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24830?vs=68436&id=68442 REVISION DETAIL https://phabricator.kde.org/D248

D24830: Add support for strikeout text in RTF documents

2019-10-21 Thread Pekka Vuorela
pvuorela added inline comments. INLINE COMMENTS > DocumentDestination.cpp:121 > + } else if ( controlWord == "strike" ) { > + m_output->setFontStrikeout(true); > } else if ( ( controlWord == "u" ) && hasValue ) { Value should be like bold and underline, i.e. something like "! h

D24830: Add support for strikeout text in RTF documents

2019-10-21 Thread Aleksandr Komakhin
akomakhin created this revision. akomakhin added reviewers: pvuorela, Calligra: 3.0. Herald added a project: Calligra: 3.0. Herald added a subscriber: Calligra-Devel-list. akomakhin requested review of this revision. REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D24830 A