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

Keybinding are misrecorded (<Primary> instead of <Ctrl>) #138

Closed
noirbizarre opened this issue Mar 14, 2016 · 8 comments
Closed

Keybinding are misrecorded (<Primary> instead of <Ctrl>) #138

noirbizarre opened this issue Mar 14, 2016 · 8 comments
Labels

Comments

@noirbizarre
Copy link

When I try to record some keybinding, the <Ctrl> is misrecorded as <Primary> and I have to change it manually with dconf-editor.

I'm using Terminix 0.52.1 on Archlinux with Gnome 3.18 and fr keyboard layout.

@noirbizarre noirbizarre changed the title Keybinding are wrongly recorded (<Primary> instead of <Ctrl>) Keybinding are misrecorded (<Primary> instead of <Ctrl>) Mar 14, 2016
@gnunn1
Copy link
Owner

gnunn1 commented Mar 14, 2016

That's the way the shortcut in GTK works, Primary is the same as Ctrl so should work fine, at least it does for me. Are you having an issue with the shortcut not working?

@noirbizarre
Copy link
Author

Yes, I have to change them in dconf-editor in order to have them to work :/

But you are right, it might be an upstream GTK3 bug.

@gnunn1 gnunn1 added the bug label Mar 14, 2016
@gnunn1
Copy link
Owner

gnunn1 commented Mar 14, 2016

Thanks for the feedback, I'll have a look and see what I can find about it.

@egmontkob
Copy link

FYI: Shortcuts recorded as <Primary>foo do work for me. However, it's disturbingly inconsistent that the factory ones show as <Ctrl> whereas the custom ones as <Primary>.

I've no clue about Gtk+'s decision why it calls it Primary. I think it's a stupid thing to do. All computers have a Ctrl key and not a Primary key. In my opinion, the word "Primary" should never appear on the UI. I'm sure there are valid technical reasons behind the story, but users just don't care.

@gnunn1
Copy link
Owner

gnunn1 commented Mar 16, 2016

With respect to Primary being shown, I think there is something here that I can handle better, the AccelsGroup API has two similar operations, one returns the name of the accelerator while the other returns a label meant to be displayed to users. I need to look at this more and figure out to wire CellRendererAccel to use the label version for dispay but store the name. Gnome terminal seems to handle this just fine.

gnunn1 added a commit that referenced this issue Mar 17, 2016
@gnunn1
Copy link
Owner

gnunn1 commented Mar 17, 2016

I pushed a commit so the shortcuts are displayed with the label instead of the name. I can't reproduce your problem though with regards to the shortcuts not taking effect, I wonder if it's related to the fr keyboard. Does gnome terminal shortcuts work fine for you?

@noirbizarre
Copy link
Author

I just restested, and it works (wasn't working by the time I submitted the issue).
Sorry for the trouble, it seems to be fixed.

@gnunn1
Copy link
Owner

gnunn1 commented Mar 18, 2016

No trouble at all, thanks to this I fixed the shortcut preferences to get rid of Primary and it looks much better now so mission accomplished. :)

@gnunn1 gnunn1 closed this as completed Mar 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants