フォントサイズ相対指定の仕様変更をSetFontSizeの説明に反映する #1544
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
berryzplus
reviewed
Feb 17, 2021
| i1に0を設定し、i2に正数(拡大)または負数(縮小)の変更量を設定します。</br> | ||
| 設定可能な変更量とそれに対応して適用される倍率を一部示します。</br> | ||
| <table border="1" cellspacing="0" cellpadding="3"> | ||
| <tr><th>i2</th><th>倍率</th></tr> |
There was a problem hiding this comment.
SonarScanでBugsレベルの指摘があがるので先に書いておきます。
（あんましこだわる気はありませんが、Bugsレベルはさすがに嫌ですｗ）
HTML仕様的に、tableタグはcaptionを含めるべきらしいです。
そして、テーブルヘッダ th には scope 属性を含めるべきらしいです。
周辺の table と似た雰囲気にしたら通るはずっす。
There was a problem hiding this comment.
無事通りました。
CodeSmells の方はすぐに対応できる内容ではなさそうでしたので放置します。
|
✅ Build sakura 1.0.3455 completed (commit 74b2e98cec by @suconbu) |
|
Kudos, SonarCloud Quality Gate passed!
|
|
✅ Build sakura 1.0.3457 completed (commit 2fdfb5b560 by @suconbu) |
berryzplus
approved these changes
Feb 18, 2021
|
ありがとうございました。マージします。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
マクロコマンド
SetFontSizeのヘルプをメンテナンスします。カテゴリ
PR の背景
省略します。
PR のメリット
実際の動作とヘルプの説明との食い違いが解消されます。
PR のデメリット (トレードオフとかあれば)
特にありません。
仕様・動作説明
省略します。
PR の影響範囲
ヘルプファイル (S_SetFontSize.html) の更新のみです。
テスト内容
省略します。
関連 issue, PR
#1513
参考資料