Skip to content

Simplify launch option labels for new users, and add tooltips#32

Open
gyroplatter wants to merge 1 commit intoBenchmarkSims:developfrom
gyroplatter:gyroplatter/natural-language-launch-options
Open

Simplify launch option labels for new users, and add tooltips#32
gyroplatter wants to merge 1 commit intoBenchmarkSims:developfrom
gyroplatter:gyroplatter/natural-language-launch-options

Conversation

@gyroplatter
Copy link
Copy Markdown
Contributor

Changed 'launch option' labels to be easier to understand for new users.
For example, 'nomovie' is an EXE target addition which probably goes over people's head. A simpler 'movies on or off' type of option would have people question it less.

I added tooltips to the launch options, RTT, and override checkboxes and radio buttons (what I see people making mistakes with or questioning).

Foreground="{StaticResource UiForegroundBrush}">
<CheckBox.ToolTip>
<ToolTip MaxWidth="300">
<TextBlock Text="Starts Falcon BMS without applying any options or controls set within the Alternative Launcher."
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We're trying to move away from calling this "Alternative Launcher" now that it ships with BMS.

Maybe just

"Starts Falcon BMS without applying any changes made here."

Style="{StaticResource LauncherRadioButton}">
<RadioButton.ToolTip>
<ToolTip MaxWidth="300">
<TextBlock Text="Disables cockpit display (MFDs/HUD/etc.) export to external monitors and devices."
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[suggestion] rearrange wording slightly

"Disables export of cockpit display (MFDs/HUD/etc.) to external monitors and devices."

Style="{StaticResource LauncherRadioButton}">
<RadioButton.ToolTip>
<ToolTip MaxWidth="300">
<TextBlock Text="Enables cockpit display (MFDs/HUD/etc.) export to external monitors and devices."
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[suggestion] rearrange wording slightly (see above)

<!-- TOOLTIP -->
<Controls:ToggleSwitch.ToolTip>
<ToolTip MaxWidth="300">
<TextBlock Text="Records Air Combat Maneuvering Instrumentation flight data (position, maneuvers, combat events) for playback and debriefing. Enabling this writes .VHS files BMS's \User\Acmi\ folder."
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think .vhs files are on the brink of extinction.. we record .zip.acmi by default now, in a format that is defined by TacView. I don't think we should get into all that...

Also, there are a variety of ways to begin/end recording ACMI while in-flight. And different output directories.. even realtime streams.. Let's also not get into all that.

What this cmd line option does, is to start ACMI recording automatically when entering 3d. The main use case is for MP servers.. ie. typically a C-130 or something just sitting on the tarmac. But maybe a headless cloud server with g_bNoRender, and it's difficult for the MP host to manually start ACMI recording.

(Why is this a cmd line option when g_bNoRender is a cfg? I have no idea.)

For the tooltip, I would say something simple and direct like

Begin recording ACMI automatically, when entering 3d.  (Helpful for multiplayer servers.)

Copy link
Copy Markdown
Collaborator

@arithex arithex left a comment

Choose a reason for hiding this comment

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

some comments and suggestions

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.

2 participants