Skip to content

Conversation

@SteveErl
Copy link
Contributor

This change adds a new configuration option for mythfrontend closed captioning. Setup -> Video -> Playback OSD -> Always display Closed Captioning or Subtitles is switched to Default to display Closed Captioning or Subtitles and the user is able to pick a new configuration option.

A LastCaptions entry is added into the Settings database. This boolean gets updated each time a user toggles captions on or off, and indicates whether captions were enabled or disabled the last time it changed. The m_captionsEnabledbyDefault variable is switched from a boolean to an unsigned int so that it can hold 0, 1, or 2. It's set to 2 when the Last Captions State configuration is selected. When starting up viewing of a video recording or Live TV, by default, captions are either disabled (0), enabled (1), or set to match the last caption state (2).

When a user picks Last Captions State, mythfrontend will remember whether captions were enabled or disabled the last time something was watched, and will carry over that state for the next thing to be watched. Turn on captions, and they will continue to appear until you explicitly turn them off. Turn off captions and they will not appear until you explicitly turn them on.

Resolves: #1154

Checklist

This solution adds a LastCaptions entry into the Settings
database. This boolean gets updated each time a user toggles
captions on or off, and indicates whether captions were
enabled or disabled the last time it changed. The
m_captionsEnabledbyDefault variable is switched from a
boolean to an unsigned int so that it can hold 0, 1, or 2.
It's set to 2 when the Last Captions State configuration
is selected. When starting up viewing of a video recording
or Live TV, captions are either disabled (0), enabled (1),
or set to match the last caption state (2).

Resolves: MythTV#1154
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.

Option to default to Last Captions State

1 participant