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

サクラエディタではアクセスキーに大文字/小文字の区別がありません。 #624

Closed
m-tmatma opened this issue Nov 18, 2018 · 7 comments
Milestone

Comments

@m-tmatma
Copy link
Member

表示テキストが大文字表記になってるので、システムから送られてくる文字を大文字に変換して当て込んでいます。
このままだと P + P でpsを開く、P + ↓ + Pで管理者psを開くになります。

Originally posted by @berryzplus in #623

@m-tmatma
Copy link
Member Author

コマンドプロンプトを開く、コマンドでも同じ (#603, #618)

@berryzplus
Copy link
Contributor

high dpi 対応のついでで検討したcmenudrawerの改修検討案があるので、そのうち出します。

cmenudrawer::onmenucharという関数がその部分。
冒頭で謎の変換処理と大文字化をやっています。

// Oct. 27, 2000 genta
if( 0 <= chUser && chUser < ' '){
chUser += '@';
}
else {
chUser = (TCHAR)_totupper( chUser );
}

文字コードが空白より前ってのは制御文字なので、if文の前半は謎です。
そして、半角アルファベットでない文字をtoupperしたときの挙動は保証されていません。

@k-takata
Copy link
Member

メニューのアクセスキーで大文字小文字を区別しないのはWindowsの仕様だと思いますが、何を問題としているのでしょうか。

@m-tmatma
Copy link
Member Author

キーがかぶっているの、およびかぶっているのを assert 等で検出できるようにしたいと思います。
コマンドプロンプトの方でも同じ問題があるので、それも一緒に検討したいです。

@ds14050
Copy link
Contributor

ds14050 commented Nov 19, 2018

自信がないのですがこれってメインメニューやコンテキストメニューの話ですか? キーがかぶっている場合はキーを押す毎にサイクリックに項目が選ばれるものではないのでしょうか。つまりキーかぶりは想定内。

@berryzplus
Copy link
Contributor

このままだと P + P でpsを開く、P + ↓ + Pで管理者psを開くになります。

これが間違ってるのかな。
正しくは P Enter でpsを開く、P P Enterで管理者psを開く。
ローカルでやってた改善案と挙動が混じった。。。_| ̄|○

@berryzplus
Copy link
Contributor

メニューのアクセスキーで大文字小文字を区別しないのはWindowsの仕様だと思いますが、何を問題としているのでしょうか。

これが結論ですね。閉じておきます。

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

4 participants