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

Polish language support #14241

Closed
wojtekmaj opened this issue Oct 24, 2016 · 30 comments
Closed

Polish language support #14241

wojtekmaj opened this issue Oct 24, 2016 · 30 comments
Assignees
Labels
feature-request Request for new features or functionality l10n-platform Localization platform issues (not wrong translations)

Comments

@wojtekmaj
Copy link

Please add Polish support and Polish keyboard key bindings (Polish keyboard does not use alt+[letter] for shortcuts as this conflicts with inputting letters like ążółćń).

I'd be happy to help with translations in case you are open to contributions via services like getlocalization.

@roblourens
Copy link
Member

I know we've worked a lot on international keyboards, so can you clarify exactly what problem you're having with the polish keyboard?

@roblourens roblourens added info-needed Issue requires more information from poster keybindings VS Code keybinding issues labels Oct 24, 2016
@wojtekmaj
Copy link
Author

@roblourens, there's a lot of trouble when using Polish keyboard and trying to enter Polish diacritic signs. I believe that US keyboard copies some shortcuts like ctrl+s to alt key as well; the thing is, alt+[aceszxnlo] are used for entering letters: ąćęśżźńłó respectively. This results in a conflict and inability to enter said letters.

This is a very common problem with American software in Poland, even Windows 10 Mail and Store app were having it for a while.

The problem with keybindings has already been raised by issues: #14234 #13198 #11090 #9205

Please mind that this request is for Polish language support in general, including the UI, which was not mentioned before on GitHub, I believe.

@Tyriar Tyriar assigned dbaeumer and unassigned Tyriar Oct 24, 2016
@Tyriar Tyriar added l10n-platform Localization platform issues (not wrong translations) and removed keybindings VS Code keybinding issues info-needed Issue requires more information from poster labels Oct 24, 2016
@Tyriar
Copy link
Member

Tyriar commented Oct 24, 2016

Assigning to @dbaeumer for i18n as the keybindings issues seem to be closed or tracked elsewhere.

@dbaeumer dbaeumer added the info-needed Issue requires more information from poster label Oct 25, 2016
@dbaeumer
Copy link
Member

@wojtekmaj just to be clear: when pressing alt+e for example VS Code takes you to the main menu bar and opens the edit menu. VS Code doesn't define any keyboard shortcut alt+e for a specific command.

@wojtekmaj
Copy link
Author

Actually, that's not correct for my case.

Pressing alt+e creates letter ę for me :) And it works in VSCode as well. The only one not working in VSCode as of now is alt+l that should make the letter ł. Interestingly, capital Ł (shift+alt+l) works fine.

@dbaeumer dbaeumer assigned alexdima and unassigned dbaeumer Oct 25, 2016
@dbaeumer dbaeumer added editor and removed l10n-platform Localization platform issues (not wrong translations) info-needed Issue requires more information from poster labels Oct 25, 2016
@dbaeumer
Copy link
Member

OK thanks. Then this has nothing to do with i18n.
@alexandrudima I checked and by default we don't define any keyboard shortcut for alt+l

@wojtekmaj
Copy link
Author

wojtekmaj commented Oct 25, 2016

But it is about i18n! It's about adding Polish UI language! It's just the discussion that went towards the issue that is already in 4 other tickets.

@dbaeumer dbaeumer removed the editor label Oct 26, 2016
@dbaeumer dbaeumer assigned chrisdias and unassigned alexdima Oct 26, 2016
@dbaeumer dbaeumer added l10n-platform Localization platform issues (not wrong translations) feature-request Request for new features or functionality labels Oct 26, 2016
@dbaeumer
Copy link
Member

@wojtekmaj there are different things here: having the UI strings in Polish is indeed a i18n issue. But translating it to Polish will not fix the problem that if you press alt+I you don't get the correct input letter.

I will move the item back to i18n then. Please keep discussing the letter input in the other issues then.

@alexdima
Copy link
Member

@wojtekmaj re: keyboard layouts.

I believe it is AltGr+letter that introduces special characters (i.e. it is usually printed on the key as AltGr, otherwise it is the Alt located on the right hand side of the spacebar). Alt+letter will not produce any characters on the Polish keyboard layout.

https://www.microsoft.com/resources/msdn/goglobal/keyboards/kbdpl1.html

In VSCode (driven by Chromium) AltGr+letter is, AFAIK detected as ctrl+alt+letter.

On Windows, we don't ship with default keybindings for ctrl+alt+letter especially because ctrl+alt+ is AltGr+ and used for special characters on different keyboard layouts. Can you please tell us what keyboard combinations (only coming from the default installation -- i.e. code --disable-extensions) do not work for Polish out of the box?

@wojtekmaj
Copy link
Author

wojtekmaj commented Oct 26, 2016

That is correct, I meant AltGr for the whole time. AltGr + l does not produce the letter ł. But let's keep that discussion in related issues that has already been created:
#14234 #13198 #11090 #9205 (especially the last one, that says specifically about this specific letter)
and leave this discussion for i18n. I thought it was connected somehow, but I caused confusion becuase of it. Sorry!

@alexdima
Copy link
Member

@wojtekmaj Agreed, please also see the resolution on #9205:

#9205 (comment)

