Dragging tab across lines may cause empty space between tabs

[jupjohn]

Checklist

• I'm reporting a problem with Chatterino
• I've verified that I'm running the most recent nightly build or stable release
• I've looked for my problem on the wiki
• I've searched the issues and pull requests for similar looking reports

Describe your issue

While I haven't found a way to replicate this issue, here's my rough memory of how I caused it:

1. Have multiple lines of tabs
2. Drag the first tab on the second row up into the first row (unsure if it was left or right of the first row's first tab)
3. Maybe drag the tab back down to the second row in the same movement
4. First row's first tab is now pushed out by a tab width???

Screenshots

God I wish I had the sense to screenshot instead of try to reproduce straight away, but I did not... so here's an artistic rendition of the before & after:

Before:

After:

OS and Chatterino Version

Chatterino 2.5.1 (commit d2316af) built with Qt 6.7.1 Running on Arch Linux, kernel: 6.9.3-zen1-1-zen

[8thony]

Can reproduce

Chatterino Nightly 2.5.1 (commit 25284fc) Qt 6.7.1 Running on Windows 10

[kornes]

chatterino2/src/widgets/helper/NotebookTab.cpp

Lines 415 to 418 in 25284fc

	if (this->pos() == targetPos)
	{
	return;
	}

early return i added recently seems to be a reason, removing it masks the issue and it works as before.
Notebook layouting needs some love and refactoring which i dont have time to commit to, so for now suggest to just remove above change.