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

TTXKanjiMenu does not work with "Hide title bar" #265

Closed
sempreff opened this issue Jul 28, 2024 · 5 comments
Closed

TTXKanjiMenu does not work with "Hide title bar" #265

sempreff opened this issue Jul 28, 2024 · 5 comments
Assignees
@zmatsuo
Labels
bug Something isn't working
Milestone
5.3

Comments

@sempreff
Copy link
Contributor

概要

メニューの KanjiCode から文字コードを変更あるいは再設定すると
文字入力ができなくなります。

再現方法

  • スナップショットの最新 1.0.38のポータブル版 zipを取得し、展開します。
  • portable.ini を配置します。
  • ttermpro.exe をエクスプローラでダブルクリックして起動し、普段使っているホストに ssh で接続してログインします。
  • Setup - Window で Hide title bar のチェックを入れます。
  • Control-左クリックでメニューを出し、KanjiCode から Recv/Send: UTF-8 を選択します。（どれを選択しても事象は同じです）
  • 文字入力ができなくなります。
    • 再度 Control-左クリックでメニューを出し、KanjiCode を辿ると、選択が消失しています。

運用回避

  • Control-左クリックでメニューを出し、Setup - Terminal を出して OK すると回復します。

関連していると思われる事象

  • 上記 Hide title bar で TTXKanjiMenu の選択が消失している状態まで操作します。
  • Additional Settings の Coding タブを出して OK します。
  • メニューから KanjiCode が消失することがあります。
@sempreff
Copy link
Contributor Author

  • タイトルバーを残して Hide menu bar しても事象は発生します。
  • commit 9babd50 を戻すと、事象は発生しなくなります。

@sempreff sempreff mentioned this issue Jul 30, 2024
Closed
@sempreff
Copy link
Contributor Author

GetMenuItemInfoW が期待どおりに動作しないことがあるようです。
#266 は、小手先の回避案です。もちろんもっと良いやり方があると思いますが、まずは「文字入力ができない」という致命的な事象を逃れたい思いです。

@zmatsuo zmatsuo added the bug Something isn't working label Jul 30, 2024
@zmatsuo zmatsuo self-assigned this Jul 30, 2024
zmatsuo added a commit that referenced this issue Jul 30, 2024
@zmatsuo
漢字コードが正しく選択できない場合があったので修正 #265
a5ab61c 
- Hide title bar や Hide menu bar のとき
  - メニューがポップアップメニューになる
  - ポップアップメニューではメニューの項目を選択した直後メニューは破棄される
- メニュー項目のdwItemDataに入れたデータを利用していたが正しく読み込めなかった
- メニューが破棄されていても正しく動作するよう修正
zmatsuo added a commit that referenced this issue Jul 30, 2024
@zmatsuo
漢字コードが正しく選択できない場合があったので修正 #265
a4eb436 
- Hide title bar や Hide menu bar のとき
  - メニューがポップアップメニューになる
  - ポップアップメニューではメニューの項目を選択した直後メニューは破棄される
- メニュー項目のdwItemDataに入れたデータを利用していたが正しく読み込めなかった
- メニューが破棄されていても正しく動作するよう修正
@zmatsuo
Copy link
Member

zmatsuo commented Jul 30, 2024

レポート、PR ありがとうございます。

根本対策を行いました。
PRのtypo修正も取り込みました。
試してみていただけないでしょうか

ちょっと余談、
近い未来 ts->Language をなくそうと考えています。
この変数がなくなれば制限がなくなるのになと思う部分があります。

そのときkanjimenu は

 [日本語▼]
 |韓国語  |

みたいなドロップボックスがある
設定ダイアログを持つことになるかなと考えています。

@sempreff
Copy link
Contributor Author

ご対応ありがとうございます。
ローカルで確認してみたところ、動作良好なようです。

余談についてですが、TTX KanjiMenu をダイアログに作り替えるということでしょうか？
https://naoh16.seesaa.net/article/35220277.html
で言及されているように「メニューからワンタッチで指定できること」のために生まれたものなので、
ダイアログにしてしまうのはちょっと違うかな、と感じました。

zmatsuo added a commit that referenced this issue Jul 31, 2024
@zmatsuo
Merge pull request #268 from TeraTermProject/265-kanjimenu
0ca791e 
漢字コードが正しく選択できない場合があったので修正 #265
@zmatsuo
Copy link
Member

zmatsuo commented Aug 1, 2024

動作良好とのことで、よかったです。
クローズします。

切り替える言語(表示するメニュー)を選択できるようにしたいなと考えています。

✔ 日本語
   韓国語

みたいなサブメニューでも良いかなと 考えています。

もうビルドされていると思いますが appveyor でもビルドしました。
https://ci.appveyor.com/project/teraterm/github-main/builds/50328707/artifacts

@zmatsuo zmatsuo closed this as completed Aug 1, 2024
@nmaya nmaya added this to the 5.3 milestone Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zmatsuo zmatsuo

Labels
bug Something isn't working
Projects
None yet
Milestone
5.3
Development

No branches or pull requests
3 participants
@nmaya @zmatsuo @sempreff