Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ログファイル選択時に不要な上書き確認ダイアログが表示される #376

Closed
sempreff opened this issue Oct 20, 2024 · 3 comments
Milestone

Comments

@sempreff
Copy link
Contributor

概要

既存のファイルを「...」ボタンからログファイルとして選択する際、
追記したい場合であっても「上書きしますか?」のダイアログに「はい」を選択する必要があります。

「上書きしますか?」のダイアログに「はい」と応答するには
ファイルが破壊される可能性を覚悟する必要があるので、抵抗があります。

また、「上書きしますか?」のダイアログに「はい」を応答したとしても
その後に Log ダイアログで「New / Overwrite」を選択して OK しない限り上書きされないので、
メッセージと動作が食い違っていることになり、混乱します。

再現手順1

  1. デスクトップに dummy.log ファイルを作成します。
  2. Setup -> Log で Log ダイアログを出します。
  3. Filename の右側の「...」ボタンで Tera Term:Log ダイアログを出します。
  4. デスクトップに作成済の dummy.log ファイルを選択して「保存」します。
  5. 「名前を付けて保存の確認」ダイアログが表示されます。
  6. ファイルを指定後に Write mode 指定するつもりだったとしても上書きに同意しなければならず、混乱します。

再現手順2

  1. Setup -> Log で Log ダイアログを出します。
  2. 既存ファイルを指定し、Write mode で「Append」を選択します。
  3. Filename の右側の「...」ボタンで Tera Term:Log ダイアログを出します。
  4. 既存ファイルを選択して「保存」します。
  5. 「名前を付けて保存の確認」ダイアログが表示されます。
  6. Write mode 指定を Append にしてあっても上書きに同意しなければならず、混乱します。

期待の動作

「Tera Term: Log」ダイアログでファイルを選択する際は、
「名前を付けて保存の確認」ダイアログが表示されないことが妥当です。

@nmaya
Copy link
Member

nmaya commented Oct 20, 2024

提案ありがとうございます。
ソースを追いかけられていないのですが、この関数は "Setup - Save setup" のような「上書き確認が必要な GetSaveFileName()」の呼び出しに使われていることはありませんか?

@sempreff
Copy link
Contributor Author

呼び出し側で OVERWRITEPROMPT を未設定の場合のみ未設定になれば良いと考えています。
(そのようにしたつもりでしたが、誤っておりましたので修正いたしました)

@nmaya
Copy link
Member

nmaya commented Oct 20, 2024

わかりました。確認ありがとうございます。

nmaya added a commit that referenced this issue Oct 20, 2024
…tePrompt

avoid meaningless overwrite prompt when choosing log file
@nmaya nmaya added this to the 5.4 milestone Oct 22, 2024
@nmaya nmaya closed this as completed Oct 22, 2024
sempreff added a commit to sempreff/teraterm that referenced this issue Oct 24, 2024
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

No branches or pull requests

2 participants