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

Update sv keymap to include neighbouring nordic characters #208

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nospam3159
Copy link

As supported on virtually all other Swedish keyboard keymaps.

@derfian
Copy link
Member

derfian commented Jan 2, 2018

The patch looks fine, and I was hoping to do a quick test and merge this, but I can't seem to write æøÆØ characters with this patch applied. No characters appear, although keyboard debug logging indicates that they're translated and sent to the server.

I'm confused. Maybe someone with more insight into the keymap/input side of rdesktop can help?

@nospam3159
Copy link
Author

Thanks for looking at and attempting to merge my patch!

For what it is worth, I have successfully been using this change since before I first posted it to rdesktop-devel in June. Yet only with rdesktop running on Linux connecting to VirtualBox hosts exposed through RDP. It is possible that I underestimate the complexity required to implement it properly for other environments.

Even though I lack the understanding of rdesktop internals, I could try to help debugging what happens in other setups. What kind of RDP server were you connecting to when it did not work @derfian ?

@derfian
Copy link
Member

derfian commented Jan 3, 2018

What kind of RDP server were you connecting to when it did not work @derfian ?

Windows 2016. I tried looking at the keyboard code in rdesktop, but it didn't alleviate my confusion.

@nospam3159
Copy link
Author

I've verified with a Windows server too now, and sure enough it does not work.

Did some more debugging and read the doc/keymapping.txt properly. Think I understand the issue and should have a new patch to post shortly.

@nospam3159
Copy link
Author

Having looked a few more minutes at the issue, I quite confident that the patch is correct as it is. It is however insufficient in delivering everything needed for Windows hosts. Applying the patch still improves functionality for us connecting from unix to unix.

Keystrokes are successfully sent if adding sequence-lines to the keymap file. For example adding "sequence oslash dead_diaeresis e" yields an ë for each press on ø, even on Windows. I can't figure out how to add a sequence that actually generates the desired characters though. Attempting combinations with dead_stroke, compose and Multi_key in them fails in different interesting ways. Maybe it is possible to fix, but maybe a fix requires patches on other levels.

I'm giving up at getting this to work with Windows.

As supported on virtually all other Swedish keyboard keymaps.
@nospam3159
Copy link
Author

Branch rebased to current master. No other changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants