evaera / matter

A modern ECS library for Roblox.
https://eryn.io/matter/
MIT License
144 stars 34 forks source link

Improve usability of world inspect widget #95

Closed memorycode closed 8 months ago

memorycode commented 8 months ago

Includes some minor improvements to the world inspect widget, though larger changes to the debugger are necessary in the future.

The ability to sort components by their count is also included, but there is currently no way to switch to this. Should be included in the future once the Plasma table widget lets you detect header selection.

image
memorycode commented 8 months ago

There was some discussion if the Size label (and subsequently the empty entity counter) is even necessary. I feel if we decide to keep the Size label, we should include the empty entity counter since its hard to use the Size label without it. Otherwise if we decide to remove the Size label, we can also ditch the empty entity counter.

Personally, I think that the number of entities in the world is useful information to have. I think that there are better ways to present this information (and more context that should be included), but those are changes that should be made alongside bigger debugger changes.

jackTabsCode commented 8 months ago

@memorycode Could you make linting pass? Then I think we should be able to merge this. Nice work!

memorycode commented 8 months ago

@memorycode Could you make linting pass? Then I think we should be able to merge this. Nice work!

Should be passing now. Thanks for the reminder :)

As a sidenote, I did the work necessary to support sorting via the table headers here: https://github.com/evaera/plasma/pull/15 (might be a while before a new Plasma release is cut..)