Resolves #109, File Tree Lines can be Added to Exclude/Include Pattern Through Click

[DavidUlloa6310]

Made a couple of changes to finish the feature:

• Added some javascript code in result.jinja to add or remove file names to the patter input.
• Placed an onchange handler on the pattern selector to change CSS classes depending on whether "exclude" or "include" is selected. The simplest way to do this was to toggle all the classes, since it handles already selected files as well without any extra logic.

@cyclotruc feel free to let me know what you think! You also mentioned wanting to add some kind of notice to users that the items were clickable - do you think adding some text above the file tree would be the best way to do that?

[cyclotruc]

@DavidUlloa6310 Thank you for this PR, taking a look asap

[cyclotruc]

Incredible!

Very clean lightweight implementation thank you so much

[cyclotruc]

@DavidUlloa6310 great pr, thanks again

There has been a recent restructure of the repo which requires a rebase with main
I would do it myself but I can't find a way to push on your branch

You can give me access or do it yourself:
simply move
git_form.jinja and result.jinja into their new location in src/server/templates

[DavidUlloa6310]

@DavidUlloa6310 great pr, thanks again

There has been a recent restructure of the repo which requires a rebase with main I would do it myself but I can't find a way to push on your branch

You can give me access or do it yourself: simply move git_form.jinja and result.jinja into their new location in src/server/templates

@cyclotruc thank you for the compliments! I went ahead and did the rebase - let me know if everything looks good to merge

[cyclotruc]

While testing I noticed a bug:

The copy button on the directory structure doesn't work anymore
I think it's because you used divs in the output box, and the current js for the copy button is looking for another container.

I think it could be solved with a hidden variable containing the content that needs to be copied when the button is clicked

Also but less important, for the CI to pass, you need to run the pre-commit hooks, you can install them by running pre-commit install

If you me to want help on this, you can allow me to your repo i'd be happy to fix this :)

[DavidUlloa6310]

Sounds good! Thanks again for the help - I was totally confused as to why the test was failing. I just sent you an invite to my fork of the repo

[cyclotruc]

@DavidUlloa6310 I ran the pre-commit hooks and fixed the issue with the copy button using a hidden field containing the original tree content

Merging this, Thanks a lot for your contribution!