Opened 10 years ago

Closed 10 years ago

#7776 closed patch (fixed)

Add support for black background behind EIA-708 (ATSC) closed captions

Reported by: Clayton Smith <argilo@…> Owned by: danielk
Priority: minor Milestone: unknown
Component: MythTV - ATSC Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Currently, analog closed captions can be displayed in front of a black background for maximum contrast, but not EIA-708 (ATSC) captions. This patch adds support for a black background behind ATSC closed captions. When the black background option is turned on, captions appearing over light backgrounds are much easier to read.

Attachments (2)

black-background.patch (3.5 KB) - added by Clayton Smith <argilo@…> 10 years ago.
Add support for black background behind EIA-708 closed captions
black-background-2.patch (5.1 KB) - added by Clayton Smith <argilo@…> 10 years ago.
Force black background and white foreground

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by Clayton Smith <argilo@…>

Attachment: black-background.patch added

Add support for black background behind EIA-708 closed captions

comment:1 Changed 10 years ago by Clayton Smith <argilo@…>

To enable the black background, go to Utilities/Setup? --> Setup --> TV Settings --> Playback OSD and enable the "Black background for closed captioning" setting on the third page. This setting previously applied only to analog captions, and I thought it made the most sense to generalize it to both analog and ATSC rather than adding in a new setting.

comment:2 Changed 10 years ago by danielk

Clayton, CEA-708 allows the broadcaster to define both the foreground and background color, to implement this feature we should override both color choices in the 708 window, we don't want to end up using a dark background with a broadcaster's a dark foreground (intended to be on a light background). It would also be nice to be able to override the character outline, so instead of overriding the color one could just force characters to have a contrasting character outline (white for black characters, black for white characters, etc.)

Broadcasters do appear to have trouble sending out readable text, so I think this is a good feature to have. But we also don't want to make the text sent by more "with it" broadcasters to be harder to read.

Changed 10 years ago by Clayton Smith <argilo@…>

Attachment: black-background-2.patch added

Force black background and white foreground

comment:3 Changed 10 years ago by Clayton Smith <argilo@…>

Okay, I've changed the patch to force a white foreground when the black background option is turned on. It works well in my testing. Previously I was seeing grey text; now it's bright white and very readable. With the option turned off, everything is as before.

comment:4 Changed 10 years ago by danielk

(In [23408]) Refs #7776. Render character background boxes.

comment:5 Changed 10 years ago by danielk

Resolution: fixed
Status: newclosed

(In [23409]) Fixes #7776. The patch was not correct, but it got me off my duff to fix this properly. The CCBackground setting is now respected by the CEA-708 caption renderer.

Note: See TracTickets for help on using tickets.