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.

Sign up for GitHub

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

Jump to bottom

Roadmap #3

Open
6 tasks
fdschmidt93 opened this issue Nov 14, 2021 · 2 comments
Open
6 tasks

Roadmap #3

fdschmidt93 opened this issue Nov 14, 2021 · 2 comments

Comments

@fdschmidt93
Copy link
Member

fdschmidt93 commented Nov 14, 2021 •
edited
Loading

This issue will keep users informed of any roadmap, gotchas, breaking changes, and the like.

Roadmap

The next focus will be on refactoring

  • the entry maker and associated path display
  • previewer (maybe with upstream PRs) to facilitate customization

TODOs (PRs welcome :)

TODOs are not really sorted by priority.

  • A view of multi-selections (cf this comment)
  • Warning: Batch Rename (renaming multi-selections) does not account for links between renamed objects yet (difficult fix). In any case, it should be safe to rename disjoint paths.
  • Better handle moving files/folders to targets in which files/folders already exist, see "skipping" in fb_actions
  • Being able to toggle depth in the file browser as false is unlimited depth, could be interesting
  • Maybe allow to set application by extension with which a file is opened
  • Own previewer for making previewer truly customize-able (will happen upstream)

The aforementioned list will be continuously updated.

Contributions

Any help on the above mentioned issues is greatly appreciated! To get started, familiarize with the code base and plenary's pathlib. In fact, I'm very much welcome a main maintainer of the project as my work on telescope core already takes a lot of my time allocated to open source.
@fdschmidt93 fdschmidt93 pinned this issue Nov 14, 2021
@fdschmidt93 fdschmidt93 changed the title Notices Roadmap Nov 14, 2021
@fdschmidt93 fdschmidt93 mentioned this issue Nov 15, 2021
Closed
@fdschmidt93
Copy link
Member Author

fdschmidt93 commented Nov 17, 2021 •
edited
Loading

Important notice: the Batch Rename should not yet be used with paths that have interdependence.

For instance, you select a folder and then a sub-file or sub-folder to rename. Since there is no path resolution right now, this fails for the latter paths as they have become invalid. This is non-trivial to resolve.

Nevertheless, atomically renaming multiple files and folders without interdependence should work fine.

@fdschmidt93
Copy link
Member Author

As an update: apart from deprecating the file-browser upstream and maybe cleaning up remappings (in co-op with upstream PR), I'll unfortunately won't have too much time up until end of February as work keeps me preoccupied until end of this year and I'll (hopefully) will be travelling throughout January.

With 0.6 now the basic functionality should be working reasonably well and I'll do my best to attend to the small issues in the meantime, too.

Once more, I'd be very happy to get other people on board to help out with developing the extension :)

@steve-lorimer steve-lorimer mentioned this issue Dec 8, 2021
Closed
@jamestrew jamestrew mentioned this issue Dec 29, 2021
Merged
@fdschmidt93 fdschmidt93 unpinned this issue Jan 10, 2022
@fdschmidt93 fdschmidt93 pinned this issue Jan 10, 2022
jamestrew added a commit to jamestrew/telescope-file-browser.nvim that referenced this issue Jan 17, 2022
@jamestrew
refactor: standardize fb_action user IO
96a4b6e 
Using vim.notify for output messages (log level = WARN for areas
indicating missteps by users), vim.ui.input for inputs.

All messages prefixed by `[telescope]` to be clear about the origins of
the message.

Checks off an item from nvim-telescope#3
jamestrew added a commit to jamestrew/telescope-file-browser.nvim that referenced this issue Jan 17, 2022
@jamestrew
refactor: standardize fb_action user IO
dcf3729 
Using vim.notify for output messages (log level = WARN for areas
indicating missteps by users), vim.ui.input for inputs.

All messages prefixed by `[telescope]` to be clear about the origins of
the message.

Checks off an item from nvim-telescope#3
jamestrew added a commit to jamestrew/telescope-file-browser.nvim that referenced this issue Jan 17, 2022
@jamestrew
refactor: standardize fb_action user IO
ed715ef 
Using vim.notify for output messages (log level = WARN for areas
indicating missteps by users), vim.ui.input for inputs.

All messages prefixed by `[telescope]` to be clear about the origins of
the message.

Checks off an item from nvim-telescope#3
@jamestrew jamestrew mentioned this issue Jan 17, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fdschmidt93