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

list variables in sorted order to make navigation easier #798

Closed
dscherger-stripe opened this issue Nov 10, 2022 · 4 comments · Fixed by #806
Closed

list variables in sorted order to make navigation easier #798

dscherger-stripe opened this issue Nov 10, 2022 · 4 comments · Fixed by #806

Comments

@dscherger-stripe
Copy link

Your proposal
When debugging ruby in vscode the variables are displayed in seemingly random order which makes it difficult to locate a specific variable when the list is long. It would be helpful and hopefully easy to display this list in sorted order.

Additional context
Add any other context or screenshots about the feature request here.

image

@ko1
Copy link
Collaborator

ko1 commented Nov 11, 2022

I see. simple sort works?

@wenderjean
Copy link

wenderjean commented Nov 13, 2022

It seems we're not able to search variables in VSCode like we usually do in the console with info c /my_variable/ even though I think the sort feature might be useful for the info command in general, I mean, not only for the VSCode addon.

@ko1
Copy link
Collaborator

ko1 commented Nov 14, 2022

It seems we're not able to search variables in VSCode like we usually do in the console with info c /my_variable/

Yes. I'm thinking to support debug command on the debug console. This is different idea so I'll make new issue or PR.

BTW I think it is better that the debug console should support only Ruby REPL (not accept debug commands) because VSCode (or IDE) has GUI buttons for step/next/.... and i should show the variable i (not info command).
So I'm thinking to support escape character such as # and #i c /pat/ runs i c /pat/ command. Current problem is which character is better for the escaping.

@wenderjean
Copy link

@ko1 Do you think the simple sort is useful? It sounds to me like a good first issue. I'd like to give it a try.

ko1 added a commit that referenced this issue Nov 17, 2022
ko1 added a commit that referenced this issue Nov 17, 2022
@ko1 ko1 closed this as completed in #806 Nov 17, 2022
ko1 added a commit that referenced this issue Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants