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.

Sign up for GitHub

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

Jump to bottom

[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
TS-Patches

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 •
edited
Loading

@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 •
edited
Loading

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

Before dropship

@Crimsonum Crimsonum mentioned this issue Sep 16, 2022
Open
3 tasks
@CCHyper CCHyper linked a pull request Dec 26, 2022 that will close this issue
Open
@CCHyper CCHyper removed their assignment Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
new-feature New feature or request.
Projects
None yet
Milestone
TS-Patches
4 participants
@Rampastring @Crimsonum @MikillRosen @CCHyper