feat: remove and create tags from tag database panel

[DandyDev01]

Added a button in the tag database panel for creating new tags
added ability to delete tags from the tag database panel

[VasigaranAndAngel]

avoid lambda as possible.
#131 (comment)

[VasigaranAndAngel]

use functools.partial instead of lambda.

Suggested change

	tag_widget.on_remove.connect(lambda t=tag: self.remove_tag(t))
	tag_widget.on_remove.connect(partial(self.remove_tag, tag))

[seakrueger]

I believe, the codebase uses lambda throughout when using slots. Is there a real world benefit to using a partial here (or throughout)?

[VasigaranAndAngel]

in this case, partial feels comfortable for me because that makes the intent more explicit and ties directly to the idea of binding tag to self.remove_tag. however i would say it's a matter of style so i'm okay if @DandyDev01 decides to go with lambda here.

[DandyDev01]

In this case, I will just stick with the lambda, because it seems that using a partial will not guarantee a memory leak won't occur and @seakrueger didn't run into any leaks while testing #131 (comment). Also, I haven't seen partials used too much elsewhere in the code base, so for consistency and readability (most people already know lambda and might have to lookup partials) lambda make more sense for now.

[seakrueger]

Two quick things, sorry. Thanks for the pull!

1. Currently, you can remove the built-in tags via the tag panel. Doing so, causes the program to error when clicking the respective icon on a thumbnail.
2. A pretty minor detail that probably isn't a huge deal, removing a tag doesn't refresh the tag boxes on the preview panel. Removing a tag continues to show the tag until the entry is clicked off and reselected.

[seakrueger]

Works great, thanks!

[CyanVoxel]

Can you rebase this to main now that parent tags have been added (#596)? I wouldn't want to pull this until we can verify that it's compatible

[CyanVoxel]

Looks very good! Seems to work mostly well with the new subtag and alias implementation, however I did notice that any subtag and alias connections that reference deleted tags will stick around in the db afterwards. Should just be a straightforward addition of making sure those affected rows get purged as well. I've also got a log warning that happens that's related to this:

F:\GitHub\TagStudio/tagstudio\src\core\library\alchemy\library.py:691: SAWarning: Object of type <TagAlias> not in session, delete operation along 'Tag.aliases' will not proceed
  session.commit()
F:\GitHub\TagStudio/tagstudio\src\core\library\alchemy\library.py:691: SAWarning: Object of type <Tag> not in session, delete operation along 'Tag.subtags' won't proceed
  session.commit()

Also, I know this is a late ask so I understand if you'd rather pass on doing this, but I think a confirmation message box when deleting tags would be a much needed addition. If you'd rather not spend any more time on that, I understand and can implement one myself afterwards (or push to this PR if you're comfortable with that!)

[DandyDev01]

Also, I know this is a late ask so I understand if you'd rather pass on doing this, but I think a confirmation message box when deleting tags would be a much needed addition. If you'd rather not spend any more time on that, I understand and can implement one myself afterwards (or push to this PR if you're comfortable with that!)

All good! I can add a confirmation message box.

[CyanVoxel]

Thank you so much for all of your work on this!