kndndrj / nvim-dbee

Interactive database client for neovim
GNU General Public License v3.0
758 stars 51 forks source link

Mattias/progress tick #36

Closed kndndrj closed 1 year ago

kndndrj commented 1 year ago

adds progress ticker

cancel query?

kndndrj commented 1 year ago

@MattiasMTS I hope you don't mind me opening this PR. I have a small commit, which addresses your comment. If you give me push permissions for this pr, I can push it.

MattiasMTS commented 1 year ago

@MattiasMTS I hope you don't mind me opening this PR. I have a small commit, which addresses your comment. If you give me push permissions for this pr, I can push it.

I sent you an invitation request as a collaborator just now! Let me know if that works. Feel free to delete the leftover commit 36fb188 since it is not meant to be in this one.. My bad.

kndndrj commented 1 year ago

Here is the commit, feel free to overwrite it if you see fit

MattiasMTS commented 1 year ago

Here is the commit, feel free to overwrite it if you see fit

Awesome! I'll probably take a look at this on Sunday (busy tonight and tomorrow hehe). Thanks for the commit!! 😃

kndndrj commented 1 year ago

No rush. I also won't be able to spend any time on this fora while

MattiasMTS commented 1 year ago

Ready for review @kndndrj . I left the implementation of this PR to only add the query progress bar - otherwise, it might have been messier..

Here are some video examples of how it looks like. Feel free to try it out.

Default config

https://github.com/kndndrj/nvim-dbee/assets/86059470/d6492898-c8f2-41e2-bf4d-e6a54fdc1cda

Custom config

https://github.com/kndndrj/nvim-dbee/assets/86059470/315f3f24-432a-47fd-82de-220b87c0ccd6

kndndrj commented 1 year ago

@MattiasMTS Looks very pretty! I agree to narrow the scope of this PR to a single feature as well.

I need to test it first, but otherwise it looks ok!

Since this is a pretty straight forward feature I'll use this branch to do the changes if i find something is not working (if you don't mind ofcourse)

kndndrj commented 1 year ago

same here, just moved some stuff around, the way it makes more sense for me.

Thanks again!