Skip to content

Integration with Unite, if installed - #56

Merged
MattesGroeger merged 1 commit into
MattesGroeger:masterfrom
rafi:feature/unite
Aug 20, 2014
Merged

Integration with Unite, if installed#56
MattesGroeger merged 1 commit into
MattesGroeger:masterfrom
rafi:feature/unite

Conversation

@rafi

@rafi rafi commented Aug 20, 2014

Copy link
Copy Markdown
Contributor

Unite is a multi-purpose plugin platform for Vim. I've changed BookmarkShowAll() to check if Unite is installed and has a source named 'quickfix', if it's installed, open it instead of the quickfix window.

@MattesGroeger MattesGroeger added this to the 1.3.0 milestone Aug 20, 2014
MattesGroeger pushed a commit that referenced this pull request Aug 20, 2014
Integration with Unite, if installed
@MattesGroeger
MattesGroeger merged commit 7ed4d97 into MattesGroeger:master Aug 20, 2014
@MattesGroeger

Copy link
Copy Markdown
Owner

Thanks for your contribution @rafi 👍

@alexandnpu

Copy link
Copy Markdown

Thanks to make this happen. it is really what i want.
Thanks again for introducing Unit to me.

To use this functionality, besides Unite, you have also to clone https://github.com/osyo-manga/unite-quickfix.

@rafi
rafi deleted the feature/unite branch August 21, 2014 07:01
@rafi

rafi commented Aug 21, 2014

Copy link
Copy Markdown
Contributor Author

Cheers guys. By the way, to set a context globally for the 'quickfix' source:

call unite#custom#profile('source/quickfix,source/location_list', 'context', {
    \   'winheight': 13,
    \   'direction': 'botright',
    \   'start_insert': 0,
    \   'keep_focus': 1,
    \   'no_quit': 1,
    \ })

@MattesGroeger

Copy link
Copy Markdown
Owner

Hey @rafi, would you mind to add some small documentation to the Readme.md and docs/bookmarks.txt? That would be great!

@rafi

rafi commented Aug 21, 2014

Copy link
Copy Markdown
Contributor Author

@MattesGroeger pushed, see #57.
@alexandnpu checkout my Unite configuration

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.

3 participants