search

dzirtusss / vifari

Vimium/Vimari for Safari without browser extension in pure Lua
MIT License
15 stars 1 forks source link

refactor: rename pasteUrl to be more semantically correct #3

Closed pkazmier closed 4 days ago

pkazmier commented 2 weeks ago

Small stuff, just wanted to contribute back in some way.

This commit refactors pasteUrl to make it more generic and eliminates the confusion with the name of the function (paste implies pasting versus copying). Plus, who knows what else you might want to copy to the clipboard as part of some future action? Also, renamed copyUrlToClipboard to be eliminate confusion about copying the current URL versus copying a URL selected via a mark.

dzirtusss commented 4 days ago

@pkazmier thank you

overall, it was all very quick done for personal use mostly

plus there are some changes coming, which may be a 0.1 version something - mostly a better logic compartmentalization to proper modules and bindings rewrite, so they will be not hardcoded in code like now anymore, thus easily configurable