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

GTK_IM_MODULE=cedilla doesn't work to get "ç" #670

Closed
edersondisouza opened this issue Nov 5, 2022 · 5 comments
Closed

GTK_IM_MODULE=cedilla doesn't work to get "ç" #670

edersondisouza opened this issue Nov 5, 2022 · 5 comments
Labels
question Further information is requested

Comments

@edersondisouza
Copy link

Describe the bug
Can't type <dead_acute><c> to get "ç" on us intl keyboard. I use GTK_IM_MODULE=cedilla to achieve that (otherwise I'd get a "ć").

To Reproduce
Have a US intl keyboard configured. Use GTK_IM_MODULE=cedilla to get "ç" instead of "ć" when typing <dead_acute><c>. Type <dead_acute><c>.

Expected behavior
Get a "ç".

Desktop (please complete the following information):

  • OS or Linux Distribution: Arch Linux
  • Display Technology: X11

Additional context
It seems 1e89400 (fix for #78) causes this. There's also another aspect for this: if I start any application from terminator (such as a browser with some command line parameters), it also won't get the GTK_IM_MODULE env var and cedilla will be broken. For this reason, I'm not sure if terminator can go removing env vars like in this fix.

@mattrose
Copy link
Member

mattrose commented Nov 8, 2022

If I created a test branch for you would you feel comfortable pulling down code and running it?

I could try and reproduce your situation, but I think it's better if I fix what I think is the problem, and have you run a few tests to make sure it works.

@mattrose mattrose added the question Further information is requested label Nov 8, 2022
@edersondisouza
Copy link
Author

If I created a test branch for you would you feel comfortable pulling down code and running it?

Yes, it shouldn't be a problem.

@mattrose
Copy link
Member

Can you look at the code in #674 and test out to see if it fixes the problem?

@edersondisouza
Copy link
Author

I confirm that the code in #674 works for me.

@mattrose
Copy link
Member

good, code is merged into master now. If I don't put out 2.1.3 by January poke me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants