Bug 66645 - XHTML export: bad alignment/spacing of mathematical expressions
Summary: XHTML export: bad alignment/spacing of mathematical expressions
Status: RESOLVED DUPLICATE of bug 118637
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: filters and storage (show other bugs)
Version:
(earliest affected)
4.2.0.0.alpha0+ Master
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:4.2.0
Keywords:
Depends on:
Blocks: HTML-Export
  Show dependency treegraph
 
Reported: 2013-07-06 13:58 UTC by Frédéric Wang
Modified: 2021-07-14 08:08 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:
Regression By:


Attachments
testcase (110.50 KB, application/vnd.oasis.opendocument.text)
2013-07-06 13:58 UTC, Frédéric Wang
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang 2013-07-06 13:58:33 UTC
Created attachment 82122 [details]
testcase

draw:frame with MathML expressions contain margin-left, margin-right and vertical-align styles that make the alignment and spacing of mathematical formulas incorrect. Try to export the testcase to XHTML.
Comment 1 Frédéric Wang 2013-07-06 14:06:28 UTC
I've submitted a patch for review:

https://gerrit.libreoffice.org/4751

This should fix the bad vertical alignment of formulas. A space before/after the <math> element is now added which may be less artificial than margin properties (issues remain when an inline formula is followed by e.g. a comma or period).
Comment 2 Commit Notification 2013-07-09 09:11:53 UTC
Frederic Wang committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=fb2edd686c8b8ba4318e957f7cf23060ffd3e2c3

  fdo#66645 XHTML export: ignore incorrect style on math objects



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 3 retired 2013-07-09 10:36:09 UTC
Already fixed?

Setting to NEW with compliments from jorendc :)
Comment 4 Xisco Faulí 2016-10-23 12:44:23 UTC
Hello Frederic,
Is this bug already fixed?
If so, could you please close it as RESOLVED FIXED?
Comment 5 QA Administrators 2018-06-27 02:47:31 UTC Comment hidden (obsolete)
Comment 6 QA Administrators 2020-06-27 03:49:20 UTC Comment hidden (obsolete)
Comment 7 stragu 2021-07-03 12:47:01 UTC
It is hard to know what exactly were the original visual issues reported here, but I tested in LO 7.2 beta1 and could only find the attributes mentioned in a .fr1 CSS style, which is not even used in the rest of the document.

So as far as I understand it, the issue is now fixed. Closing as WFM.

Some extra issues with the example document could be reported separately (or might already have been reported), like the lack of space before some formulas objects, like Object3 and Object19.

Tested with:

Version: 7.2.0.0.beta1 / LibreOffice Community
Build ID: c6974f7afec4cd5195617ae48c6ef9aacfe85ddd
CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: en-AU (en_AU.UTF-8); UI: en-US
Calc: threaded
Comment 8 Xisco Faulí 2021-07-14 08:08:42 UTC
I believe this was fixed by these two commits mentioned here: https://bugs.documentfoundation.org/show_bug.cgi?id=118637#c6
Closing as duplicate of bug 118637

*** This bug has been marked as a duplicate of bug 118637 ***