Fill in extra buttons via Event

Attempt to get rid of tweak_buttons_func completely

Insert inner widgets in container

Only consume event in correct button, consume event when correct item… …

… wasn't due yet, call function on parent widget

Inserting VocabularyBuilderWidget in plugin table does weird things, … …

…so avoid it

remove unnecessary comment and debug id

remove debug message I missed

Make sure iter returns nil once we go out of bounds

Only try showing vocabbuilder specific dict popup when we're not deal… …

…ing with nested lookups

get rid of Close events, we already have access to the dict popup win… …

…dow we're trying to close

small cleanup - move VocabItemWidget related function up where it bel… …

…ongs

Get rid of onClose as much as possible. Return to initial implementat… …

…ion now that event propagation issue seems to be gone

PR feedback - use WidgetContainer:clear()

rollback - onClose() is the way

Don't consume Close event, this widget still exists after we've been … …

…closed, meaning we are now consuming Close events which are meant to go to the reader/filemanager

fewer changes

Another attempt: make sure VocabularyBuilderWidget is destroyed fully… …

… when closing the window

Some of the callback stuff can just sit in the VocabularyBuilderWidget directly too so change that as well