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

Can't enter some characters. Without @ I can't login to my remotes. #52

Closed
harakiwi opened this issue Apr 2, 2023 · 15 comments · Fixed by #75
Closed

Can't enter some characters. Without @ I can't login to my remotes. #52

harakiwi opened this issue Apr 2, 2023 · 15 comments · Fixed by #75
Milestone

Comments

@harakiwi
Copy link

harakiwi commented Apr 2, 2023

4.8.27
installed using winget

tested on Windows 10 Pro (set to German, Germany)
and a Win 10 (VM) set to English

some other symbols I can't type: € Ä Ö Ü ß \ §

I can't even copy/paste them what makes me think bug.
Do I have to change something on my end or is this a bug?

I tried to search that problem but I might not know the correct term so Id didn't find anything useful.

@adamyg
Copy link
Owner

adamyg commented Apr 9, 2023

output is wide-char/utf-8, whereas input is currently not.
keyboard handling internals aren't Unicode safe, need to emulate utf-8

@adamyg adamyg added this to the build-229 milestone Sep 11, 2023
@buzzqw
Copy link

buzzqw commented Jan 14, 2024

same problem, following...

@adamyg
Copy link
Owner

adamyg commented Jan 16, 2024

build-229 released, utf-8 input enabled

@adamyg adamyg closed this as completed Jan 16, 2024
@buzzqw
Copy link

buzzqw commented Jan 18, 2024

with version 229 i am unable to input "@" for remote session command
even copy/paste will trim it away

with 229 version i can input ùàò characters, BUT not @

@adamyg
Copy link
Owner

adamyg commented Jan 18, 2024

ShellLink
Please confirm dialog to which you are referring? (screen shot if possible)
Confirmed that none of "FTP", "SFTP", or "SSH" popups nor command line have issues with @ input.

WIn10 Pro (ita), guessing locale related: code-page? font?
Code page 1012, for example doesn't have an "@"

@buzzqw
Copy link

buzzqw commented Jan 18, 2024

both left and right plane, FTP, SHELL, SFTP unable to input @ . neither able to paste "@"
other special characters are inputable |!"£$%&/()è+òàù§°ç
win10 pro (ita)

immagine

"@" not avaiable

immagine

@adamyg
Copy link
Owner

adamyg commented Jan 18, 2024

How are you exec'ing: directly, power-shell, console-legacy, console-tty or other env?
Shall attempt to emulate.

@adamyg adamyg reopened this Jan 18, 2024
@buzzqw
Copy link

buzzqw commented Jan 18, 2024

typing MC directly form CMD or POWERSHELL -> no @
clicling on mc.exe from explorer > no @

but from CMDER, but only with ALT+64 , is working!

immagine

it's working even with "Alternative font" disabled

@adamyg
Copy link
Owner

adamyg commented Jan 23, 2024

Not able to reproduce issue,

  • Confirm Input=UTF-8 (Display Bits)
  • Console Font in use
  • VM, Latest Windows build ?

image

@adamyg
Copy link
Owner

adamyg commented Feb 10, 2024

KB layout in use?

@buzzqw
Copy link

buzzqw commented Feb 11, 2024

onfirm Input=UTF-8 (Display Bits) > yes, 8 bit
Console Font in use > tested with lucida and consolas (and courier new..)
VM, Latest Windows build ? > latest windows home build, fully update, no VM
KB layout in use? Italian

BHH

@harakiwi
Copy link
Author

can confirm. Still a problem on my end too.

4.8.30 - build 229 - downloaded the ZIP, extracted the whole folder and run the exe. No changes to the config afaik.

Alt Gr + Q does not input @
copy & paste (right click and CTRL+V) does not input @ to mc at any point (the console $ line and the SFTP login popup)
Alt + 64 on Numpad does not input the symbol

German Win 11 Pro 22631.3007
German Keyboard layout.
real USB kb and virtual kb both tried.

@adamyg
Copy link
Owner

adamyg commented Feb 11, 2024

AltGr translation issue

@harakiwi
Copy link
Author

⇮ AltGr (Alternate Graphic)

Same as CTRL + ALT

@adamyg adamyg modified the milestones: build-229, build-230 Feb 27, 2024
@adamyg adamyg mentioned this issue Mar 30, 2024
@adamyg adamyg linked a pull request Mar 30, 2024 that will close this issue
adamyg added a commit that referenced this issue Mar 31, 2024
build-230:
* mc merge 4.8.31 (#70)
* mbedtls 3.5.2 (#67)
* AltGr detection (#71, #52)
* mcdiff integration (#74)
* cd/cdpath (#72)
@buzzqw
Copy link

buzzqw commented Mar 31, 2024

confirm fixed with 230 build!
thanks

BHH

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 a pull request may close this issue.

3 participants