開いているファイルのフォルダのパスをクリップボードにコピーできるようにする

[m-tmatma]

#658: 開いているファイルのフォルダのパスをクリップボードにコピーできるようにする

[KageShiron]

ParentFolderやParentFolderNameのほうが一般的ではないでしょうか？

例↓
https://msdn.microsoft.com/ja-jp/library/cc428026.aspx

[m-tmatma]

いろんな言語の状況

言語	メソッド名	参考
Java	getParent	https://docs.oracle.com/javase/jp/6/api/java/io/File.html#getParent()
WSH	GetParentFolderName	https://msdn.microsoft.com/ja-jp/library/cc428026.aspx
.NET	Path.GetDirectoryName	https://docs.microsoft.com/ja-jp/dotnet/api/system.io.path.getdirectoryname?view=netframework-4.7.2
Perl	dirname	http://perldoc.perl.org/File/Basename.html
Python	dirname	https://docs.python.org/2/library/os.path.html
Ruby	dirname	https://docs.ruby-lang.org/ja/latest/method/File/s/dirname.html

[beru]

COPYFOLDER だとフォルダ自体をコピーするのかな？と勘違いしてしまいそうです。

[m-tmatma]

では copydirname とかにしましょうか?

[beru]

では copydirname とかにしましょうか?

そうですね、name とか path を後ろにつける事でより分かりやすくなると思います。

[m-tmatma]

copydirname より copydirpath かな?

copydirname だとディレクトリだけの名前を意味する感じがして、
絶対パス全体という感じがしない。

[m-tmatma]

マクロ名を変更しました。

[beru]

無題の文書で このファイルのパス名をコピー は無効化されていますが、このファイルのフォルダ名をコピー は有効化されています。こちらも無題の文書の場合は無効化した方が良いかと思います。

[m-tmatma]

抜けてました。修正しました。

メニューの有効化条件の指定れ漏れが修正された為

[beru]

タブバーを表示してタブを右クリックした際に表示されるメニューには今回追加された機能用のメニューアイテムが無いですがそれは意図したデザインでしょうか？

[m-tmatma]

タブバーを表示してタブを右クリックした際に表示されるメニューには今回追加された機能用のメニューアイテムが無いですがそれは意図したデザインでしょうか？

パスのコピー機能がタブバーにないと思っていたので入れてませんでした。
確認したところ入っていたのでいれました。

なお、#680 で入れたアイコンの修正は入れてません。
#607 を再オープンしておきました。

[m-tmatma]

パスのコピー機能がタブバーにないと思っていたので入れてませんでした。

#666 で自分で入れたのに忘れてた。

[m-tmatma]

F_COPYDIRPATH の定義値がまずいみたいです。

以下の276行目で IDD_PROP_TOOLBAR の IDC_LIST_FUNC 項目を追加するときに (長いのでスクロール可能)

sakura/sakura_core/prop/CPropComToolbar.cpp

