X Tutup
Skip to content

Misc. improvements#5937

Merged
sturnclaw merged 12 commits intopioneerspacesim:masterfrom
sturnclaw:microfixes
Oct 13, 2024
Merged

Misc. improvements#5937
sturnclaw merged 12 commits intopioneerspacesim:masterfrom
sturnclaw:microfixes

Conversation

@sturnclaw
Copy link
Member

This is a batch of changes and improvements harvested from my equip-v2 branch (#5734). I'm pulling these out to make the rebase workload less insane, as they are all standalone changes with limited scope and are uncontroversial in nature.

Note that many of these additions don't appear to have users (yet) - the downstream code still lives in equip-v2 awaiting merge.

I don't expect much in the way of PR review here and plan to merge in <24hrs unless some sharp eye spots a bug of some sort.

- Add missing treePush(id) to complement treePop()
- Add basic functions to access ImGui's state storage. Limited to booleans for now with automatic ID scoping.
- Fix ui.invisibleButton() requiring an explicit flag parameter even if empty
- Small delay before displaying button tooltip, better matches the rest of the UI
- Consistent font choice with rest of UI
- Ensure the tab function receives a reference to the self argument
- Reload only the actively selected tab when pressing Ctrl+R
- Allows taking an existing table value and making a copy with some values overridden
- Primary usecase is "inline modification" of a plain table (i.e. not a prototype or class instance)
- Make table.merge() and table.append() more efficient by only checking validity of the predicate once
- Add helpers to iterate a table and return the value with the greatest or least score according to the passed scoring function along with the "winning" score
- Add constant-time class inheritance lookup via cached inheritance map
- Fix erroneous use of table.append() instead of table.merge()
- Add type information to Event.Register
- Add additional method prototypes to Ship.meta.lua
@sturnclaw sturnclaw merged commit 2884328 into pioneerspacesim:master Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

X Tutup