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

Add overlays to the debugger. #1155

Merged
merged 1 commit into from
Jun 22, 2015
Merged

Add overlays to the debugger. #1155

merged 1 commit into from
Jun 22, 2015

Conversation

Malabarba
Copy link
Member

Too much? :)

end-of-line

cider-debug-overlay-2

inline

cider-debug-overlays

@Malabarba
Copy link
Member Author

I figured this is safe enough to go in 0.9.1, so it's based of master.

@bbatsov
Copy link
Member

bbatsov commented Jun 21, 2015

I like it, although it should probably be configurable, as people used to edebug might prefer the use of the minibuffer. Btw, it'd be great if you expand the use of overlays for interactive evaluation as well. This will make it possible for us to do something like instarepl / litable long term.

@Malabarba
Copy link
Member Author

@bbatsov

I like it, although it should probably be configurable, as people used to edebug might prefer the use of the minibuffer.

Oh, the screenshot doesn't show, but it still uses the minibuffer. And there's a custom option to turn the overlays off.

Btw, it'd be great if you expand the use of overlays for interactive evaluation as well. This will make it possible for us to do something like instarepl / litable long term.

The only problem is that these overlays are a little annoying if they stick around. In order to not change the contents of the buffer, they have 0 width, which means the user is completely unable to erase them. There's probably a better way to do this though, as I'm sure I've seen somewhere overlays that vanish when you write over them.

@bbatsov
Copy link
Member

bbatsov commented Jun 21, 2015

There's probably a better way to do this though, as I'm sure I've seen somewhere overlays that vanish when you write over them.

Yep, that'd be best. Another thing to consider is their exact position - some people might not like the overlay shifting things to the right, so an alternative showing the overlay outside the text on the line might be nice to have as well.

Only applies to \"*cider-debug ...*\" buffers, which are used in debugging
sessions."
:type 'boolean
:group 'cider)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget the :package-version properties.

@bbatsov
Copy link
Member

bbatsov commented Jun 21, 2015

On an unrelated note - add a changelog entry for this and drop the . from the commit message's title.

@Malabarba
Copy link
Member Author

All done, and I've factored out the overlay handling as well so it can be reused. Should these functions go in another file or do I leave them there?

@Malabarba
Copy link
Member Author

Also, see new pic at the top.

bbatsov added a commit that referenced this pull request Jun 22, 2015
@bbatsov bbatsov merged commit 2c4ae72 into clojure-emacs:master Jun 22, 2015
@bbatsov
Copy link
Member

bbatsov commented Jun 22, 2015

👍 Fantastic work!

@arrdem
Copy link
Contributor

arrdem commented Jun 22, 2015

👍

@mavbozo
Copy link

mavbozo commented Jun 22, 2015

👍 Awesome!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants