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

文字間隔のオプションを追加した #78

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

I-Otsuki
Copy link

概要

こういう感じに字詰めを調整したり重ねたりできるようにしました。
image

気になってる点

  • 既存の関数の引数の並びのど真ん中に項目足しちゃってます
    • 画面の並びと一致を取ってるだけなのでこだわりはないです
  • 以下の理由で、短命なコードだと思ってます
    • Chromium系以外では動作しないっぽいです
    • CSSにpxの付いたDOMStringで喰わせてますが、textLetterSpacingがブラウザ側で実装されればdouble値としてctx.textLetterSpacing = letterSpacing;すればいいということになるっぽい

@zk-phi
Copy link
Owner

zk-phi commented Jun 2, 2021

ありがとうございます!

canvas を body 以下に追加したときの副作用が他にあるか (letter-spacing 以外に引き継ぐスタイルがある?) などが自信ないのと、別件で複数のパーツを自由にレイアウトできるとちょっと楽しそうだなと思っていて、それが実現できれば同様の絵文字を作って遊べそうなので、ちょっと保留させてくださいmm

@I-Otsuki
Copy link
Author

I-Otsuki commented Jun 2, 2021

ありがとうございます!複数パーツ配置気になる、楽しみにしてます!

@I-Otsuki
Copy link
Author

ウワッ!「なんか支離滅裂なことを書いた気がする」と思って見に来たら本当に支離滅裂なことを書いている!すみません!

レビューありがとうございます。
確かにbodyに一時的に追加した状態でのレンダリングはbody側で持ってるスタイルを継承する可能性を考慮できてませんでした。
仮に現段階で影響がなかったとしてもUI側のスタイル変更等でレンダリング内容が影響を受ける可能性があるのは危険ですね。
やはりtextLetterSpacing待ち……。ある程度実装されたころに覚えてたらまた投げます。

それはそれとして複数パーツ配置は楽しみにしておりますので何卒🙏

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.

2 participants