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

Add mod settings option to toggle main menu promos #784

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

itscynxx
Copy link
Contributor

@itscynxx itscynxx commented Jan 14, 2024

A much, much cleaner implementation of how I did it in vanillaplus lol
I originally found out this was possible due to a bug in vanillaplus where it would attempt to get the promos from Northstar, and fail (as it doesn't have a connection to ms)

Mod settings menu:

image

Setting set to On:

image

Setting set to Off, after reloading:

image

If anyone has any sort of workaround to needing to reload mods after setting it, feel free to let me know, but I'm not sure if there is (apart from making Mod Settings uiscript_reset upon leaving

Notes:

The convar is added in Northstar.Client as that's where it's used. If someone (say, an old vanilla+ install) were to use Northstar.Client, where the convar is used, but not have Northstar.Custom, where it's added in Mod Settings, they would compile error with an unregistered convar when hitting the main menu

Why?

I like how it looks lol. I would make a mod for it, but I really don't want to deal with maintaining a main menu mod and keeping it up to date. The promos are also still enabled by default

Copy link
Contributor

@Alystrasz Alystrasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

I dunno if this is planned but the option is not taken into account after rebooting the game, making main menu promos appear again.
I could notice that mod settings values do not appear without a mod reload (maybe this is why the option isn't taken into account?); this behavior however appears without this PR, which means it wasn't introduced by it.

@@ -5,4 +5,7 @@ void function NSCustomModSettings()
ModSettings_AddModTitle( "Northstar Custom" , 2 )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess the new option should be listed under a Northstar Client category

@ASpoonPlaysGames
Copy link
Contributor

the option is not taken into account after rebooting the game, making main menu promos appear again.

The convar needs FCVAR_ARCHIVE for this, the convar above in the file does the same thing

},
{
"Name": "menu_show_promos",
"DefaultValue": "1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As suggested by @ASpoonPlaysGames in #784 (comment), this change makes the parameter persist through game reboots (tested and approved, thanks again Spoon)

Suggested change
"DefaultValue": "1"
"DefaultValue": "1",
"Flags": "ARCHIVE_PLAYERPROFILE"

@GeckoEidechse
Copy link
Member

Given that we started updating mainmenu promos a bit more to mention blog posts etc I'm not sure if this PR is still needed/wanted.

@GeckoEidechse
Copy link
Member

Setting as draft for now until requested changes are addressed or a decision has been made whether we want this or not ^^

@GeckoEidechse GeckoEidechse marked this pull request as draft September 4, 2024 10:13
@github-actions github-actions bot added the merge conflicts Blocked by merge conflicts, waiting on the author to resolve label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge conflicts Blocked by merge conflicts, waiting on the author to resolve
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

6 participants