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

Fix opening of console #3471

Merged
merged 4 commits into from
Dec 23, 2017
Merged

Conversation

blink1073
Copy link
Contributor

Fixes #3465.

@blink1073 blink1073 added this to the Beta milestone Dec 23, 2017
@ellisonbg
Copy link
Contributor

I have verified that the this does fix the issue. But I see something else (also in master) that may be related. If you load JupyterLab with session state and a running Console, the title in the Console stays missing for quite a long while (about 10 seconds after lab restores):

screen shot 2017-12-22 at 10 52 46 pm

As a clue, only the last created one suffers from this issue:

screen shot 2017-12-22 at 10 55 32 pm

@blink1073
Copy link
Contributor Author

Hmm, I'm not seeing that behavior anymore, that's what this line was for:

https://github.com/jupyterlab/jupyterlab/pull/3471/files#diff-e58fee2f1b7602bb7d031ae677fe2008R96

@ellisonbg
Copy link
Contributor

Hmmm, let me rebuild and make sure...

@blink1073
Copy link
Contributor Author

Actually, looking at that again we need a fallback title for when we don't have the session name, updated to use Console until it is available.

@ellisonbg
Copy link
Contributor

ellisonbg commented Dec 23, 2017 via email

@blink1073
Copy link
Contributor Author

Because we don't have a name until the server interaction.

@ellisonbg
Copy link
Contributor

The fallback title does work now. But I am still confused. When I have 3 consoles open and I refresh the page, Console 1 and Console 2 get title almost immediately, but Console 3 takes 10 seconds between getting Console and Console 3. Is this the expected behavior?

@blink1073
Copy link
Contributor Author

No, it's not clear to me why that would be happening.

@ellisonbg
Copy link
Contributor

Can you reproduce that behavior?

Copy link
Member

@afshin afshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@blink1073 blink1073 changed the title Fix opening of console [WIP] Fix opening of console Dec 23, 2017
@blink1073
Copy link
Contributor Author

I can reproduce the issue on a page refresh, investigating.

@blink1073 blink1073 changed the title [WIP] Fix opening of console Fix opening of console Dec 23, 2017
@blink1073
Copy link
Contributor Author

Fixed the title issue.

Copy link
Contributor

@ellisonbg ellisonbg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All issues resolved and code looks good. Merging!

@ellisonbg ellisonbg merged commit 906b1fd into jupyterlab:master Dec 23, 2017
@blink1073 blink1073 deleted the console-command-focus branch December 30, 2017 02:10
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Aug 9, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Aug 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug pkg:console status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Console opened with command doesn't get focused/active
3 participants