nvimdots icon indicating copy to clipboard operation
nvimdots copied to clipboard

Add some new plugins

Open charliie-dev opened this issue 1 year ago • 4 comments

  1. advanced-git-search.nvim
  2. search.nvim
  3. ...

charliie-dev avatar Apr 19 '24 11:04 charliie-dev

do i need to change the target branch to 0.10? @ayamir

charliie-dev avatar Apr 20 '24 03:04 charliie-dev

do i need to change the target branch to 0.10? @ayamir

it's better to switch 👍

ayamir avatar Apr 20 '24 05:04 ayamir

May you consider add the vim-orgmode plugin?

gezhanling avatar Apr 28 '24 07:04 gezhanling

i'm not sure if adding orgmode would benefit most neovim users. If we really need to add orgmode, i think https://github.com/nvim-orgmode/orgmode or https://github.com/nvim-neorg/neorg might be better.

charliie-dev avatar Apr 28 '24 09:04 charliie-dev

if adding orgmode would benefit most neovim users.

If memory serves me correctly, I think this is exactly why we dropped orgmode around two years ago. It's undeniably powerful, but getting the hang of it still demands some effort. Plus IMHO it actually diverges from the primary purpose of using neovim as a pure text editor for the majority of neovim users - its code is notably self-contained and straightforward to configure, but it's also massive(!), which (I think) categorizes it as an excellent example of custom plugins rather than being included in our default plugin list. My two cents on using orgmode is that it's a great "app" for note-taking related stuff, but it's definitely not a must-have for our base config.

Jint-lzxy avatar Apr 30 '24 09:04 Jint-lzxy

My two cents on using orgmode is that it's a great "app" for note-taking related stuff

By the way, do you use the Emacs orginal orgmode?

gezhanling avatar May 09 '24 02:05 gezhanling

My two cents on using orgmode is that it's a great "app" for note-taking related stuff

By the way, do you use the Emacs orginal orgmode?

@gezhanling I experimented with that years ago before transitioning to Vim, and imo it worked nicely with Emacs :D

Jint-lzxy avatar May 13 '24 04:05 Jint-lzxy

aha, totally forget to change that lol. tks for the fix up!

charliie-dev avatar Jun 11 '24 07:06 charliie-dev

hmmm, i'm not sure why luacheck didn't pass.

charliie-dev avatar Jun 13 '24 11:06 charliie-dev

I set https://github.com/ayamir/nvimdots/blob/400e77d977c0a5d5da04194a2aa82d09d536cd0e/lua/modules/configs/tool/search.lua#L4 to default setup instead our custom setup function since it might cause issue for current universal telescope collections panel.

charliie-dev avatar Jun 14 '24 02:06 charliie-dev

I'm a bit occupied right now, but I'll check out this PR later this week lol

Jint-lzxy avatar Jun 14 '24 03:06 Jint-lzxy

... to default setup instead our custom setup function since it might cause issue for current universal telescope collections panel.

@CharlesChiuGit I've tried my best to comprehend this PR, but I'm still a bit perplexed about the current implementation. iiuc we have five different search groups for Telescope (namely file, live_grep, git, workspace and misc). However, in lua/keymap/tool.lua, I only see two of these groups (file and git) being actively used, not including the "mega" key <leader>f. Is there a specific design reason for this, or did I miss something?

Jint-lzxy avatar Jun 17 '24 09:06 Jint-lzxy

tbh, i think we can remove those keymaps and just keep the <leader>f, i leave <leader>f'X' there just to keep consistent w/ our old config, so we won't be bombarded by issues lol

https://github.com/ayamir/nvimdots/blob/f70e6850490e004dd2d5ed74076dab7362fd9c79/lua/keymap/tool.lua#L92-L97

isn't this what u mean by mega key?

charliie-dev avatar Jun 17 '24 09:06 charliie-dev

just to keep consistent w/ our old config, so we won't be bombarded w/ issues lol ... isn't this what u mean by mega key?

