PR対象

ドキュメント(md、ヘルプファイル等)

カテゴリ

改善

PR の背景

#1505 (ヘルプHTMLの修正)の見直しを中心に気づいたところを修正しています。

仕様・動作説明

をに変更し からに変更する前のこの時点でだった22か所をに戻しています。強調箇所が黒太文字に変わります
の変更漏れを直しています
文字の色指定をstyle属性からclass属性に変更しています。 CSSでのデザインを行いやすくするのが目的です。重要性を表す色指定は必要ならば改めて別のclass名またはタグを当てます。
CSS中の必要なセミコロンが無いミスを修正します。合わせてスペースなどCSSの表記を整理しました。
CSSとHTMLのどちらにも同じstyle指定がある<table>タグについて、CSSに一本化します。
タグの閉じかっこの重複を削除します。https://github.com/sakura-editor/sakura/pull/1895#discussion_r1071702766

PR の影響範囲

強調箇所が黒太文字に変わります。後は見た目に変更はないと思います。

テスト内容

数ページブラウザーで確認しています。

参考資料

Dec 31 '22 14:12 dep5

<spanとでGrepした時の結果が違うのは HLP000145で閉じタグのないが使われているからです。

などもclassに変えようかと思いましたが、保留にしています。 という使用例もありました。 blue green red gray white oliveの６種あります。 gray white oliveは使用例も少ないのでcssに動かしていいのかな、と悩んでいます。

Dec 31 '22 14:12 dep5

