Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KSP 1.4 compatibility: #2241

Merged
merged 2 commits into from Mar 13, 2018
Merged

Conversation

Dunbaratu
Copy link
Member

@Dunbaratu Dunbaratu commented Mar 7, 2018

Fixes #2240

0 - To compile this, remember to drop in the new dll's from
KSP 1.4 into your Resources first.

1 - Unity dropped the WWW.audioClip method and obsoleted it.
SQUAD apparently added an extension method to WWW called
WWW.GetAudioClip() that does the same thing and works.

2 - SQUAD added a new type: HsvColor, in the root namespace,
that masked our own HsvColor class. Rather than using "using"
statements or explicit namespaces, I renamed our HsvColor to
HsvaColor, since technically that is a better name that matches
up with our RgbaColor class.

 0 - To compile this, remember to drop in the new dll's from
 KSP 1.4 into your Resources first.

 1 - Unity dropped the WWW.audioClip method and obsoleted it.
 SQUAD apparently added an extension method to WWW called
 WWW.GetAudioClip() that does the same thing and works.

 2 - SQUAD added a new type: HsvColor, in the root namespace,
 that masked our own HsvColor class.  Rather than using "using"
 statements or explicit namespaces, I renamed our HsvColor to
 HsvaColor, since technically that is a better name that matches
 up with our RgbaColor class.
@ghost ghost mentioned this pull request Mar 9, 2018
@hvacengi hvacengi added this to the v1.1.6.0 milestone Mar 10, 2018
@hvacengi hvacengi mentioned this pull request Mar 13, 2018
@hvacengi hvacengi merged commit f3f8953 into KSP-KOS:develop Mar 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants