「このファイルのフォルダ名をコピー」の機能のアイコンを実装する

[berryzplus]

PR の目的

「このファイルのフォルダ名をコピー」のツールバーアイコン（ツールバー番号178)を追加して、次回リリースができるようにします。

カテゴリ

• 不具合修正

PR の背景

「このファイルのフォルダ名をコピー」の機能追加時に、
対応するツールバー番号のアイコンを実装してなくて、
いままでずっと放置されていたのを対応します。

PR のメリット

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

仕様・動作説明

タブバーの右クリックメニューに表示されるアイコンを修正します。

「このファイルのフォルダ名をコピー」のアイコンが欠落しており、赤枠に「178」の仮アイコンが表示されていました。
とりあえず、類似機能「このファイルのファイル名をコピー」（ツールバー番号171）をコピーしました。

PR の影響範囲

タブバーの右クリックメニューに表示されるアイコンに影響があります。

テスト内容

赤枠に「178」ではなく、絵が出ることを確認しました。

テスト1

手順

関連 issue, PR

• resolves 「このファイルのフォルダ名をコピー」の機能のアイコンを実装する #1166
• related 「エクスプローラーで開く」、「コマンドプロンプトを開く」等のコマンドのアイコンを対応する #607(マージできたらタスクリストの更新が必要。)
• related ツールバー用のビットマップを結合するツール：インデックスカラー画像の場合はなるべく元のパレットを保持して結合するように対処を追加 #1137 修正箇所の切り出しには ToolBarImageSplitter を使いました。分割便利ですね。
  ToolBarImageMuxerはエラーが出て結合に使えませんでした。
  この件に .NET Frameworkのターゲットを変更する #1756 が関連するかどうかは未検証です。

参考資料

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[AppVeyorBot]

✅ Build sakura 1.0.4050 completed (commit 578b460354 by @berryzplus)

[beru]

動作確認はしていませんが、画像ファイルを目視で確認しました。

my_icons.bmp と mytool.bmp では結構絵柄が違う事に気づきましたが元からなので問題無いと思います。

mytool.bmpの404番の位置に405番の仮アイコンがあるように見えますがこれも多分元からだと思うので問題無いと思います。

[berryzplus]

mytool.bmpの404番の位置に405番の仮アイコンがあるように見えますがこれも多分元からだと思うので問題無いと思います。

この件を簡単に確認できるようなバッチがあったら便利かな？と思いました。

[berryzplus]

確認しました。404番に赤枠で「405」の仮アイコンが入ってますね。

C:\work\sakura-editor\sakura> cd tools\ToolBarTools
C:\work\sakura-editor\sakura\tools\ToolBarTools>ToolBarImageSplitter\bin\Debug\ToolBarImageSplitter.exe ..\..\resource\mytool.bmp .\mytool

[berryzplus]

レビューありがとうございます。マージしちゃいます。
問題あれば別PRで対応します。
問題なくても、カッコいいアイコンへの差し替え提案は歓迎です。