mirror of
https://github.com/DominikDoom/a1111-sd-webui-tagcomplete.git
synced 2026-11-05 00:17:25 +00:00
[GH-ISSUE #49] [Feature request] make autocomplete suffix (, ) optional in config file #37
Labels
No labels
bug
duplicate
enhancement
help wanted
invalid
low priority
low priority
pull-request
question
waiting for feedback
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
DominikDoom/a1111-sd-webui-tagcomplete#37
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @byzod on GitHub (Oct 29, 2022).
Original GitHub issue: https://github.com/DominikDoom/a1111-sd-webui-tagcomplete/issues/49
Originally assigned to: @DominikDoom on GitHub.
tagcomplete add comma when select a tag from result unless it's
However there're many situation that comma after a tag is not needed
multiple tags in parenthesis for example complete
(wet swimming:1.2)Some people suppose that even for novelai model, natural sentence works better than tag style when describe background and actions, I feels it's true so I use more natural sentences now, that's when the auto suffix is unnecessary or even bothering

In this demo I have to press
backspacemany times to write a short sentenceThe smart detection is limited,
,is not added to(<tag>:<number>), but still added to(<tag>)or[<tag>], it only works when editing too, writing(dog ears:1.2)character by character with tagcomplete you get(dog ears, :1.2)It's not needed in editing, change
cattocat earsingirl with cat and tailwith tagcompletethe result will be
girl with cat ears, and tail(note there are two space after,)@DominikDoom commented on GitHub (Oct 29, 2022):
I'll add a config or UI option for that tomorrow, I've heard multiple times now that some would like to turn it off. Since it comes down to personal preference there's no reason not to have an option for it.
1 and 2 are definitely a valid argument for it as well.
3 and 4 too, although 3 is probably just an oversight which I can add. Similarly 4 with the double space is because it currently always adds comma and space together, I can add a check if there already is a space after the comma which should mitigate that.
@DominikDoom commented on GitHub (Oct 30, 2022):
The way I chose to do it for now is via both config and UI, since I think some users might want to switch this functionality on/off frequently.

So it has its own checkbox:
and whether
appendCommais true or false in the config dictates if it is on or off by default.