The 'Show Profiling Data' button should reuse a single Table Inspector instance

WowRarity / Rarity

Collectibles and statistics tracking addon for World of Warcraft

https://www.curseforge.com/wow/addons/rarity

GNU General Public License v2.0

56 stars 35 forks source link

The 'Show Profiling Data' button should reuse a single Table Inspector instance #783

Open rdw-software opened 1 month ago

rdw-software commented 1 month ago

Source: Discord (DM)

Dack: Why can I open so many new windows? lol RDW: [...] I completely forgot about that TODO :P RDW: It might suffice to cache the instance and reuse it (or show/hide, not sure what the inspector API supports)

https://github.com/user-attachments/assets/e2e6e04f-3a97-4b6c-9ee1-f715d6b6ae95

Seems that I deliberately ignored this problem when adding the button: There's a TODO comment indicating as much.