-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Improvements to dialog UX #5659
Conversation
@seawolf42 and @ellisonbg what is the current status of this feature? I start looking into #5241 for which persistent user choice will be needed. It will help and ensure consistency if I can build on this feature. Cheers |
@fcollonval I started work but got stuck. @ellisonbg asked me to go ahead and create the PR so somebody else could pick up where I left off and I haven't been tracking since that point. |
@seawolf42 thanks for the quick feedback. Could you grant me "collaborator" status on your Jupyterlab fork, so I can commit directly on the PR branch - avoiding starting a new one? |
@fcollonval done. |
Closing as stale, please feel free to reopen if you'd like to pick this back up |
Fixes #5617
This is the beginnings of a fix for this issue. I'll need help understanding exactly how to resolve this for the
console-extension
tool, and finalizing actually prompting the user to remember the selected result. As per direction from @ellisonbg creating the PR to determine what should happen next.Screenshots
If applicable, add before and after screenshots to help show the issue being fixed, or the appearance of a new feature.