lol thanks for clarifying! imo we can just remove these "compatible" keymaps and update the wiki. After all, we're going to introduce breaking changes when #1260 is merged anyway :D

Jint-lzxy avatar Jun 17 '24 09:06 Jint-lzxy

isn't this what u mean by mega key?

Exactly! We probably just need to invoke TabCollection:all() to get this (picker_name) then. Easy peasy lmao!

EDIT: Seems like the following one-liner would do the job for us:

local collections = vim.tbl_keys(require("search.tabs").collections)

Jint-lzxy avatar Jun 17 '24 09:06 Jint-lzxy

https://github.com/ayamir/nvimdots/blob/3d77f4abcf81f788bccdb322218b137fb855b9f5/lua/core/mapping.lua#L41

i moved <leader>bn to core/mapping.lua since i don't see a reason to leave it under telescope lol

charliie-dev avatar Jun 17 '24 10:06 charliie-dev

WAM I think with the current state of the PR, we also need to update dots.tutor (Lesson 1.2) ┬─┬ノ( º _ ºノ)

Jint-lzxy avatar Jun 17 '24 10:06 Jint-lzxy

@CharlesChiuGit Would u mind undertaking the illustrious yet arduous endeavor of updating both the wiki and dots.tutor, or shall we divide the tasks and complete them collaboratively? tbf I'm finding myself oddly fatigued given the current situation 🤣


Jint-lzxy avatar Jun 17 '24 10:06 Jint-lzxy

sure! i can update the wiki part lol take your time! rest as u need

charliie-dev avatar Jun 17 '24 10:06 charliie-dev

sure! i can update the wiki part lol

🫡Thanks a ton! btw I used to use this one-liner (after cloning our repo's wiki pages git clone https://github.com/ayamir/nvimdots.wiki.git)

$ awk '/^\|/{ print $(NF-1) }' < Keybindings.md | sed -e '/^`/!d' -e 's/^`\(.*\)`$/\1/g'

to retrieve the list of possible keymaps. However, that page hasn't been updated in a long time, so u might need to put in some extra effort to ensure all the entries are valid 😆

(Just came up with the following to retrieve the list of defined keymaps in our config. I haven't tested it thoroughly though, but it should work fine :D)

$ rg -IN '\s*\[".*?\|(.*?)"\] = .*$' -r '$1' | sed '/^$/d'

take your time! rest as u need

nvm I'm just joking after seeing the potential paperwork lol I'll take care of dots.tutor, but I might need ur input to ensure everything aligns correctly with respect to search.nvim XD

Jint-lzxy avatar Jun 17 '24 11:06 Jint-lzxy

i think we can merge this pr and update tutor and wiki later?

charliie-dev avatar Jun 18 '24 03:06 charliie-dev

i think we can merge this pr and update tutor and wiki later?

Sure! ur PR, ur choice lol (But I'll try to fix the checks tho)

Jint-lzxy avatar Jun 18 '24 03:06 Jint-lzxy

Sorry for the late reply, currently I'm busy with internship work so I have little time to participate in discussions. IMO current Telescope Collections as a middle selector may hinder fast operation and break the workflow. Perhaps it would be more acceptable to users to keep the key bindings that can directly open different search category interfaces. For example, use <leader>ff can open the file tab directly. In addition, we also need to modify the alpha-nvim UI for the new key bindings

ayamir avatar Jun 18 '24 14:06 ayamir

Sorry for the late reply, currently I'm busy with internship work so I have little time to participate in discussions. IMO current Telescope Collections as a middle selector may hinder fast operation and break the workflow. Perhaps it would be more acceptable to users to keep the key bindings that can directly open different search category interfaces. For example, use <leader>ff can open the file tab directly. In addition, we also need to modify the alpha-nvim UI for the new key bindings

ok, will do

charliie-dev avatar Jun 19 '24 03:06 charliie-dev

Sorry for the late reply, currently I'm busy with internship work so I have little time to participate in discussions.

And best of luck with ur internship lol!

Jint-lzxy avatar Jun 19 '24 05:06 Jint-lzxy