CLIDragon
CLIDragon
I'm not sure why this is failing. Building succeeds on my machine, and I've included point.h in all of the referenced files.
Build failures appear to be unrelated, though I would prefer to rerun to check.
Merge conflicts resolved.
> Is clang angry from this PR or some other reason? Yes, it's this PR. I should probably mark as draft until I get around to fixing the issues.
This is failing on `vehicle_split_test` again and should be fine to merge.
Marking as draft for now, as I suspect that linked PR includes everything I mentioned here.
It definitely does. This is what I get for not checking open PRs before I make something 😿.
> I‌ suppose we can go ahead and do this. I had been intending to investigate to find out where the extra spacing comes from, since it indicates to me...
> Sure, it apparently comes from `ItemSpacing` but if it’s supposed to be there why doesn’t `CalcTextSize` include it? That's easy, CalcTextSize only calculates the size of the text. See...
`MOVE_SINGLE_ITEM` being slow is unavoidable. For `MOVE_VARIABLE_ITEM` and `MOVE_ITEM_STACK`, it should be possible to create a separate function similiar to `MOVE_ALL_ITEMS` that avoids invalidating the UI.