「ファイルの場所をエクスプローラーで開く」追加 #549

berryzplus · 2018-10-14T05:30:59Z

概要

タブメニューに「ファイルの場所をエクスプローラーで開く」のメニューを追加します。(issue #420)

利用方法

メニュー追加は初期設定に対して行っています。
この対応が導入済みのバイナリをクリーンインストールする場合に追加作業はありません。
既存のsakura.iniを使う場合には共通設定「カスタムメニュー」を変更して使える状態にします。

カスタムメニューの右上にある「選択」から「タブメニュー」を選んでください。

右側ペインにある一覧がカスタムメニューの設定内容になります。
左側ペインに機能カテゴリごとのコマンド一覧があるので、「ファイルの場所をエクスプローラーで開く」を選択して右側ペインにコピーします。

課題

メニューの名前が長すぎて他メニューと並べたとき違和感があります。
メニューの名前が長すぎてカスタムメニュー設定の表示が見切れています。
メニューアイコンが未作成です。
ヘルプの実装がやや微妙です。

その他

「無題」の状態で「ファイルの場所をエクスプローラーで開く」のメニューが非活性になるのは仕様です。「無題」はファイルパスが確定しておらず「ファイルの場所」がないためです。

ステータス

機能として問題なく利用できそうに思いますのでこのままマージ可能です。

m-tmatma

help/sakura/sakura.hh が削除されてますが
操作ミス?

m-tmatma · 2018-10-14T22:07:33Z

sakura_lang_en_US/sakura_lang_rc.rc

@@ -2260,6 +2260,7 @@ BEGIN
    F_VIEWMODE              "View Mode"
    F_PROPERTY_FILE         "File Property"
    F_PROFILEMGR            "Profile Maneger"
+	F_OPEN_FOLDER_IN_EXPLORER "Open Folder in File Explorer"


in File Explorer で File は余計だと思います。
Explorer で固有名詞だと思う。

Vs2017のソリューションビューでプロジェクトを右クリックした時に出るメニューに合わせました。

あんまりこだわりないですが、既存ソフトで使われている文言に合わせたいです。

Vs2017 でタブで右クリックすると Open Containing Folder と表示されるみたいです。

Windows Explorer と Internet Explorer という使い分けがあると思いました。けれどエクスプローラー(Explorer)がファイラを指すことに疑問の余地はないとも思います。

ぼくが言ったのこれね。エクスプローラービューでプロジェクトを右クリックしたメニューの下の方にでるやつ。

@m-tmatma さんが言ってるやつこれね。ファイルタブの右クリックで出るやつ。

Windows Explorer と Internet Explorer という使い分けがあると思いました。

そうなんですよね。日本人が「エクスプローラー」と呼んでるものの正式名称は Windows Explorer なんです。MSの中の人が何故 File Explorer という文言を使ったのかは謎です。

m-tmatma · 2018-10-14T22:09:15Z

help/sakura/sakura.hh が削除されてますが

sakura_core/sakura.hh というファイルもありますね。
これって重複して登録されているのですか?
だとしたら、この PR で削除するのではなく別 PR にした方がわかりやすいと思います。

m-tmatma · 2018-10-14T22:14:27Z

タブメニュー だけでなく、文章を編集するウィンドウの中で右クリックしたときも
表示するようにしていただけますか?
タブを表示しない設定のときも簡単にアクセスできるようにするためです。

m-tmatma · 2018-10-14T22:27:23Z

sakura_core/sakura.hh というファイルもありますね。
これって重複して登録されているのですか?

4c25594 のコメントに書いてますね。

この修正はこの PR に混ぜない方がいいです。

berryzplus · 2018-10-15T00:13:19Z

あとで開発環境保守系のコミット2件を別PRに分けときます。

右クリックメニューの話、初期設定を入れるのは簡単なので後でやっときます。どこに？のご希望があればどうぞ♪

だいぶごちゃごちゃしてるので人によっては設定で削ってそうですが 😭

berryzplus · 2018-10-15T14:25:38Z

#554 と #555 マージ後のリベース待ちです。

beru · 2018-10-15T18:11:14Z

ファイルの場所をエクスプローラーで開く が長すぎる場合は ファイルの場所のフォルダを開く とかにするのが良いと思います。

タブメニューに追加した項目にアイコンが設定されていないのが気になりました。

m-tmatma · 2018-10-15T21:45:06Z

#554 と #555 マージ後のリベース待ちです。

マージしました。

KageShiron · 2018-10-16T14:13:33Z

sakura_core/cmd/CViewCommander_File.cpp

+	std::wstring strFolder(GetDocument()->m_cDocFile.GetFilePath());
+	::PathRemoveFileSpec(&*strFolder.begin());
+
+	auto hInstance = ::ShellExecute(GetMainWindow(), L"explore", strFolder.c_str(), NULL, NULL, SW_SHOWNORMAL);


explorer.exe /select,パスかSHOpenFolderAndSelectItemsを使う方がファイル数が多い場合に便利かもしれません。

処理を共通化してどこかに外出しする場合には有用だと思います。
ここは開いているファイルの親フォルダを開く処理なので、対象は0 or 1です。
対象が0のときはエラーにしてしまうので、実質は対象が1つの場合専用の処理になります。

ただ、SHOpenFolderAndSelectItems を使った場合の「付加効果」は意外と魅力的で、やや悩みます。

explorer.exe /select,"path" というのも、フォルダを開いてファイルを選択するという意味です。「ファイルが多い」というのは、開いたフォルダにファイルが多くてエディタで開いているファイルがそこに埋もれることを指していると思います。

「ファイルの場所を開く」のだから、開いた場所でファイルが選択されているのは、まったくもって自然なことです(力説)。

日本語：　ファイルの場所を開く英語：　Open Containing Folder

メニュー：　「ファイルの場所を開く」挿入位置：　右クリックメニュー最下部、プロパティの上。

berryzplus · 2018-10-16T16:15:54Z

メニュー名を変更してリベースしました。

@KageShiron さんの提案への賛同が多い場合はこのPRに含めてしまおうかとも思います。
（ファイルの場所を開く、ではなくて、ファイルの場所を開いてファイルを選択する、にする）

当初と意味が違ってしまうので分けたい感じなんですけどね。

berryzplus · 2018-10-17T03:54:03Z

修正量も少ないですし、該当ファイルを選択する仕様の方が良さそうなので後でコミット積んでおきます。

m-tmatma · 2018-10-17T10:44:36Z

sakura_core/cmd/CViewCommander.cpp

@@ -233,6 +233,7 @@ BOOL CViewCommander::HandleCommand(
 	case F_BROWSE:				Command_BROWSE();break;				/* ブラウズ */
 	case F_VIEWMODE:			Command_VIEWMODE();break;			/* ビューモード */
 	case F_PROPERTY_FILE:		Command_PROPERTY_FILE();break;		/* ファイルのプロパティ */
+	case F_OPEN_FOLDER_IN_EXPLORER:  Command_OPEN_FOLDER_IN_EXPLORER();break;	/* ファイルの場所をエクスプローラーで開く */


スタイルに関する話ですが、
: ~ Command_OPEN_FOLDER_IN_EXPLORER の間はこの関数の他の場所に合わせて
タブの方がいいと思います。

メニュー名だけでなく、ソースコードのコメントも修正しておく。

ヘルプ内の項目名に変更を適用。説明文は「ファイルの場所をエクスプローラーで開く」とした。

変更前）　ファイルの場所を開く変更後）　ファイルの場所を開いてファイルを選択

m-tmatma · 2018-10-17T22:13:31Z

sakura_core/cmd/CViewCommander_File.cpp

+	LPCWSTR pszDocPath = GetDocument()->m_cDocFile.GetFilePath();
+
+	// Windows Explorerの引数をいれるバッファは固定サイズで用意しておく(かなり多め1.5倍)
+	WCHAR parameters[MAX_PATH * 3 / 2];


固定長バッファにしなくても、CNativeW で CNativeW::AppendStringF を
使えばいいと思います。

ここもう少し洗練させたいと思ってます

いま必ず二重引用符付ける風ですが、二重引用符は必ずしも要らんので

m-tmatma · 2018-10-17T22:18:28Z

sakura_core/func/Funccode.cpp

@@ -1207,6 +1209,7 @@ bool IsFuncEnable( const CEditDoc* pcEditDoc, const DLLSHAREDATA* pShareData, EF
 	case F_BROWSE:						//ブラウズ
 	//case F_VIEWMODE:					//ビューモード	//	Sep. 10, 2002 genta 常に使えるように
 	//case F_PROPERTY_FILE:				//ファイルのプロパティ	// 2009.04.11 ryoji コメントアウト
+	case F_OPEN_FOLDER_IN_EXPLORER:		//ファイルの場所を開く
 		return pcEditDoc->m_cDocFile.GetFilePathClass().IsValidPath();	// 現在編集中のファイルのパス名をクリップボードにコピーできるか


1213行目のコメントは F_COPYPATH に対するものですか?
それで機能拡張が繰り返されて、1213行目が複数の nId に対して呼ばれるようになっているのですか?

yesです。
誤解のないようにコメント削った方がいいかもです。

c++では複数のIDに対して同じ処理を行わせるためにbreak;を書かずにcaseラベルを並べることができます。既存コードが複数IDを処理するこうぞうになってたっぽいので乗っかりました。

switch文の書き方にも流派があるらしいです。
途中で宗旨替えしたパターンかな。

m-tmatma · 2018-10-17T22:20:07Z

sakura_core/sakura_rc.rc

@@ -2252,6 +2252,7 @@ BEGIN
    F_VIEWMODE              "ビューモード"
    F_PROPERTY_FILE         "ファイルのプロパティ"
    F_PROFILEMGR            "プロファイルマネージャ"
+	F_OPEN_FOLDER_IN_EXPLORER "ファイルの場所を開く"


細かいですが、2252行～2254行目はインデントがスペースなので合わせた方がいいと思います。

指摘ありがとうございます。
後で修正します。

別件ですがrcの書きっぷりが全体的に非標準なので見直しかけたいです。

重複文字列リソースとか、明らかに変なリソースとか、色々課題があるので。

m-tmatma · 2018-10-17T22:20:39Z

sakura_lang_en_US/sakura_lang_rc.rc

@@ -2257,6 +2257,7 @@ BEGIN
    F_VIEWMODE              "View Mode"
    F_PROPERTY_FILE         "File Property"
    F_PROFILEMGR            "Profile Maneger"
+	F_OPEN_FOLDER_IN_EXPLORER "Open Containing Folder"


こちらも sakura_rc.rc と同様

うえにおなじ。

周囲の記述に合わせて空白に変更。

MAX_PATHを使ったバッファサイズ決め打ちをやめ、やりたいことが分かるように記述を組み替え。

berryzplus · 2018-10-20T03:48:27Z

単に一回の書式指定で文字列を整形した方が効率が良く、わかりやすいと思います。
また、ダブルクオートも常につける方が、
シンプルで、メンテもしやすくて、実行効率もいいと思います。

この方法で修正しました。

berryzplus · 2018-10-20T04:32:44Z

事実誤認。

英語版explorerの正式名称は File Explorer で正しいっぽい・・・:sob:

m-tmatma

ありがとうございます。

berryzplus · 2018-10-20T04:49:23Z

あざーす。

完成品キャプチャ作ったので貼っときます 😄

…der_in_explorer 「ファイルの場所をエクスプローラーで開く」追加

berryzplus added the enhancement ■機能追加 label Oct 14, 2018

berryzplus force-pushed the feature/open_folder_in_explorer branch from 6dfe600 to 84d8ae0 Compare October 14, 2018 10:51

m-tmatma reviewed Oct 14, 2018

View reviewed changes

This was referenced Oct 15, 2018

LF改行が混入していたのを修正 #554

Merged

sakura.hhの参照を変更 #555

Merged

m-tmatma added this to the next release milestone Oct 15, 2018

KageShiron reviewed Oct 16, 2018

View reviewed changes

berryzplus added 2 commits October 17, 2018 00:52

「ファイルの場所をエクスプローラーで開く」追加

8a29f1d

ヘルプ項目「ファイルの場所をエクスプローラーで開く」追加

550438d

berryzplus force-pushed the feature/open_folder_in_explorer branch from 84d8ae0 to 7acf650 Compare October 16, 2018 15:56

berryzplus added 2 commits October 17, 2018 00:57

メニュー名を変更

7acf650

日本語：　ファイルの場所を開く英語：　Open Containing Folder

右クリックメニューの初期値に新メニューを追加

c63cf71

メニュー：　「ファイルの場所を開く」挿入位置：　右クリックメニュー最下部、プロパティの上。

m-tmatma reviewed Oct 17, 2018

View reviewed changes

berryzplus added 3 commits October 17, 2018 22:47

コマンド名修正の横展開

187863f

メニュー名だけでなく、ソースコードのコメントも修正しておく。

コマンド名変更をヘルプに反映

06a2835

ヘルプ内の項目名に変更を適用。説明文は「ファイルの場所をエクスプローラーで開く」とした。

ファイルの場所を開いて選択を実装

7dc7e05

変更前）　ファイルの場所を開く変更後）　ファイルの場所を開いてファイルを選択

m-tmatma reviewed Oct 17, 2018

View reviewed changes

berryzplus added 2 commits October 18, 2018 22:49

リソーススクリプトの空白を修正。

10c570c

周囲の記述に合わせて空白に変更。

コマンド文字列の作り方を変更

3855076

MAX_PATHを使ったバッファサイズ決め打ちをやめ、やりたいことが分かるように記述を組み替え。

berryzplus mentioned this pull request Oct 18, 2018

ドロップダウンメニューに履歴の管理を追加 #551

Merged

バッファサイズ固定以外のところを元に戻す

806dc54

m-tmatma approved these changes Oct 20, 2018

View reviewed changes

m-tmatma merged commit 25ce975 into sakura-editor:master Oct 20, 2018

berryzplus deleted the feature/open_folder_in_explorer branch October 20, 2018 04:49

This was referenced Oct 20, 2018

エディタの各ドキュメントのタブ部分の右クリックメニューでエクスプローラを開く機能を検討する #420

Closed

機能追加したときの既存 sakura.ini の扱いに関して #567

Closed

m-tmatma mentioned this pull request Oct 29, 2018

ファイルの場所をコマンドプロンプトで開く、および、ファイルの場所をPowerShellで開く を検討する #556

Closed

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019

Merge pull request sakura-editor#549 from berryzplus/feature/open_fol…

cff2581

…der_in_explorer 「ファイルの場所をエクスプローラーで開く」追加

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

「ファイルの場所をエクスプローラーで開く」追加 #549

「ファイルの場所をエクスプローラーで開く」追加 #549

berryzplus commented Oct 14, 2018

m-tmatma left a comment

m-tmatma Oct 14, 2018

berryzplus Oct 15, 2018

m-tmatma Oct 15, 2018

ds14050 Oct 15, 2018

berryzplus Oct 15, 2018

m-tmatma commented Oct 14, 2018

m-tmatma commented Oct 14, 2018

m-tmatma commented Oct 14, 2018

berryzplus commented Oct 15, 2018

berryzplus commented Oct 15, 2018

beru commented Oct 15, 2018

m-tmatma commented Oct 15, 2018

KageShiron Oct 16, 2018

berryzplus Oct 16, 2018

ds14050 Oct 17, 2018 •

edited

Loading

ds14050 Oct 17, 2018

berryzplus commented Oct 16, 2018

berryzplus commented Oct 17, 2018

m-tmatma Oct 17, 2018

m-tmatma Oct 17, 2018

berryzplus Oct 18, 2018

m-tmatma Oct 17, 2018

berryzplus Oct 18, 2018 •

edited

Loading

m-tmatma Oct 17, 2018

berryzplus Oct 18, 2018

m-tmatma Oct 17, 2018

berryzplus Oct 18, 2018

berryzplus commented Oct 20, 2018

berryzplus commented Oct 20, 2018 •

edited by beru

Loading

m-tmatma left a comment

berryzplus commented Oct 20, 2018

「ファイルの場所をエクスプローラーで開く」追加 #549

「ファイルの場所をエクスプローラーで開く」追加 #549

Conversation

berryzplus commented Oct 14, 2018

概要

利用方法

課題

その他

ステータス

m-tmatma left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

m-tmatma commented Oct 14, 2018

m-tmatma commented Oct 14, 2018

m-tmatma commented Oct 14, 2018

berryzplus commented Oct 15, 2018

berryzplus commented Oct 15, 2018

beru commented Oct 15, 2018

m-tmatma commented Oct 15, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ds14050 Oct 17, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

berryzplus commented Oct 16, 2018

berryzplus commented Oct 17, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

berryzplus Oct 18, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

berryzplus commented Oct 20, 2018

berryzplus commented Oct 20, 2018 • edited by beru Loading

m-tmatma left a comment

Choose a reason for hiding this comment

berryzplus commented Oct 20, 2018

ds14050 Oct 17, 2018 •

edited

Loading

berryzplus Oct 18, 2018 •

edited

Loading

berryzplus commented Oct 20, 2018 •

edited by beru

Loading