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

[New Feature] Displaying briefing screen on mission start #28

Open
Rampastring opened this issue May 6, 2021 · 4 comments · May be fixed by #945
Open

[New Feature] Displaying briefing screen on mission start #28

Rampastring opened this issue May 6, 2021 · 4 comments · May be fixed by #945
Labels
new-feature New feature or request.
Milestone

Comments

@Rampastring
Copy link
Member

Description:

Red Alert displays the briefing screen on mission start after the FMVs, which is handy. Tiberian Sun does not do this, which means that the player has to manually press Options -> Restate Briefing at the start of each mission, unless they realized the objectives from potential FMVs (which no mod has to a significant degree).

Tiberian Sun should also display the briefing screen on mission start.

Optional User/Mission Configuration:

This could potentially be made configurable through SUN.INI or even with a mission-based key ([Basic] ShowBriefingOnMissionStart= for example), although I personally would just leave it always enabled.

Additional Files:

Implemented in CnCNet ts-patches: https://github.com/CnCNet/ts-patches/blob/master/src/briefing_screen_mission_start.asm

@Rampastring Rampastring added the new-feature New feature or request. label May 6, 2021
@CCHyper
Copy link
Contributor

CCHyper commented May 6, 2021

As we do not want to change the default behaviour of the game unless it is an optional QOL change, I vote for adding this enabled behind a new scenario INI key.

@Crimsonum
Copy link

Didn't Red Alert also do this only for missions that did not have an FMV intro cutscene (or at least one that didn't explain the mission)? I would also make this optional.

@CCHyper CCHyper added this to the TS-Patches milestone May 11, 2021
@CCHyper CCHyper self-assigned this Jun 6, 2021
@CCHyper
Copy link
Contributor

CCHyper commented Jun 13, 2021

@Crimsonum @Rampastring

Didn't Red Alert also do this only for missions that did not have an FMV intro cutscene (or at least one that didn't explain the mission)? I would also make this optional.

You are absolutely correct. See the released Red Alert source code.
https://github.com/electronicarts/CnC_Remastered_Collection/blob/7d496e8a633a8bbf8a14b65f490b4d21fa32ca03/REDALERT/SCENARIO.CPP#L384

It would be easy to restore this code and add a new [Basic] section INI key to show this.

Another question; Should this be shown before or after the Dropship Loadout?

@MikillRosen
Copy link

MikillRosen commented Jun 14, 2021

Another question; Should this be shown before or after the Dropship Loadout?

Before dropship

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature New feature or request.
Projects
None yet
4 participants