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.

Sign up for GitHub

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

Jump to bottom

Send File ダイアログのオプションが記憶されない #380

Open
nmaya opened this issue Oct 20, 2024 · 10 comments
Open

Send File ダイアログのオプションが記憶されない #380

nmaya opened this issue Oct 20, 2024 · 10 comments
Assignees
@zmatsuo
Milestone
5.4

Comments

@nmaya
Copy link
Member

nmaya commented Oct 20, 2024

少なくとも "Send in same as way as Tera Term 4" は記憶されない。
記憶というのは「INIへの記憶」「INIへは記憶されないがインスタンスがある間は覚えている」の2種類ある。

  • このダイアログの設定値がどこまで記憶されているのか現状の調査
  • どうあるべきかの検討
  • 必要なら修正

origin: #375
@nmaya nmaya mentioned this issue Oct 20, 2024
Open
3 tasks
@zmatsuo
Copy link
Member

zmatsuo commented Nov 5, 2024 •
edited
Loading

INIへ記憶(ts構造体に保存→設定の保存でINIへ書き出しできる)するよう
修正するのがよさそうに思います。

実際に使用するUI(新しい接続ダイアログ等)で行った設定と、
設定ダイアログなどで行った設定は同じもので、
設定ダイアログなどで設定しなくても
設定-設定の保存で保存すれば保存されますね。

同じような例はいくつかあります

1

  • ファイル-ログ
  • 設定-その他の設定-logタブ

2

  • ファイル-新しい接続-新しい接続ダイアログのシリアルのポート
  • 設定-その他の設定-全般タブの標準ポート

3

  • ファイル-新しい接続-新しい接続ダイアログのTCPポート#(番号)
  • 設定-その他の設定-TCP/Iタブのポート#(番号)

実際に使用するUI(新しい接続ダイアログ等)で行う設定は、
ダイアログが出たときはINIファイル(ts構造体)の内容で表示されるが
"ok"ボタンが押されたときts構造体に書き戻さないのが
良いのではないか?と 最初は思っていました。

今はこの動作は意図したものなんだろうと感じています。

@nmaya
Copy link
Member Author

nmaya commented Nov 6, 2024

Tera Term 4 では Binary をチェックすると ts 構造体に記憶され、明示的に INI に保存したときに保存されます。
Tera Term 5 ではチェックを入れて送信し、再度開いてもチェックが入っていないので、ts 構造体に記憶していないようです。

INIへ記憶(ts構造体に保存→設定の保存でINIへ書き出しできる)するよう修正する

これも「以前と同じ動作をするよう修正する」以外ないと思います。

Tera Term 4 ts Tera Term 4 INI Tera Term 5 現状 ts 修正案
Same as 4 - - 記憶しない ※2
Binary 記憶する TransBin 記憶しない ※1
delay type - - 記憶しない ※2
send size - - 記憶しない ※2
delay time - - 記憶しない ※2

※1 は上記の「以前と同じ動作をするよう修正する」です。
※2 項目への案は「記憶するように変更する = ts構造体への追加」と「INI への設定項目追加」です。

同じような例はいくつかあります

例を挙げられたのは従来からの Tera Term の動作なので、何の疑問もなく「そういうもの」です。

"ok"ボタンが押されたときts構造体に書き戻さないのが良いのではないか?と 最初は思っていました。

この感覚のほうに強い違和感を覚えます。

「自分よりも Tera Term の既存の動作に慣れていて、それを変えられては不便になる」ユーザがいること、自分の感覚だけで動作を変えるとこのように困るユーザがいることに留意してください。

※1だけでなく、※2の修正をする（#375 の「ダイアログをスキップするとき」のために必要）という方向でよいですか？

@zmatsuo
Copy link
Member

zmatsuo commented Nov 6, 2024

"ok"ボタンが押されたときts構造体に書き戻さないのが良いのではないか?と 最初は思っていました。

この感覚のほうに強い違和感を覚えます。

そうなのかな、といろいろUIを見て回ったところ、
ダイアログ設定はOKが押されると記憶されて
次に表示したとき前回のものが設定されているのが一般的のようです。
かなり間違っていました。失礼しました。

となると、ダイアログなどでチェックボックスなど設定があったら
(インスタンスがある間(exeが起動している間)だけ有効なのか
INIへ保管するかは置いといても)
すべて記憶すべきですね。

※1だけでなく、※2の修正をする（#375 の「ダイアログをスキップするとき」のために必要）という方向でよいですか？

とてもよさそうです。

ts構造体にどんどん追加するのがちょっと気が引けますが
いまは代替案がないです。

@nmaya nmaya mentioned this issue Nov 8, 2024
Open
@nmaya nmaya mentioned this issue Nov 23, 2024
Open
@zmatsuo zmatsuo self-assigned this Nov 24, 2024
zmatsuo added a commit that referenced this issue Nov 26, 2024
@zmatsuo
Send Fileダイアログのオプションを記憶 #380
0c7b571 
- Send Fileダイアログ
  - send size を dropdownlist から dropdown に変更した
  - コントロールを enable/disable するようにした
  - help/cancel/ok ボタンの順序を修正
  - ファイル名長制限をなくした(MAX_PATH を使用しなくなった)
