screenrecorder icon indicating copy to clipboard operation
screenrecorder copied to clipboard

Integrating Screenkey for key capturing

Open peteruithoven opened this issue 6 years ago • 10 comments
trafficstars

Fixing: #26

peteruithoven avatar Apr 24 '19 21:04 peteruithoven

@Mohelm97 what do you think about this idea? Would it be an issue to include screenkey as a default dependency?

peteruithoven avatar Apr 24 '19 21:04 peteruithoven

@peteruithoven Well, that was soo fast :D Thanks a lot mate, I think it's a good idea until we write something else, I will test it tomorrow and see how it looks like <3

mohelm97 avatar Apr 24 '19 21:04 mohelm97

@peteruithoven It's awesome, and I don't know why I need to unset XMODIFIERS to make it works, see this issue https://github.com/wavexx/screenkey/issues/36, and what do you think about showing only modifier sequences?

mohelm97 avatar Apr 25 '19 22:04 mohelm97

I still need to do that as well, I totally forgot about that issue I just opened a new question on stack exchange: https://elementaryos.stackexchange.com/questions/18810/running-screenkey-on-elementary-os

One of the biggest risks in integrating this into Screen Recorder is that we might get a lot of screenkey related issues reported to us. Like this Cannot initialize input method issue.

peteruithoven avatar Apr 25 '19 22:04 peteruithoven

I've reopened and updated this old issue of mine: https://github.com/elementary/gala/issues/27

peteruithoven avatar Apr 25 '19 23:04 peteruithoven

I should try overriding environment variables for screenkey by using SubprocessLauncher.

peteruithoven avatar May 14 '19 10:05 peteruithoven

Temporary workaround:

unset XMODIFIERS
unset GTK_IM_MODULES
unset QT_IM_MODULES
com.github.mohelm97.screenrecorder

peteruithoven avatar Jun 13 '19 23:06 peteruithoven

While reading other issues this comment from @ryonakano caught my attention:

elementary OS uses IBus out of the box, but it does not start IBus daemon.

https://github.com/elementary/switchboard-plug-keyboard/issues/231#issuecomment-517914033

Looks like screenkey works just file after starting: ibus-daemon -drx.

peteruithoven avatar Aug 04 '19 19:08 peteruithoven

@peteruithoven So instead of unsetting stuff, we can run the ibus daemon? cool I'm working now (offline) on an advance options screen so we can add the screenkey, changing the background, circling the cursor, and the gif encoding in it, so get ready :D

mohelm97 avatar Aug 04 '19 20:08 mohelm97

I personally think running IBus Daemon should be handled by the OS itself: the OS should start IBus Daemon automatically even in any language. As far as I confirmed on Ubuntu 18.04 (en_US) Ubuntu does this.

ryonakano avatar Aug 06 '19 10:08 ryonakano