advice2020 left a comment (geany/geany-plugins#1585)

@Skif-off
I was able to test these two scripts out.
Thank you very much for providing these.
At first I had noticed some possible issues compared to what I had mentioned 
but then realized the purpose of the decisions.
I will share some information below.

---------------------------------------------------------------

`SCRIPT 1`
-Works really well, applies to entire document and selection only

-This script opens up a dialog instead just applying to spaces.
At first was confused when I saw dialog but makes sense because gives the 
script/users more versatility, allowing user to be able to type in the 
"delimiter" that appears between each item, to know what to look for in making 
the split.

-The field contains `,` for the default entry.
Makes sense, because can then see this symbol and figure out what to change
For any users reading this/interested in this script in the future.
Can change this line
`local sDelim = geany.input("Delimiter:", ",")`

`local sDelim = geany.input("Delimiter:", "<default-field-value-here>")`
I changed mine to the following to put a space in the field to then be able to 
just hit ENTER key to apply
`local sDelim = geany.input("Delimiter:", " ")`

---------------------------------------------------------------

`SCRIPT 2`
-This script works really well also.
-Only thing I noticed is that it only works based on selection.  It does not 
work to apply this script to the entire document.
Could be for the better because might be more of a rare use case, document 
would have to completely be in list form to want to apply to the entire 
document.  This also eliminates possibility of user error apply this to an 
entire document.

---------------------------------------------------------------

Thank you again for providing these scripts, I really appreciate it.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/issues/1585#issuecomment-4949181563
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany-plugins/issues/1585/[email protected]>

Reply via email to