dep5

Results 20 comments of dep5

CodeSmellsを書きなおしてみましたが、 bool m_aSearchKeysFav[MAX_SEARCHKEY]; std::array m_aSearchKeysFav ビルド失敗したので対応はやめておきます appendを使うのがいいらしいと読んだのでAppendStringFが使えるCNativeWがよさそうと思ったのですが、 目的外使用だったんですね。std::wstringで書いてみましたが、どうですか? 履歴MAXは表示のみだったのですね。それに関しては考え違いでした。 https://github.com/sakura-editor/sakura/pull/1640#issuecomment-827212635 berryzplusさんのおっしゃるように やはり「ファイル」「フォルダ」は性質が違うし、 お気に入りの上限を変更するならユーザーさんのことを考えると増やすことしかできないと思うので 元から制限のなかった「ファイル」「フォルダ」は私は対象外にしたいです。 Grep除外条件についてですが、わたしも #1639 で追加した機能で使うことが多く、 除外関連はデフォルトのままでしか使ってないのでよくわかりません。 元のパッチをまねて途中まで対応しましたが、ダイアログの設定でうまくいきませんでした。 文字数的にタブが大きくなるし、タブのスクロールや共通設定のように多段タブなども使う必要があるかもしれません。 このPullRequestがマージされるかどうかわからないですし、 わたしの技量的にも編集量は少なめに済ませたいと思っていますので、Grep除外条件への対応はほかの方にお願いしたいと思います。

1行書き換えればOKというわけにはいかないんですね。 fillはここに書くのか、と勉強になります。ぜひプルリクエストお願いします。

GitHub Actionsってダウンロードにログインが必要じゃないですか? [Artifact download URL only work for registered users (404 for guests) #51](https://github.com/actions/upload-artifact/issues/51) InPrivateウィンドウ(Edge)で試してみるとわかると思います。 [nightly.link](https://nightly.link/) こんなサイトもあるようですけど、githubが提供しているものではないような? テストに参加してくれる人が減ってしまうと思います…

試してみたところ、 ○継承するを選んだ場合、「最後にサイズ変更したウィンドウ」ではなくて 「最後に移動したウィンドウ」のようですね。 2つとも継承するを選ぶ場合は 「ウィンドウの位置と大きさ」ダイアログを「閉じる」で閉じた後、 「共通設定」を「OK」で閉じて、 (ウィンドウの枠でマウスポインターが矢印になってから) ウィンドウのサイズを調整して、 ウィンドウを好みの位置に移動する とできるみたいです。

beruさん berryzplusさん コメントありがとうございます。 対応してみました

berryzplus さん コメントありがとうございます。 2. 一部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 ``` 3. font閉じタグの不具合を修正(人為的ミスの余地あり) についてですが...

style属性だと `` のように空白が入ったりセミコロンが入ったり検索しづらいので class属性に変更し、cssで指定します あわせて `#800000`を`maroon`に ``を``に変更しています blue green red gray white olive #800000(maroon)についての変換です cssのスペースの使い方、行末のセミコロンについて、揃えました。

対象ファイルが大量なのでいくつもPRをだすのは負担なのですが、どうにかならないですか? 1. cssのセミコロン [このコミット](https://github.com/sakura-editor/sakura/commit/d4409b4f4f82a429423cdcde9602e498f6b5c4e1)の際にこう変更されていますが、セミコロンを忘れたままくっ付けています。 ``` 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:...

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に移動するだけにしたいと思います。 必要ならば個別にタグかクラス名を変更、または新規追加しようと思いますが、どうでしょうか