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

new sr-speedbar layer for !tools #2002

Closed
wants to merge 4 commits into from
Closed

new sr-speedbar layer for !tools #2002

wants to merge 4 commits into from

Conversation

tshemeng
Copy link
Contributor

cause spacemacs already include neotree, so i leave the function spacemacs/sr-speedbar-show-or-hide for users' to binding.

@tshemeng
Copy link
Contributor Author

sr-speedbar

@robbyoconnor
Copy link
Contributor

Squash all these commits into one tidy commit...also why are there no evil keybindings?

@robbyoconnor
Copy link
Contributor

bump

@CarlQLange
Copy link
Contributor

God I want this, can it please be merged?

@robbyoconnor
Copy link
Contributor

@blinkd -- rebase and squash this...

@CarlQLange
Copy link
Contributor

I'll probably add some evil bindings in a few days as I use it more often. @blinkd if you're not around anymore I can take care of this for you

@robbyoconnor
Copy link
Contributor

Yeah needs to be rebased and add some keybindings. The fact this was not done is beyond me. Also the fact this hasn't been touched and @blinkd has gone AWOL...have at it @CarlQLange

@edrex
Copy link
Contributor

edrex commented Sep 27, 2015

sr-speedbar windows get replicated geometrically (1 -> 2 -> 4 ...) whenever the helm window comes up. #29 (comment)

Edit: actually I can't reproduce this reliably - it just happens sometimes.

@edrex
Copy link
Contributor

edrex commented Sep 27, 2015

Maybe add (setq speedbar-show-unknown-files t) to show non-code files, as @tuhdo suggested in #29 (comment)

@CarlQLange
Copy link
Contributor

sr-speedbar windows get replicated geometrically (1 -> 2 -> 4 ...) whenever the helm window comes up. #29 (comment)

I haven't seen this.

By the way, just running (evilify) seems to be enough to add suitable vim bindings.

@edrex
Copy link
Contributor

edrex commented Sep 28, 2015

I realized that I hadn't actually enabled the sr-speedbar layer. The included post-init function fixes the cloning issue I was seeing.

It seems like this is a good candidate for replacing NeoTree.

@edrex
Copy link
Contributor

edrex commented Sep 28, 2015

Spoke too soon: I'm still seeing the cloning issue.

This consistently reproduces for me (may depend on frame size):

launch emacs
from spacemacs buffer, SPC : sr-speedbar-toggle, now there's one speedbar window
SPC - now there are two speedbar windows - ESC
SPC - now there are four

@CarlQLange
Copy link
Contributor

I still can't reproduce that at all @edrex. However, it does seem like golden-ratio mode will make strange things happen...

@edrex
Copy link
Contributor

edrex commented Oct 1, 2015

I don't have golden ratio toggled on. It may be a platform specific thing (I'm on a mac), or a difference in our emacs configuration/spacemacs layers. I'm pretty new so my config is close to stock. At least one other person, @syl20bnr, reported experiencing the issue. For me it happens quite a lot (but seems slightly nondeterministic).

@CarlQLange
Copy link
Contributor

want to try and include a video of it? and are you using (spacemacs/sr-speedbar-show-or-hide)?

@edrex
Copy link
Contributor

edrex commented Oct 1, 2015

Oh, I had been using (sr-speedbar-toggle). (sr-speedbar-show-or-hide) is behaving better with Helm.

So yeah.. other than the keybindings, seems good.

@CarlQLange
Copy link
Contributor

nice one.

@CarlQLange CarlQLange mentioned this pull request Oct 1, 2015
@CarlQLange
Copy link
Contributor

OK, since I can't change where this PR is coming from, there's a new one here: #3224
this one can be closed.

@edrex edrex mentioned this pull request Oct 26, 2015
@TheBB
Copy link
Collaborator

TheBB commented Oct 26, 2015

Closing this in favour of #3559.

@TheBB TheBB closed this Oct 26, 2015
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.

5 participants