- ini ファイルにファイル送信のパラメータを追加
@zmatsuo zmatsuo mentioned this issue Nov 26, 2024
Closed
3 tasks
@zmatsuo
Copy link
Member

zmatsuo commented Nov 26, 2024 •
edited
Loading

TERATERM.INI に保存するようにしました。

バイナリを作りました。
https://ci.appveyor.com/project/teraterm/github-snapshot/builds/51061618/artifacts

zmatsuo added a commit that referenced this issue Nov 28, 2024
@zmatsuo
Send Fileダイアログを修正 #380
9799f7d 
- 終了時、開放済みメモリを参照しないよう修正
- "Tera Term 4 と同じ方法で送信" チェックボタンを押した後の動作が不定だったので修正
@zmatsuo
Copy link
Member

zmatsuo commented Nov 28, 2024

いったん main にマージしました。
https://ci.appveyor.com/project/teraterm/github-main/builds/51076139/artifacts

@nmaya
Copy link
Member Author

nmaya commented Dec 2, 2024

修正ありがとうございます。

SendfileDelayTick

INI ファイルに保存したとき、常に 2 という値が保存されるようです。

send size を dropdownlist から dropdown に変更した

これはこの issue の範囲、ならびに変更履歴「ファイル送信オプションをTERATERM.INIに保存するようにした。」を超えることなので、別の issue にする内容だと思います。

@nmaya
Copy link
Member Author

nmaya commented Dec 2, 2024 •
edited
Loading

SendfileSkipOptionDialog

#375 で扱われている内容が、この issue のコミット 0c7b571 に含まれています。
そのことはコミットメッセージにも変更履歴にも書かれていません。

zmatsuo added a commit that referenced this issue Dec 7, 2024
@zmatsuo
TERATERM.INI の SendfileDelayTick が正しく保存されるよう修正 #380
f5e5b81 
- TERATERM.INI の SendfileDelayTick が正しく保存されるよう修正
  - 常に2が保存されていた
- send size コントロールの修正を元に戻した
- 元になる修正は 0c7b571 で行われ、
  #375 と #380 の内容が含まれている
@zmatsuo zmatsuo mentioned this issue Dec 7, 2024
Merged
zmatsuo added a commit that referenced this issue Dec 7, 2024
@zmatsuo
Merge pull request #397 from TeraTermProject/380-save_senddialog_options
5a261e0 
TERATERM.INI の SendfileDelayTick が正しく保存されるよう修正 #380
@zmatsuo
Copy link
Member

zmatsuo commented Dec 7, 2024

修正しました。5a261e012fbe7ecf20d1ee69154fdaebc538d3c6

SendfileDelayTick

INI ファイルに保存したとき、常に 2 という値が保存されるようです。

修正しました。

send size を dropdownlist から dropdown に変更した

これはこの issue の範囲、ならびに変更履歴「ファイル送信オプションをTERATERM.INIに保存するようにした。」を超えることなので、別の issue にする内容だと思います。

元に戻しました。この修正は誤りです。
CBS_SORT ははずすほうがよさそうです。
ちょっと考えます。

zmatsuo added a commit that referenced this issue Dec 7, 2024
@zmatsuo
File - Send file についてマニュアル追記 #375
b58a28d 
- 改版履歴追記
- CTRL + File - Send file... について追記
- ソースにコメント追記
- Cancel時にファイル名にNULLが入るよう修正
- 元になる修正は 0c7b571 で行われ、
  #375 と #380 の内容が含まれている
@nmaya
Copy link
Member Author

nmaya commented Dec 18, 2024

修正ありがとうございます。

zmatsuo added a commit that referenced this issue Dec 21, 2024
@zmatsuo
ファイル送信ダイアログを改善 #380
f0ce2cb 
- 数値を入力するエディットボックスで、入力を数字のみに限定した
- ドロップダウンをソートなしに変更
- ダイアログのボタンの並びを OK Cancel Help に変更
  - 変更前は Help Cancel OK
zmatsuo added a commit that referenced this issue Dec 21, 2024
@zmatsuo
ファイル送信ダイアログを改善 #380
c1e20db 
- 数値を入力するエディットボックスで、入力を数字のみに限定した
- ドロップダウンをソートなしに変更
- ダイアログのボタンの並びを OK Cancel Help に変更
  - 変更前は Help Cancel OK
zmatsuo added a commit that referenced this issue Dec 21, 2024
@zmatsuo
ファイル送信ダイアログを改善 #380
d1d0025 
- 数値を入力するエディットボックスで、入力を数字のみに限定した
- ドロップダウンをソートなしに変更
- ダイアログのボタンの並びを OK Cancel Help に変更
  - 変更前は Help Cancel OK
@zmatsuo
Copy link
Member

zmatsuo commented Dec 21, 2024

修正しました。 d1d0025 です。

@nmaya nmaya mentioned this issue Dec 22, 2024
Open
@nmaya nmaya added this to the 5.4 milestone Dec 22, 2024
zmatsuo added a commit that referenced this issue Dec 23, 2024
@zmatsuo
ダイアログのボタンの並び修正をrevert #380
1c032ba 
- 他のブランチで修正を行っているため
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zmatsuo zmatsuo

Labels
None yet
Projects
None yet
Milestone
5.4
Development

No branches or pull requests
2 participants
@nmaya @zmatsuo