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

word_data でフォントサイズの変更を可能にする #757

Merged
merged 1 commit into from
Jun 29, 2020

Conversation

goe0
Copy link
Contributor

@goe0 goe0 commented Jun 28, 2020

🔨 変更内容 / Details of Changes

  • word_data で使用するフォントのサイズを変更できる設定を追加しました
  • [fadein][center]等と同様に[fontSize=XX]として指定します

400,0,[fontSize=20]

🔖 関連Issue, 変更理由 / Related Issues, Reason for Changes

歌詞表示全体をデフォルトサイズ(14px)以外で統一したい場合、
全ての歌詞に対してHTMLタグの付与が必要であることから記述簡略化のため

📷 スクリーンショット / Screenshot

📝 その他コメント / Other Comments

  • 理論的には[fontSize=XX]という歌詞を持つ作品が存在すれば影響を与えますが、考慮しなくていいでしょう

@codeclimate
Copy link

codeclimate bot commented Jun 28, 2020

Code Climate has analyzed commit 4f54e50 and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Owner

@cwtickle cwtickle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request ありがとうございます!
フォントサイズの別指定が必要か迷いましたが、
サイズは色と違って途中で変えるケースが少ないので、これはこれでありだと思いました。

フォントサイズが直指定されていますが、これは Issue #758 対応時に合わせて対応することにします。

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.

None yet

2 participants