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

Focus Atom application when text-synced from chrome browser #19

Merged
merged 1 commit into from
Jul 27, 2017

Conversation

t9md
Copy link
Contributor

@t9md t9md commented Apr 19, 2017

This is we discussed in danhper/atomic-chrome#28

But it reveals it's achievable only by atomic-chrome-atom side.

And I could see slight activation time reduction from 40ms/avg to 30ms/avg by deferred require for ws-handler.

I can make it config option (e.g. automaticallyActivateAtomWindow ), but IMO this is all user's expectation so didn't do that.

How do you think?

@danhper
Copy link
Owner

danhper commented Apr 27, 2017

Thanks for the PR, and sorry for the delay, I will take a look during the weekend!

@t9md
Copy link
Contributor Author

t9md commented Apr 30, 2017

I'm working for improving package activation time(faster package activation).
And noticed atomic-chrome-atom always appear on the list of Timecop: View as slow package.
Here is my workaround for not interfering startup of Atom.

https://github.com/t9md/atomic-chrome-atom/blob/experiment/lib/atomic-chrome.coffee

If you are OK for that I can send separate PR for that. Or hope you apply this change by yourself.

@danhper danhper merged commit 4608ef1 into danhper:master Jul 27, 2017
@danhper
Copy link
Owner

danhper commented Jul 27, 2017

Very sorry for the delay, I completely forgot about this PR 😓
Anyway, thank you very much!

@KDPRoss KDPRoss mentioned this pull request Jan 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants