search

vivliostyle / vivliostyle.js

📖 The power of CSS typesetting, right at your fingertips.
https://vivliostyle.org
GNU Affero General Public License v3.0
606 stars 53 forks source link

Broken text layout at page/column break caused by inline SVG #1406

Closed MurakamiShinyu closed 1 day ago

MurakamiShinyu commented 1 month ago

Issue #1401 のコメント https://github.com/vivliostyle/vivliostyle.js/issues/1401#issuecomment-2437266565 で報告された不具合ですが、 #1401 とは別な原因によるものなので新しいissueにします。 (thanks @skmtjny):

インラインでSVGを挿入した際にも似た現象が発生していたのでお知らせします。

image 241025.zip

https://github.com/vivliostyle/vivliostyle.js/issues/1401#issuecomment-2437524528:

https://github.com/vivliostyle/vivliostyle.js/issues/1401#issuecomment-2437298647 で報告をもらったインラインSVGでの不具合は、別の原因のようです。 v2.19.2 での次の修正に問題があるようです。（この修正をする前のものでは問題が起きない）

  • PR #1037

この修正を見直します。

MurakamiShinyu commented 1 month ago

Fixed in #1407.

Test: https://vivliostyle-git-fix-issue1406-vivliostyle.vercel.app/#src=https://gist.githubusercontent.com/MurakamiShinyu/b496dbc2a0e181f34f5ab740f80249b7/raw/svgtest1.html

skmtjny commented 6 days ago

@MurakamiShinyu 改修ありがとうございました。 ご対応頂いたおかげでかなり改善されていたのですが、 まだいくつか崩れる場面が見られました。

ケース1　先頭の文字が2回表示される image

ケース2　SVGの表示が崩れる image

241122.zip

新しくissueを起こすべきか迷いましたが いったんこちらに報告させていただきます。

MurakamiShinyu commented 6 days ago

@skmtjny 報告ありがとうございます。修正が不完全だったようなので再オープンします。

MurakamiShinyu commented 1 day ago

PR #1424 で修正しました。

テスト：