Skip to content

Conversation

@TamaMcGlinn
Copy link
Contributor

Saw this pull request on a weird not-quite fork called vim-scripts/mru. Rebased, tested and works.

command! -nargs=0 MruRefresh call s:MRU_Refresh()
command! -nargs=? -complete=customlist,s:MRU_Complete MRUToggle
\ call s:MRU_Toggle(<q-args>)

Copy link
Owner

Choose a reason for hiding this comment

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

Thanks for the patch. Can you also update the help text with
a description of this new command?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was my first time actually documenting a vim plugin. I hope this was all that you meant.

@TamaMcGlinn
Copy link
Contributor Author

I just read this:

By default, the MRU window is opened as the bottom-most window. If you are
using Vim version 8.0 and above, then you can use command modifiers like
|:topleft| or |:botright| or |:vertical| with the :MRU command to control
where the window is opened. Example: >

    :topleft MRU			" horizontally split topmost window
    :botright MRU			" horizontally split bottommost window
    :vertical topleft MRU		" vertically split far-left window
    :vertical botright MRU		" vertically split far-right window

and realised my mistake. I guess I need to put MRUToggle into the same if and pass the other args.

@TamaMcGlinn
Copy link
Contributor Author

Should be good to merge now.

@yegappan yegappan merged commit a484baa into yegappan:master Aug 13, 2021
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.

3 participants