ヘルプのcssの修正と無いファイルへのリンクを削除

[dep5]

PR の目的

1. cssの文法のミスの修正
2. 存在しないファイルへのリンクを削除

カテゴリ

• 不具合修正
• リファクタリング

PR の背景

PR のメリット

使われない部分の削除で、文法のエラーも消えて、コードがシンプルになります。

PR のデメリット (トレードオフとかあれば)

もともと表示には影響していない場所なのでデメリットはないと思います

仕様・動作説明

STRONG {
	COLOR: #909; BACKGROUND-COLOR: transparent;
	BORDER-RIGHT: 3px double; BORDER-TOP: 3px double; BORDER-LEFT: 3px double; BORDER-BOTTOM: 3px double
}

例えばこのように指定するとEdgeではこう表示されます。

下線しか反映されていない状態だったので
効いていなかった指定を削除します

PR の影響範囲

テスト内容

テスト1

手順

関連 issue, PR

#1443

参考資料

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[AppVeyorBot]

✅ Build sakura 1.0.4224 completed (commit 5d737b2d9c by @dep5)

[kengoide]

引用されているスタイルを見て「なぜ反映されないのか？」と頭の中に疑問符が浮かびましたが、元のスタイルシートを確認して状況を理解しました。

BORDER-RIGHT: 3px; BORDER-TOP: 3px; BORDER-LEFT: 3px; BORDER-BOTTOM: 3px double

right, top, leftのborder-styleが指定されていないので何も表示されない、ということですね。
ヘルプ内のどこかでstrongに別のスタイルを適用している可能性が考えられますが、追加スタイルの有無は確認されましたでしょうか？

その他の変更についてはコメントアウトされたマークアップの除去と文法ミスの修正ですので問題ないと思います。

[kengoide]

その他の変更についてはコメントアウトされたマークアップの除去と文法ミスの修正ですので問題ないと思います。

「文法ミスの修正」という認識は誤りでした。CSSの文末セミコロンは必須ではありませんので、あえて追加する修正を加える必要はないと思います。
https://developer.mozilla.org/en-US/docs/Web/CSS/Syntax

[berryzplus]

参考情報です。

strong タグを使った強調は、当初の意図とズレていると思います。

HTMLでテキストを太字強調するタグには <b>(非推奨) と <strong> があります。
過去、 <b> を <strong> に置き換える対応を行っているため、cssで特殊な強調を施される対象が当初の意図に反して増えている可能性があります。

自分はこの件、そのままやってもいいかな？と思ってましたが
せっかくなら定義を行ごとにバラした方がメンテナンスしやすい気がして
それを指摘するかどうかで迷って放置した感じです。

※行ごとにバラす

    BORDER-RIGHT: 3px;
    BORDER-TOP: 3px;
    BORDER-LEFT: 3px;
    BORDER-BOTTOM: 3px double

[dep5]

kengoideさん
コメントありがとうございます。

構文 - CSS カスケーディングスタイルシート - MDN
2つ目の画像を見てください
セミコロンが省略できるのは最後だけです。
このPRでは次の行がある時だけセミコロンを追加しています。

現在のstrongタグのスタイルの適用状態です

セミコロンを追加しました

[dep5]

berryzplusさん
コメントありがとうございます。
borderの線種を指定しなければ表示されませんので
下線だけでよいのならborder-bottomの指定だけでよいように思います。
<b>(太字)の置き換えにしては<strong>のヘルプのスタイル指定はちょっと強いようにも思いますが、許容範囲かなと思っています。

[dep5]

セミコロンの追加にしぼって
#1895 で対応することにしました。
コメントありがとうございました