Lines 253 to 289 in 9b2f273

	case WM_COMMAND:
	wNotifyCode = HIWORD( wParam ); /* 通知コード */
	wID = LOWORD( wParam ); /* 項目ID､ コントロールID､ またはアクセラレータID */
	hwndCtl = (HWND) lParam; /* コントロールのハンドル */
	if( hwndResList == hwndCtl ){
	switch( wNotifyCode ){
	case LBN_SELCHANGE:
	return TRUE;
	}
	}else
	if( hwndCombo == hwndCtl ){
	switch( wNotifyCode ){
	case CBN_SELCHANGE:
	nIndex2 = Combo_GetCurSel( hwndCombo );
	List_ResetContent( hwndFuncList );
	/* 機能一覧に文字列をセット (リストボックス) */
	// From Here Oct. 15, 2001 genta Lookupを使うように変更
	nNum = m_cLookup.GetItemCount( nIndex2 );
	for( i = 0; i < nNum; ++i ){
	nIndex1 = m_cLookup.Pos2FuncCode( nIndex2, i );
	int nbarNo = m_pcMenuDrawer->FindToolbarNoFromCommandId( nIndex1 );
	if( nbarNo >= 0 ){
	/* ツールバーボタンの情報をセット (リストボックス) */
	lResult = ::Listbox_ADDDATA( hwndFuncList, (LPARAM)nbarNo );
	if( lResult == LB_ERR || lResult == LB_ERRSPACE ){
	break;
	}
	lResult = List_SetItemHeight( hwndFuncList, lResult, nListItemHeight );
	}
	}
	return TRUE;
	}
	}else{

以下の関数で追加しますが、

sakura/sakura_core/uiparts/CMenuDrawer.cpp

Lines 1444 to 1463 in 9b2f273

	/*
	ツールバー登録のための番号を返す。
	プラグインのみボタンのindexのかわりにidCommandをそのまま返す
	@date 2010.06.24 Moca 新規作成
	@note この値がiniのツールバーアイテムの記録に使われる
	*/
	int CMenuDrawer::FindToolbarNoFromCommandId( int idCommand, bool bOnlyFunc ) const
	{
	// 先に存在確認をする
	int index = FindIndexFromCommandId( idCommand, bOnlyFunc );
	if( -1 < index ){
	// 固定部分以外(プラグインなど)はindexではなくidCommandのままにする
	if( m_nMyButtonFixSize <= index ){
	// もし コマンド番号が明らかに小さいと区別がつかない
	assert_warning( idCommand < m_nMyButtonFixSize );
	return idCommand;
	}
	}
	return index;
	}

以下のチェックが行われています。

sakura/sakura_core/uiparts/CMenuDrawer.cpp

Lines 1475 to 1483 in 9b2f273

	int CMenuDrawer::FindIndexFromCommandId( int idCommand, bool bOnlyFunc ) const
	{
	if( bOnlyFunc ){
	// 機能の範囲外（セパレータや折り返しなど特別なもの）は除外する
	if ( !( F_MENU_FIRST <= idCommand && idCommand < F_MENU_NOT_USED_FIRST )
	&& !( F_PLUGCOMMAND_FIRST <= idCommand && idCommand < F_PLUGCOMMAND_LAST )){
	return -1;
	}
	}

F_MENU_NOT_USED_FIRST = 32768

sakura/sakura_core/Funccode_x.hsrc

Line 495 in 9b2f273

F_MENU_NOT_USED_FIRST = 32768,

F_COPYDIRPATH = 32820 なのでメニュー項目に追加されないみたいです。

sakura/sakura_core/Funccode_x.hsrc

Line 261 in 9b2f273

F_COPYFOLDER = 32820, //このファイルのフォルダ名をクリップボードにコピー なし

[m-tmatma]

別の問題を見つけたので 432251d で修正しています。

[beru]

F_COPYDIRPATH の定義値がまずいみたいです。

おおお、知らなかったです。メニュー項目のIDは、F_MENU_FIRST 以上、F_MENU_NOT_USED_FIRST 未満にする必要があるんですね。デバッグビルドだと自動的に検出してくれると良いですねぇ。

[beru]

別の問題を見つけたので 432251d で修正しています。

キーマクロは自分は普段使っていないので確認が漏れてました。
記録内容は、

CopyDirPath(0);	// このファイルのフォルダ名をコピー

となっていて引数の 0 は有っても無くてもちゃんと動きますね。何故追加されるのかは良く分かっていません。。いやまぁ調べれば良いんですが。

[beru]

問題無いと思うので再度 Approve します。

[m-tmatma]

共通設定 の ツールバー の 種別が クリップボード系 の一覧に項目が無いですがアイコンがまだ無いからでしょうか？

表示できました。ただアイコンは暫定のものです。

[m-tmatma]

F_COPYDIRPATH の定義値がまずいみたいです。

これもまずいが、 F_COPYDIRPATH の値の追加も抜けていた。 c1c6792

[beru]

表示できました。ただアイコンは暫定のものです。

確認しました。アイコンはもし自分が用意するなら C: に下線が付いただけにします。
このファイルのパス名をコピー が C:\… に下線が付いたものなのでそれをちょっと変えるだけという単純な考えですが、センタリングはした方が良いかも知れないですね。

[beru]

問題無いと思うので再度 Approve します。