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

regression: <C-Tab> keybinding no longer works #1358

Closed
sodiumjoe opened this issue Jun 8, 2018 · 6 comments
Closed

regression: <C-Tab> keybinding no longer works #1358

sodiumjoe opened this issue Jun 8, 2018 · 6 comments

Comments

@sodiumjoe
Copy link
Contributor

Which operating system does the issue occur on? macos 10.10.5

If on linux, are you using X11 or Wayland? n/a

the specific keybinding is:

  - { key: Tab,      mods: Control, chars: "\x01n",                      }

git bisect points at this commit, which is just an upgrade of glutin/winit.

I haven't really figured out whether it's an issue with an upstream breaking change or what. Running winit's window example, I can see the event come through on the keypress:

WindowEvent { window_id: WindowId(Id(140330957578608)), event: KeyboardInput { device_id: DeviceId(DeviceId), input: KeyboardInput { scancode: 48, state: Released, virtual_keycode: Some(Tab), modifiers: ModifiersState { shift: false, ctrl: true, alt: false, logo: false } } } }

A println here also does show the event coming through, so maybe it has something to do with config parsing or something. I'm going to continue investigating, but wanted to make an issue in case anyone has ideas.

@snoblenet
Copy link
Contributor

related to this? #1336

@sodiumjoe
Copy link
Contributor Author

sodiumjoe commented Aug 16, 2018

this was fixed in winit in rust-windowing/winit#629

@chrisduerr
Copy link
Member

Once winit's cargo version is updated I'll run cargo update on the #1403 PR so it will resolve this problem.

Thanks for the update @sodiumjoe.

@sodiumjoe
Copy link
Contributor Author

@chrisduerr I think we'll have to get it into glutin first, right?

@chrisduerr
Copy link
Member

chrisduerr commented Aug 16, 2018

@sodiumjoe Not if it's just a semver patch (or minor after 1.0). So if glutin says it just wants 0.6.0, then everything until 0.7.0 should be viable.

@sodiumjoe
Copy link
Contributor Author

oh great! hm, I guess I need to review my understanding of cargo

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

No branches or pull requests

3 participants