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

either display granger or adv granger in evo menu #1138

Open
wants to merge 1 commit into
base: master
from

Conversation

@Gireen
Copy link
Member

Gireen commented Nov 28, 2019

Only one of them is needed at the same time depending on if the adv granger is unlocked.

@illwieckz

This comment has been minimized.

Copy link
Member

illwieckz commented Nov 28, 2019

looks good

@@ -1647,6 +1647,12 @@ void CG_Rocket_BuildAlienEvolveList( const char *table )
{
if ( BG_Class( i )->team == TEAM_ALIENS )
{
if ( ( i == PCL_ALIEN_BUILDER0 && ( BG_ClassUnlocked( PCL_ALIEN_BUILDER0_UPG ) && !BG_ClassDisabled( PCL_ALIEN_BUILDER0_UPG ) ) )||

This comment has been minimized.

Copy link
@DolceTriade

DolceTriade Nov 28, 2019

Member

I believe that we should not include this change in the "backend". This is a strictly UI change that should be made in the UI...The circleMenu can use a filter function or something that can filter things out like this...

This comment has been minimized.

Copy link
@Gireen

Gireen Nov 28, 2019

Author Member

Is CG_Rocket_DFCMAlienEvolve the right place for it?

This comment has been minimized.

Copy link
@DolceTriade

DolceTriade Dec 2, 2019

Member

Yeah, it's fine to put it there.

@Gireen Gireen force-pushed the Gireen:advgranger branch from 4ccd1bd to f38fb97 Dec 2, 2019
@illwieckz

This comment has been minimized.

Copy link
Member

illwieckz commented Dec 9, 2019

@DolceTriade is it ok for you now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.