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

On Card Pressed Handler #164

Merged
merged 1 commit into from Aug 8, 2023
Merged

Conversation

Morji789
Copy link
Contributor

@Morji789 Morji789 commented Aug 7, 2023

An handler when card pressed, the same as OnButtonPressed

example:

ScaleformUI.Scaleforms.JobMissionSelector.Cards[1].OnCardPressed = function()
     print("Card Pressed")
end

The Same As OnButtonPressed Handler
@manups4e
Copy link
Owner

manups4e commented Aug 8, 2023

this is awesome! Could you add the C# event too? if not i'll merge and do it myself.. i'd love to have all in 1 commit

@Morji789
Copy link
Contributor Author

Morji789 commented Aug 8, 2023

I'm not strong at C#, i would rather you doing it

@manups4e
Copy link
Owner

manups4e commented Aug 8, 2023

a good opportunity to start learning the language of the Masters u.u anyway don't worry i'll handle it

@manups4e manups4e merged commit da7b3f8 into manups4e:master Aug 8, 2023
@manups4e
Copy link
Owner

manups4e commented Aug 8, 2023

also.. i just noticed you handled JobSelectionCard.. what about the 3 buttons?
EDIT: nvm.. buttons already have the event.. it's only the card missing it.

@Morji789
Copy link
Contributor Author

Morji789 commented Aug 8, 2023

a good opportunity to start learning the language of the Masters u.u anyway don't worry i'll handle it

I will don't worry

@Morji789 Morji789 deleted the morji-pullrequest branch August 8, 2023 13:04
@manups4e
Copy link
Owner

manups4e commented Aug 8, 2023

also i would suggest testing your code before doing a pull request.. i didn't have time to check.. and your code resulted in an error...
image
by missing a comma.

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