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

Shift-tab unindent vs. docstring/signature information #15901

Closed
schmidi314 opened this issue Mar 3, 2024 · 5 comments
Closed

Shift-tab unindent vs. docstring/signature information #15901

schmidi314 opened this issue Mar 3, 2024 · 5 comments
Labels
bug status:Duplicate status:Needs Triage Applied to new issues that need triage

Comments

@schmidi314
Copy link
Contributor

Description

Shift-tab has (at least) two functions: (1) unindenting one level and (2) showing signature and docstring of a function. If typed inside of an indented function

Peek 2024-03-03 12-54

Reproduce

  1. New notebook
  2. Define some function
def test(a,b):
    print(a, b)
  1. Use test(a,b) indented
if True:
    test(a, b)
  1. With cursor inside test(a,b) parentheses, hit shift-tab. --> info is shown AND test is unindented

Expected behavior

Only info should be shown. NO un-indentation.

Context

  • Operating System and version: Linux and Windows tried
  • Browser and version: Firefox
  • JupyterLab version: 4.1.2
@schmidi314 schmidi314 added the bug label Mar 3, 2024
@jupyterlab-probot jupyterlab-probot bot added the status:Needs Triage Applied to new issues that need triage label Mar 3, 2024
@krassowski
Copy link
Member

@schmidi314 thanks for opening the issue, I believe this is a duplicate of #15821 (regression in 4.1); there is a pull request fixing it (#15898) - help in review of the PR would be appreciated!

@schmidi314
Copy link
Contributor Author

Ah yes, @krassowski you're right. My apologies. I didn't understand the word "dedent" :)

I tried your PR via binder and it solves my problem. Thanks for tackling this! Is there anything I can do? (you mentioned help in review..??) Unfortunately I'm pretty busy these days so I couldn't really get active in this repo like I planned.

@krassowski
Copy link
Member

Thanks @schmidi314! If you could leave a review comment on #15898 just saying that you tried this via binder and it does the intended thing it would already be of a help to other reviewers as they no longer need to validate that it does the thing but only to look at the code to check if it looks right. Of course if you feel like looking at the code as well and leaving any comments that would be appreciated too but I was suggesting just testing it out and leaving a comment over in the PR :)

@krassowski
Copy link
Member

I didn't understand the word "dedent" :)

It is confusing with so many code verbs used for roughly the same things ("dedent", "unindent", "outdent", "indent less")

@JasonWeill
Copy link
Contributor

Closing in favor of #15821.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug status:Duplicate status:Needs Triage Applied to new issues that need triage
Projects
None yet
Development

No branches or pull requests

3 participants