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

allow gamemode mods to use banners as mode images and patches as icons #836

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

JMM889901
Copy link
Contributor

@JMM889901 JMM889901 commented Aug 12, 2024

image

Modifies menu_mode_select to add GetPlaylistBannerImage and RegisterPlaylistBannerImage, also modifies ModeButton_GetFocus to show/hide correct rui since basic_menu_image.rpak cannot show banner images, also adds mode_select.menu, only difference here is NextModeImageCallsign which is just an identical panel using the "ui/callsign_icon_button.rpak" rui instead of basic menu image

ImageTest.zip
test mod

JMM889901 and others added 7 commits August 12, 2024 17:57
i hate github i hate github i hate github
some lines were incorrectly tabbed
Only change from default should be 
```
		NextModeIconPatch
		{
			ControlName				RuiPanel
			wide					72
			tall					72
			visible					0
			scaleImage				1
			image					""

			rui                     "ui/basic_image_add.rpak"

			pin_to_sibling			NextGameModeName
			pin_corner_to_sibling	BOTTOM_RIGHT
			pin_to_sibling_corner	TOP_RIGHT
		}

```
@JMM889901 JMM889901 changed the title More mode images allow gamemode mods to use banners as mode images and patches as icons Aug 12, 2024
@JMM889901 JMM889901 marked this pull request as ready for review August 12, 2024 17:45
@JMM889901
Copy link
Contributor Author

Seems to work but i would test, some funky stuff happened trying to this is work locally with it just not updating some files and causing random stuff to crash

@JMM889901
Copy link
Contributor Author

This is an old PR fixed up, Unfortunately, i did not include the original unmodified mode-select menu file in a separate commit :boowomp:

@GeckoEidechse GeckoEidechse added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Aug 13, 2024
@GeckoEidechse GeckoEidechse added the commits vanilla file For PRs that commit vanilla files from VPKs label Sep 2, 2024
@github-actions github-actions bot added the merge conflicts Blocked by merge conflicts, waiting on the author to resolve label Sep 9, 2024
@Alystrasz
Copy link
Contributor

@JMM889901 merge conflicts atm

@GeckoEidechse
Copy link
Member

@JMM889901 if you resolve merge conflicts I can add the necessary vanilla files to get this PR merged o7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commits vanilla file For PRs that commit vanilla files from VPKs merge conflicts Blocked by merge conflicts, waiting on the author to resolve needs code review Changes from PR still need to be reviewed in code needs testing Changes from the PR still need to be tested
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants