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

clicking on code cell output doesn't select cell #13026

Closed
1 of 2 tasks
yuuuxt opened this issue Mar 8, 2023 · 6 comments · Fixed by microsoft/vscode-jupyter-keymap#27
Closed
1 of 2 tasks

clicking on code cell output doesn't select cell #13026

yuuuxt opened this issue Mar 8, 2023 · 6 comments · Fixed by microsoft/vscode-jupyter-keymap#27
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@yuuuxt
Copy link

yuuuxt commented Mar 8, 2023

Applies To

  • Notebooks (.ipynb files)
  • Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?

clicking the celoutput area doesn't select that cell

temp

which is not intuitive and is not consistent with jupyter notebook

temp2

I think there should not be a state of not selecting any cell; which is meaningless - users always want to interact with cell(s). No matter what happens or what is done, one (or multiple) cell should be selected.

This issue along with #12738 makes operation less fluent - you are not sure if your action will be executed.

VS Code Version

1.76.0

Jupyter Extension Version

v2023.2.1000592019

Jupyter logs

No response

Coding Language and Runtime Version

No response

Language Extension Version (if applicable)

No response

Anaconda Version (if applicable)

No response

Running Jupyter locally or remotely?

None

@yuuuxt yuuuxt added the bug Issue identified by VS Code Team member as probable bug label Mar 8, 2023
@DonJayamanne DonJayamanne assigned rebornix and amunger and unassigned DonJayamanne Apr 28, 2023
@amunger
Copy link
Contributor

amunger commented Apr 28, 2023

@yuuuxt - is this still occurring for you on the latest version? The closest I can get to a repro is the cell can be unfocused, but it is still selected, so not as bad as your gif shows

@amunger amunger added the info-needed Issue requires more information from poster label Apr 28, 2023
@yuuuxt
Copy link
Author

yuuuxt commented Apr 29, 2023

yes it's still happening on 1.77 and v2023.3.1201040234. Besides, I'm not sure what you meant by "unfocused but still selected":

  • if my operations are all on the notebook, there should be (at least) one cell selected. When Focusing on notebook (editor) and don't click anything outside, somehow having no cell selected doesn't make sense to me;
  • The consequence is operations like pressing "B", "A" not working. some are still functional like arrow up & down, but inserting cells are frequent moves as well. (so this is strange to me- if it's unfocused & selected, what's the logic of some operations working while some not working, and how to memorize it? what's the purpose of such a state?)

@amunger amunger removed the info-needed Issue requires more information from poster label May 1, 2023
@amunger
Copy link
Contributor

amunger commented May 1, 2023

ok, I wasn't seeing the same UI issues as shown in your gif, but there are a few commands that stop working in that case because they have !notebookOutputFocused in the when clause. It looks like the JupyterKeyMap extension is adding those conditions and I'm trying to see if they could just be removed.
Just to confirm tho - if you remove that condition in those specific keybindings, does that enable the commands correctly?

@amunger amunger added the info-needed Issue requires more information from poster label May 1, 2023
@yuuuxt
Copy link
Author

yuuuxt commented May 2, 2023

yes removing !notebookOutputFocused makes it working. I will check all related shortcuts to make it work for me.

@amunger
Copy link
Contributor

amunger commented May 2, 2023

In that case, I don't believe we can make the obvious change to fix this since the hotkeys are unmodified. All we know from the core side is that the output webview is focused, but it could be in an input or text area if the output is a widget, in which case those letter keys should not perform extra actions.

@amunger
Copy link
Contributor

amunger commented May 19, 2023

verification steps:

  1. open the attached notebook (change the file's extension to .ipynb)
    inputsInOuputs.txt
  2. click in the output area, outside of the text fields
  3. use the raw hotkeys a b j k (insert cell above/below, list previous/next) and ensure they trigger the commands
  4. click within the text area in the output, those same keys should just write text without running commands

@amunger amunger added this to the May 2023 milestone Jun 5, 2023
@rebornix rebornix added the verified Verification succeeded label Jun 5, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants