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

The design #1

Closed
Shougo opened this issue Dec 7, 2017 · 60 comments
Closed

The design #1

Shougo opened this issue Dec 7, 2017 · 60 comments

Comments

@Shougo
Copy link
Owner

Shougo commented Dec 7, 2017

  • Not depends on denite.nvim

  • Vim8/neovim compatible(nvim-yarp is needed for Vim8)

  • Implemented by Python3

  • No double filer feature

  • Column feature

  • Source feature

  • Options

  • Highlight is defined by column

  • Few commands (:Defx command only?)

  • Extended rename

  • Mark

  • Windows supporters are needed

  • The original trashbox feature

  • No minimal features

  • Subtree feature like vimfiler

  • No default mappings

  • No global variables

  • No explorer feature(like g:vimfiler_as_default_explorer)

Release date: In 2018 someday

@petobens
Copy link

petobens commented Dec 8, 2017

I think the double filer feature is really useful (i use it a lot). Also it would be nice to keep vimfiler's bookmark feature.

@wsdjeg
Copy link

wsdjeg commented Dec 8, 2017

@Shougo nice work,

@petobens what is double filer feature?

@kirin123kirin
Copy link

@Shougo
待ってました。
いつ頃リリース予定ですか?

@kutsan
Copy link

kutsan commented Dec 10, 2017

@kirin123kirin According to my Google Translate, I am also wondering when will it be released.

@kirin123kirin
Copy link

Can I open UNC Path for windows platform.

@Shougo
Copy link
Owner Author

Shougo commented Dec 11, 2017

I think the double filer feature is really useful (i use it a lot). Also it would be nice to keep vimfiler's bookmark feature.

It is useful. But I think jump/mark feature is better.
bookmark feature is to be determined.

@Shougo
Copy link
Owner Author

Shougo commented Dec 11, 2017

@petobens what is double filer feature?

:VimFilerDouble

You can copy/move the files to another vimfiler directory.

@Shougo
Copy link
Owner Author

Shougo commented Dec 11, 2017

Can I open UNC Path for windows platform.

The support is very hard.
I don't want to support it. You can add it though.

Please see:
Shougo/vimfiler.vim#170

@Shougo
Copy link
Owner Author

Shougo commented Dec 11, 2017

@kirin123kirin The release will be in 2018 someday.

@Shougo
Copy link
Owner Author

Shougo commented Jan 11, 2018

Column feature

Column means, "name", "type", "size", "date" should be customize-able more generic.
Users can add more columns like "git" column.

@lambdalisue
Copy link
Collaborator

It would be nice if it does not rely on filesystem.

For example, I would like to make a plugin which cooperate with this one to show a directory tree on a particular commit.
In that case, probably the INPUT for this plugin would be a tree dictionary with custom kind, actions, etc.

@Shougo
Copy link
Owner Author

Shougo commented Jan 16, 2018

It is already todo listed.

Source feature

@Shougo
Copy link
Owner Author

Shougo commented Jan 17, 2018

The source feature is similar to denite source.
But it is not compatible.

@Shougo
Copy link
Owner Author

Shougo commented Jan 21, 2018

I have started the vaporware development.

4b4dfae

@kutsan
Copy link

kutsan commented Feb 16, 2018

@Shougo Is there any way to set colors of items in the tree explorer according to $LS_COLORS env. variable (a.k.a. dircolors)?

Will you provide an API for vim-devicons symbols? Showing symbols on the right side of the explorer wasn't right choice. They should be just before the filename.

@Shougo
Copy link
Owner Author

Shougo commented Feb 18, 2018

@Shougo Is there any way to set colors of items in the tree explorer according to $LS_COLORS env. variable (a.k.a. dircolors)?

No.

Will you provide an API for vim-devicons symbols? Showing symbols on the right side of the explorer wasn't right choice. They should be just before the filename.

Yes. The column feature will be improved than vimfiler.

@Shougo
Copy link
Owner Author

Shougo commented Feb 28, 2018

I have decided defx has no default mappings and no global variables.

@Shougo
Copy link
Owner Author

Shougo commented Feb 28, 2018

defx.nvim does not support explorer feature like g:vimfiler_as_default_explorer.

It is very buggy and hard to implement.

@amikai
Copy link
Contributor

amikai commented Jul 19, 2018

I think the design can work like "file manager". Anything file related feature can be supported, not just explorer.
My suggestion is:
Inspire by Vscode.
Implement the special window which manages number of buffers.
The default buffer in this window is file explorer(like vscode).
The second buffer maybe is git related information(like vscode).
Then other buffer can be implemented by who they want other feature.

@Shougo
Copy link
Owner Author

Shougo commented Jul 30, 2018

I have re-started defx.nvim development.
You can test it.

Note: Current defx.nvim is neovim only.

@amikai
Copy link
Contributor

amikai commented Jul 31, 2018

What does toggle_select do?

@Shougo
Copy link
Owner Author

Shougo commented Jul 31, 2018

What does toggle_select do?

It changes candidate select state.
It same with denite.

@kristijanhusak
Copy link
Contributor

@Shougo , when will custom column feature be available?

@Shougo
Copy link
Owner Author

Shougo commented Aug 29, 2018

It takes more time.

  • More default columns
  • Multiple buffers support
  • Split defx buffer
  • Copy, Move

Above features are hipriority than column customization.

@kristijanhusak
Copy link
Contributor

Would you like to have git status column as part of the default columns? I have initial version of it.

@Shougo
Copy link
Owner Author

Shougo commented Aug 29, 2018

Would you like to have git status column as part of the default columns?

No.

@Shougo
Copy link
Owner Author

Shougo commented Sep 17, 2018

Copy, Move

Implemented.

@oblitum
Copy link

oblitum commented Nov 15, 2018

Is it in the plan to remove the yarp solution for supporting Vim? Having to depend on NeoVim for Vim to work isn't great.

@oblitum
Copy link

oblitum commented Nov 23, 2018

@Shougo thx!

@oblitum
Copy link

oblitum commented Nov 23, 2018

@Shougo defx#do_action('open_directory') is still opening files too though. I just tried.

@Shougo
Copy link
Owner Author

Shougo commented Nov 23, 2018

Fixed.

@oblitum
Copy link

oblitum commented Nov 23, 2018

Working :)

@pwntester
Copy link

Can you make open_directory similar to open in that it takes an optional argument with the directory path. I want to implement a custom action and need to receive the selected directory name and it should not trigger on regular files.

Thanks!

@Shougo
Copy link
Owner Author

Shougo commented Dec 3, 2018

I don't understand.
Please upload the example.

@pwntester
Copy link

I was trying to enable a mapping in the defx windows that when used on a directory, will run a command that takes the directory path as an argument.

For example, if I enter / on the /tmp/foo directory, I want to run: call esearch#init({'cwd': '/tmp/foo'})

@Shougo
Copy link
Owner Author

Shougo commented Dec 3, 2018

OK. I get it.
Please create the new issue for it.

@pwntester
Copy link

Thanks, filed #51

@swnakamura
Copy link
Contributor

The time/date a directory is modified is not shown even when time option is enabled.
Is it an intentional design to hide the modification time of directories?

@swnakamura
Copy link
Contributor

Also, when I do :Defx with unsaved file, the unsaved changes are discarded.
I think it would be better if we have something like --force option?

@swnakamura
Copy link
Contributor

swnakamura commented Dec 11, 2018

The time/date a directory is modified is not shown even when time option is enabled.
Is it an intentional design to hide the modification time of directories?

I made a PR to it. If you don't like it, please discard.
#53

@Shougo
Copy link
Owner Author

Shougo commented Dec 11, 2018

Also, when I do :Defx with unsaved file, the unsaved changes are discarded.
I think it would be better if we have something like --force option?

Please create the new issue for it. I will test it later.
If not, I will ignore your issue.

@swnakamura
Copy link
Contributor

@Shougo Thank you! I created new issue #55 and start working on it :)

@oblitum
Copy link

oblitum commented Dec 17, 2018

Mouse support, good to have? VimFiler had I think.

@kristijanhusak
Copy link
Contributor

You can add mouse support yourself if you want:

nnoremap <silent><buffer><expr> <2-LeftMouse> defx#do_action('drop')

@Shougo
Copy link
Owner Author

Shougo commented Dec 17, 2018

Yes. @kristijanhusak is right.

@oblitum
Copy link

oblitum commented Dec 17, 2018

@kristijanhusak @Shougo nice, thanks!

@oblitum
Copy link

oblitum commented Dec 17, 2018

My recommendation:

  nnoremap <silent><buffer><expr> <2-LeftMouse> defx#do_action('drop')
  nnoremap <silent><buffer><expr> <RightMouse> defx#do_action('cd', ['..'])

@oblitum
Copy link

oblitum commented Dec 21, 2018

Someone just asked on #vim whether there's a file browser that supports <c-i> and <c-o>, it seemed like netrw did not. I ignore one but, thought it would be food for thought here. The usecase was the person liked to go back to file browser after opening a buffer. Another usecase I think would be for directory navigation as a whole, would be nice <c-i>/<c-o> working at that.

@oblitum
Copy link

oblitum commented Dec 21, 2018

Seems dirvish supports that.

@wsdjeg
Copy link

wsdjeg commented Dec 22, 2018 via email

@Shougo
Copy link
Owner Author

Shougo commented Dec 24, 2018

@oblitum I don't add the feature. Yes, dirvish already supports the feature. But some feature don't like it. I also don't like it.

Please see below issue.
justinmk/vim-dirvish#110

@petobens
Copy link

petobens commented Jan 2, 2019

bookmark feature is to be determined.

@Shougo do you plan to add integrations with https://github.com/kmnk/denite-dirmark? I think that a good file manager should have a bookmarks feature.

@Shougo
Copy link
Owner Author

Shougo commented Jan 2, 2019

@petobens Nothing. But you can request it to the author.

@Shougo
Copy link
Owner Author

Shougo commented Jan 2, 2019

I think that a good file manager should have a bookmarks feature.

I have not much time to implement bookmark feature in defx.

@petobens
Copy link

petobens commented Jan 2, 2019

Ok thanks. I did this here kmnk/denite-dirmark#2

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

No branches or pull requests