Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Preserve fractional values for RGB colors #1433

Closed

Conversation

u1f992
Copy link

@u1f992 u1f992 commented Dec 19, 2024

fix #1432

Copy link

vercel bot commented Dec 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vivliostyle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 19, 2024 8:05am

@MurakamiShinyu
Copy link
Member

プルリクエストありがとうございます。

既存のサンプルでこれまでどおりに動くかを試しています。

例:「Vivliostyle で本を作ろう Vol. 6」

修正前のVivliostyle Viewer (v2.30.7)で:
https://vivliostyle.github.io/viewer/v2.30.7/#src=https://vivliostyle.github.io/vivliostyle_doc/ja/vivliostyle-user-group-vol6/content/

このPRのVivliostyle Viewerで:
https://vivliostyle-git-fork-u1f992-fix-1432-preserv-0fe804-vivliostyle.vercel.app/#src=https://vivliostyle.github.io/vivliostyle_doc/ja/vivliostyle-user-group-vol6/content/

組版結果の見た目は変わらないようですが、DevToolsで見てみると、気になるところがあります。

まず、要素のstyle属性の内容が冗長になっていることです。比較のスクリーンショット:

Screenshot 2024-12-19 19 22 52

修正前では、style属性の内容は margin や border-width などショートハンドプロパティになっていましたが、修正後ではロングハンドになって長くなっています。また、 margin-right: 2.4000000000000004px のように小数点以下が無駄に桁数が多くて内部表現のために生じる 0 や 9 の桁の連続が目立ちます。
実行での実害はなくても、DevToolsでレイアウト結果を調べたりするときに、不便かもしれません。

それから、ViewerのUIの動作のために動的に値が設定されるような要素で問題があります。

Screenshot 2024-12-19 19 25 26

この <div data-vivliostyle-outer-zoom-box="true" …> の要素は、Viewerのウインドウサイズを変えたりするたびにwidthやheightが変わりますが、値の更新のたびに追加されるので問題が起きています。

@u1f992
Copy link
Author

u1f992 commented Dec 19, 2024

すべてのスタイル追加をこの方式で行うのは必要以上に影響が大きく手に負えないと感じています。Issueに差し戻して別案を考えます。

@u1f992 u1f992 closed this Dec 19, 2024
@u1f992 u1f992 deleted the fix/#1432-preserve-fractional-values branch December 21, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Preserve fractional values for colors
2 participants