:x: Build sakura 1.0.4241 failed (commit https://github.com/sakura-editor/sakura/commit/b57625633d by @dep5)

Dec 31 '22 16:12 AppVeyorBot

:x: Build sakura 1.0.4248 failed (commit https://github.com/sakura-editor/sakura/commit/6fbd672383 by @dep5)

Jan 04 '23 16:01 AppVeyorBot

:white_check_mark: Build sakura 1.0.4252 completed (commit https://github.com/sakura-editor/sakura/commit/442c9eeeed by @dep5)

Jan 06 '23 04:01 AppVeyorBot

:white_check_mark: Build sakura 1.0.4262 completed (commit https://github.com/sakura-editor/sakura/commit/4364fb5d81 by @dep5)

Jan 16 '23 13:01 AppVeyorBot

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

一部spanの置換を手動でrevert（人為的ミスの余地あり）についてですが でgrepなどが可能かと思います。手動revertしたのは次のページです。それ以外は1にあたるかと思います。

plugin.chm 概要
help\plugin\Text\overview.html

sakura.chm Grep
help\sakura\res\HLP000067.html

Grep置換
help\sakura\res\HLP000362.html

Windows 10 でのファイル拡張子関連付け
help\sakura\res\HLP000374.html

Windows 10 でのファイル拡張子関連付けのための準備作業
help\sakura\res\HLP000375.html

font閉じタグの不具合を修正（人為的ミスの余地あり）についてですが help/sakura/res/HLP000089.html ここでの混入となります

Jan 16 '23 13:01 dep5

style属性だと  のように空白が入ったりセミコロンが入ったり検索しづらいので class属性に変更し、cssで指定します

あわせて #800000をmaroonに をに変更しています

blue green red gray white olive #800000(maroon)についての変換です

cssのスペースの使い方、行末のセミコロンについて、揃えました。

Jan 16 '23 14:01 dep5

a3ff600 のstyleをclassに変更 「変更が必要な理由」が説明されていないように見えました。
cf52197 cssの行末にセミコロンを追加 「変更が必要な理由」が説明されていないように見えました。

何が問題で、どう対応するか？（どう対応したいか。）

詰め切らないうちにまとめてPRにしてしまった結果、「なんかよくわからんやつ」になってるように見えます。

よくよく見るとPRのタイトルもおかしいですよね。「ヘルプのタグをからへ変更する」

strongをspanに変更したいのは何故か？何か「目的」があって「変更という手段」を採るんじゃないんですかね？

何がしたいんでしたっけ？（分かってて書いてます。）

また、追加したコミットは目的と合致してますか？（合致しないと思って書いてます。）

Jan 17 '23 05:01 berryzplus

:white_check_mark: Build sakura 1.0.4265 completed (commit https://github.com/sakura-editor/sakura/commit/f0d2d7c2f0 by @dep5)

Jan 18 '23 13:01 AppVeyorBot

:white_check_mark: Build sakura 1.0.4266 completed (commit https://github.com/sakura-editor/sakura/commit/cd1b79169b by @dep5)

Jan 18 '23 14:01 AppVeyorBot

対象ファイルが大量なのでいくつもPRをだすのは負担なのですが、どうにかならないですか？

cssのセミコロンこのコミットの際にこう変更されていますが、セミコロンを忘れたままくっ付けています。

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

しかし、改行には宣言の区切りの働きはないので

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

こう解釈され、 BACKGROUND-COLOR: transparent BORDER-RIGHT: 3px; の二つが無効になっています

Edgeの開発者ツール https://github.com/sakura-editor/sakura/pull/1883#issuecomment-1365143111

これを修正するのが目的ですが、なぜこのようなミスが起きたかというと、 cssの行末のセミコロンがあったりなかったりなので無くてもよいだろうと思われたのだと思うので、この際全部にセミコロンをつけてしまおうというコミットです。

の色指定 https://github.com/sakura-editor/sakura/pull/1895#issuecomment-1384120704 変更理由はこれではだめですか？後でclass名を変える際も一括変換がしやすくなります

をで太字に戻す →のstyleが気になるで気になった部分をまとめたのですが

「#1505 で行われた変更について、再検討してみる（ついでにcssも見直す）」だとどうでしょうか？そういうつもりでまとめてもよいかと思っていたのですが…

→→ →→ 以前 → #1505 → 今回の変更

このPRへのコミットはこれで終わりにします。タイトルと概要を直せばよいならそうします。複数に分けないと駄目な場合はどう分けるか指示してください。

Jan 18 '23 14:01 dep5

追加の変更の意図は理解しました。

セミコロンの問題について、「原因コミットはこれです」に対して修正対策を行わず「別な対応」をしている点が気になりました。不具合修正なのであればそうすべきだと思います。追加対応を行うかどうかは好みですが、個人的には不要と思います。
spanの色指定の方法を変える件について、 redやblueは、boldとは違うと思うので腹落ちしていません。「文字色をredにする」は実装的には「color: red;」ですが、製作者の意図は「強い注意喚起」だったり「エラー」だったりなはずです。だとするとredやblueの色名をスタイルにして一括置換できるようにするのは問題で、技術的負債を作る行為のような気がします。

「https://github.com/sakura-editor/sakura/pull/1505 で行われた変更について、再検討してみる（ついでにcssも見直す）」
だとどうでしょうか？そういうつもりでまとめてもよいかと思っていたのですが…

その意図であればそう書いてください。「1505(ヘルプHTMLの修正)の見直し」なら理解できる内容と思います。

1903のコミット一覧では、何を不具合と修正していて、何を改善として修正しているかが分かると思います。

タイトルと概要を実態と合わせてもらえればあとの評価は実施しようと思います。

ところでこの指摘はスルーですかね？ #1505 の不具合修正が主題であれば直さないのはおかしいような。

Jan 19 '23 09:01 berryzplus

https://github.com/sakura-editor/sakura/pull/1895#discussion_r1071702766 スルーしていません。https://github.com/sakura-editor/sakura/pull/1895/commits/966978d38f2f84aaa5e59c0c6761ed6e6d8ef702 で対処しています。不具合修正がメインというわけではありません。警告対処の暫定対応と思われるので、別の方法があるのでは、という提案のつもりです。 #1505 の変更をすべて見たわけではありませんし・・・

「文字色をredにする」の使用例を見たところ、おっしゃるような「注意喚起」もありましたが、大部分は色分けする程度の意図しか無いように思います。色を指定しているのに、これは注意喚起だから何のクラスにして、これは同じ色だけど注意喚起じゃないから別のクラスにして、そして、CSSで別のタグに同じスタイルを指定したほうがいいということですか？

html中で直接スタイル指定は控えたほうがいいように思うので今回はCSSに移動するだけにしたいと思います。必要ならば個別にタグかクラス名を変更、または新規追加しようと思いますが、どうでしょうか

Jan 27 '23 15:01 dep5

Kudos, SonarCloud Quality Gate passed!