"I have extension "Bookmarks Alessandro Fragnani" and the problem is with that extension.

Please create an issue on that extension. Here is the root cause -- https://github.com/alefragnani/vscode-bookmarks/blob/master/package.json#L112

@danyeh
Copy link
Contributor

danyeh commented May 3, 2017

@wojtekmaj, in case you are interested in contributing translation. Here is the instruction. http://aka.ms/vscodeloc. Discussion is still ongoing about what % of translation need to reach before integrate to VS Code product.

@wojtekmaj
Copy link
Author

@danyeh Sweet! Just what I dreamed of. Thanks!

@replay111
Copy link

Hi,

I've just installed new version of VSCode (1.14.0 for Mac), and I have same problem as @wojtekmaj. I removed keybindings for alt+z (for Polish location still is binded to wordwrap #13198) but in integrated console I cant type alt+l => ł . I removed all extension from ~/.vscode and before installation I removed ~/Library/Application Support/Code so my installation is clean.
In iTerm alt+l works fine (terminal shows "ł").

@replay111
Copy link

replay111 commented Jul 14, 2017

Great, now when I removed binding to alt+z, word wrap is not working from menubar (view -> toggle wordwrap) in text editor :(

@n074v41l4bl34u
Copy link

n074v41l4bl34u commented Jul 29, 2017

The problem is that in Polish keyboard layout, 'right alt' + letter: [acelnoszx] produces a Polish diactric character i.e. [acelnoszx] -> [ąćęłńóśżź].

VS Code has no concept of left/right alt key - at least on Windows, hence many 'alt + letter' combinations produce Polish special letters instead of activating commands.

It also applies to to more complex keybindings like Ctrl + Alt + letter and it effectively renders using VSCode keyshortcuts useless.

There is no issue in full Visual Studio, so I do not see why it is not working as expected in VS Code.

Can you fix it somehow?

@piotrku
Copy link

piotrku commented Nov 28, 2017

Looks like this (RIGHT)ALT+l problem cannot be solved for a year now :(
Hey that can't be so difficult!
I'd gladly unbind this shortcut only if it only was possible.

@mrrau
Copy link

mrrau commented Jan 3, 2018

Fix for this should be simple: when keyboard setting in OS is "Polish (programmers) keyboard", don't trigger Ctrl+Alt shortcuts with AlrtGr (right Alt) key.

For now the only real solution is to unset/change conflicting keybindings.

@piotrku
Copy link

piotrku commented Jan 3, 2018

@mrrau the problem is that you cannot unset this binding, because it doesn't appear in the bindings configuration table :)

@mrrau
Copy link

mrrau commented Jan 3, 2018

@piotrku
Go to "Preferences: Open keyboard shortcuts".
Click "For advanced customizations open and edit keybindings.json"
Add empty keybinding for Polish letters,
{
"key": "ctrl+alt+l",
},
it will block editor and extensions to use it. Works for me.

Anyway, I think handling AltGr key can be an Electron issue.

@piotrku
Copy link

piotrku commented Jan 4, 2018

@mrrau - you're right "ctrl+alt+l" does the trick when using "Polish (programmers) keyboard" !

@hash
Copy link

hash commented Mar 14, 2018

In Version 1.21.0
Alt - l is bound to toggleFindInSelection
when removed, nothing happens on Alt - l

@pociej
Copy link

pociej commented May 8, 2018

alt + z doesn't work at all

@piotrku
Copy link

piotrku commented May 8, 2018

In my case the problem was created by on of the VSC extensions - it took over the Polish letter shortcut, so as a first step in solving the problem I'd try to run VSC with all extensions removed to see if the problem still exists.

@amcsi
Copy link

amcsi commented May 23, 2018

I see there's already an open issue here. Such a shame this has been an issue for at least 1.5 years :/ I don't have a lot of high hopes of the creator fixing this basic international issue.

This is the issue I opened: #50341
I can see another old one that's sadly been dismissed without much thought with flawed reasoning: #41225

@chrisdias
Copy link
Member

I'm resolving this as a duplicate of #50341. if that's not the case, please let me know and i'll reopen and assign accordingly. cc @alexandrudima

@wojtekmaj
Copy link
Author

Initial request was about adding Polish UI language, and along with that support for altGr keys and such, so that's not really the case.

@piotrku
Copy link

piotrku commented Sep 26, 2018

altGr keys work fine for me now, aren't for you?

@wojtekmaj
Copy link
Author

Yep. But the interface is still in English.

@danyeh
Copy link
Contributor

danyeh commented Sep 28, 2018

@wojtekmaj , Polish language was enabled in community localization tool - Transifex. Translation completion rate has not reached the threshold (50%) that we setup to start to publish beta version of the Polish language pack. It will be great if you can help to nurture Polish community to contribute more. We certainly wish can have Polish language pack as early as possible.

VS Code community localization project info is here: https://aka.ms/vscodeloc.
Language request process info is here: https://github.com/Microsoft/Localization/wiki/How-to-request-a-new-UI-language-for-VS-Code-project
Current Polish completion rate is around 23% or so.
image
@aldonetti , FYI.

@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality l10n-platform Localization platform issues (not wrong translations)
Projects
None yet
Development

No branches or pull requests