One more thing.
I notice that on reload (or close jadx-gui and open project again) tabs order changes.
To reproduce, move any tab to other position and press reload action (from toolbar or F5) and tabs will reorder.
Not sure, maybe that was broken before this commit, but as soon as we change saving code here it is better to use actual tabs order on project save.

@skylot all ready

Hm, not sure why we need to save index to restore order, is just saving/reading tabs list in correct order not working?

Hm, not sure why we need to save index to restore order, is just saving/reading tabs list in correct order not working?

TabsController now holds the generic reference to each TabBlueprint through a Map<JNode, TabBlueprint>, I avoided saving the tabs in the TabbedPane order because

• it would require a callback to TabbedPane before saving tabs that reorders TabBlueprint and saves them back, and we would need filtering for tabs that are hidden
• we would anyway use the same procedure (store index) if we wanted to also preserve order for QuickTabs and allow reordering later on

But maybe we can just consider the order of TabbedPane to be the "default" order, ok I'll implement it by saving/reading tabs list in correct order

@skylot please see commit 6ca79d4. I implemented saving/reading tabs list in correct order

OK, I think we can merge it now.
Although, here is a small list of thing for future improvements:

• There are still a lot of calls using TabbedPane instead TabsController. Like in MainWindow we call method in TabbedPane next it calls TabsController, and it calls TabbedPane again 🤣. This is confusing and not follow architecture you draw 😭
• It feels like onTabsReorder method implementation has very high complexity (like 2*n^2) and can take a lot of time for lots of tabs and bookmarks, so it is better to optimize it. And in general feels like that we are very often iterating over lists in search of one element, maybe some additional map will help.
• Please do not use ArrayList as a variable type, it is considered a bad practice in Java. I think checkstyle has rule for that (this), I will enable it and fix all such places.

Although, here is a small list of thing for future improvements

All noted. I'll open a new PR in the near future