venn.nvim icon indicating copy to clipboard operation
venn.nvim copied to clipboard

Ascii drawing

Open matu3ba opened this issue 1 year ago • 3 comments
trafficstars

matu3ba avatar May 07 '24 22:05 matu3ba

@jbyuki I dont quite understand the idea behind the code separations yet, I would rename everything with appendix '_utf' and duplicate logic before unifying. Or do you have other opinions?

I should very likely do something faster for verbatim replacements rather than manually typing

rg -l -F 'parse' | xargs -n1 sed -i 's;parse;parse_utf;g'`

matu3ba avatar May 07 '24 22:05 matu3ba

Related to #27

jbyuki avatar May 08 '24 19:05 jbyuki

@jbyuki I dont quite understand the idea behind the code separations yet, I would rename everything with appendix '_utf' and duplicate logic before unifying. Or do you have other opinions?

Why not suffix the new code with a suffix instead of renaming everything? Code duplication is fine (also file duplication).

If you need to implement a new drawing logic, this is what I understand of being one of the goal of this PR, the best would be to create a new *.t file in primitives/. Just like box_over.lua.t was implemented after the initial implementation which has a different logic for drawing.

jbyuki avatar May 08 '24 20:05 jbyuki