Skip to content

Conversation

@mujurin1
Copy link

#10
CommentDataGrid (MultiCommentViewer\Views\CommentDataGrid.xaml) の コメントの右クリックメニュー (commentContext) の DataContext の修正です

今までは FindAncestor でWindowを検索していましたがこれをやめ、
右クリック時にイベントを実行、コードベースでContextを設定するようにしました

@mujurin1
Copy link
Author

(fix-13のコミットが含まれていたので削除しました)

@DaisukeDaisuke
Copy link
Owner

NETのデザイナーには詳しくなかったのでまじでありがとうございます。今日動作確認したあと2024年11月25日の早朝にマージしてビルドしてリリースして告知する予定です!
Fixed #10

@DaisukeDaisuke
Copy link
Owner

ニコニコニュース実況などを取得するとわかりますが、ユーザービューでどのコメントをコピーしても最新のコメントしかコピーできないようです。本当に申し訳ないです

https://live.nicovideo.jp/watch/lv346359423

@DaisukeDaisuke
Copy link
Owner

SelectedCommentがメインフォームの方さしてますね。どうやったらユーザービューの方取得できるんだ

@DaisukeDaisuke
Copy link
Owner

var message = SelectedComment.MessageItems.ToText();

@mujurin1
Copy link
Author

ユーザービューモデルではコメントのコピーやURLを開く関数(コマンド)がないので元々こういう動作なはずなんですが…
調べてみます

@DaisukeDaisuke
Copy link
Owner

役に立てずすいません。

@mujurin1
Copy link
Author

元々WPFの知識があったのでそれがちょうどよく知識がハマってるだけなので大丈夫です

@mujurin1
Copy link
Author

CommentDataGrid についてElementとModelを作って共通化してるのに CommentDataGrid のためのコードが MainViewModel にしかないのが原因ですね
これをちゃんと作るとちょっと面倒なので現在のアクティブなウィンドウを取得する方法で対応しようと思います

@mujurin1
Copy link
Author

修正しました
今後 CommentDataGrid の状態(コメント一覧や選択中コメントなど)を参照する場合は必ずGetActiveWindowContextAsCommentDataGrid 関数で取得した値から参照して下さい
そうしないとメインウィンドウの場合のみ正しく、サブウィンドウの場合はバグります
(自分でもバグ作りそうなので、こんなコードになってるのが悪いんですが…)

@DaisukeDaisuke
Copy link
Owner

修正を確認できました。貢献ありがとうございます。
私はもう寝る時間なので、朝起きたら(9時ぐらい)マージしてテスト、ビルド、更新、告知します。本当にありがとうございます!

@DaisukeDaisuke
Copy link
Owner

DaisukeDaisuke commented Nov 24, 2024

告知文にmujurin_2525さんの貢献があったことを掲載するのでそこは安心してください

@mujurin1
Copy link
Author

ありがとうございます

@DaisukeDaisuke DaisukeDaisuke merged commit 0eff3de into DaisukeDaisuke:master Nov 24, 2024
@DaisukeDaisuke
Copy link
Owner

ユーザービューが自動スクロールと干渉してスクロールできないことがあるのはまあいっか...

@DaisukeDaisuke
Copy link
Owner

リリース作業が完了しました。今回は本当にありがとうございました!

@mujurin1 mujurin1 deleted the issue-10 branch November 25, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants