Skip to content

Conversation

@Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Feb 17, 2025

内容

ファイル読み込み系統の関数を統一し、重複を減らします。

引数はこうしてみました。

title: string;
name: string;
mimeType: string;
extensions: string[];
defaultPath?: string;

mimeTypeはelectronバックエンドで使えず、defaultPathはブラウザバックエンドで使えません。
まあ共通化しちゃっていいかなぁと。

関連 Issue

close #2522

その他

同様にファイル書き出し系のダイアログも統一できると思います。
といっても現状はSHOW_PROJECT_SAVE_DIALOGとSHOW_EXPORT_FILE_DIALOGだけかも。

やっぱりDRY原則は正しい気がします。

@Hiroshiba Hiroshiba requested a review from a team as a code owner February 17, 2025 16:39
@Hiroshiba
Copy link
Member Author

@sevenc-nanashi 若干不安なのでレビューお願いできると・・・ 🙇

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

良さそう。

@Hiroshiba
Copy link
Member Author

レビューありがとうございます、マージします!!

@Hiroshiba Hiroshiba enabled auto-merge February 19, 2025 11:32
@Hiroshiba Hiroshiba added this pull request to the merge queue Feb 19, 2025
Merged via the queue into VOICEVOX:main with commit 13d5333 Feb 19, 2025
11 checks passed
@Hiroshiba Hiroshiba deleted the refactor--ファイル読み込み系統の関数を統一 branch February 19, 2025 11:58
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.

showProjectLoadDialogを消してshowImportFileDialogに置き換える

2 participants