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

wait until textEntry is available before opening connections #130

Open
wants to merge 1 commit into
base: 0.x
Choose a base branch
from

Conversation

kimmoli
Copy link

@kimmoli kimmoli commented May 9, 2017

When accepting welcome page, connections are opened. If connections have commands to send, they are sent with currentPage.textEntry.sendLines() function.
If connections are opened quickly enough, currentPage is still welcome page, and there is no textEntry object, thus sending network commands fail.
this will open connections only after currentPage is such that has textEntry object.

@Venemo
Copy link
Contributor

Venemo commented Jun 8, 2017

Sorry @kimmoli for taking so long to respond. There is a proper solution to this without needing a Timer. At the weekend I'll get around to installing the latest Sailfish SDK and take an in-depth look at the code and deal with the problem.

@kimmoli
Copy link
Author

kimmoli commented Jun 8, 2017

Yes, please find a proper solution. (I have now used my workaround for a month, and works like a charm.)

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.

None yet

2 participants