Skip to content

Interface addons #248

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Interface addons #248

wants to merge 7 commits into from

Conversation

niloct
Copy link

@niloct niloct commented Jun 20, 2016

This fixes #183, and introduces new features in Vdebug:

  1. The ability to get context from stack depths > 0. In other words, when you have more than one stack line in the Stack Window, double clicking each line loads the corresponding context (variables and values) to Watch Window;
  2. New sign (#>) placed on source and stack windows: When you do the above command, there is a new blue sign placed on source and stack windows that shows the current stack selected, and the line number in source window is marked and centered vertically to make easy to read surrounding code.

niloct added 6 commits June 4, 2016 14:07
When double-clicking or hitting Enter in a stack window filename, which
causes it to load the file in the source window, this change makes it
always center the cursor vertically.
With this commit now Vdebug places a stack sign (#>) on both Stack and
Source windows, to make clear which file and line it's loading from user
interface activity on Stack window.

The refresh method on Runner clears both signs.
Now Vdebug can query any stack depth, and the corresponding variables
and values will appear in the watch window. Also, the stack depth file
and line number will also appear correctly.

A visual sign (#>) was introduced to mark clearly the source and stack
windows for the current stack depth and file_and_line. When the
debugging resumes, these stack signs are removed from the ui.

Also updated the docs with a short description of these features.
A bug in previous commit was not setting runner.context_stack_depth to
zero when resuming the debugging session with a step or run command.

Also the Watch window was not showing the current file and line number
in the situation above.
Now a property expansion in Watch window list all children from that
property. The previous behavior was listing only <pagesize> elements
(i.e. based on the pagesize attribute of parent element).

As before, to invoke a property expansion either double-click it or
place the cursor over it and hit Enter.
The previous commit introduced the feature of getting all children from
parent elements in the watchwin tree.

This worked for property get UI commands issued from the Watch window
itself. Now, the default context get (when hit <F11> or stepping through
code or coming from Stack window) shows all children of tree depth==1.

Before this commit, context get would only show <pagesize> number of
children.
@przepompownia
Copy link
Contributor

Works for me. 👍

@thomasbaumann
Copy link

Great! Thank you @joonty for vdebug and @niloct for the "show more than 10 props"-fix :) worx 👍

Now the setting continuous_mode, if set, is working again: debugger
starts listening on finished debugging session.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DebuggerStatus: Only ten visible elements in Perl hash or list that is greater
4 participants