Multiplayer icon indicating copy to clipboard operation
Multiplayer copied to clipboard

Minor maintenance on CaravanSplittingProxy

Open SokyranTheDragon opened this issue 2 years ago • 1 comments

  • The cancel button will be red to match the behaviour of other session dialogs
  • The proper translation key is used for travel supplies tab
  • GUI.Begin/EndGroup was replaced by Widgets.Begin/EndGroup to match vanilla dialog (it seems Widgets call fixes some Unity bugs on Linux/Steam Deck)
  • Removed a lot of temporary variables in DoBottomButtons
  • Cleaned up CaravanUIUtility.DrawCaravanInfo call by removing default values, removing new int?(), and inverting the conditional operator

SokyranTheDragon avatar Nov 24 '23 15:11 SokyranTheDragon

Can you take a look at the conflict?

Zetrith avatar Apr 16 '24 19:04 Zetrith

Oops, completely forgot about this. Will try to fix it in the next 1-2 days.

SokyranTheDragon avatar May 18 '24 21:05 SokyranTheDragon

I think that this should be it. I've tested it and it seemed fine on my end.

SokyranTheDragon avatar May 19 '24 15:05 SokyranTheDragon