Modify

Ticket #7776 (closed patch: fixed)

You must read the TicketHowTo before creating a new ticket or commenting on an existing ticket.

Opened 2 years ago

Last modified 2 years ago

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

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

Change History

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

Add support for black background behind EIA-708 closed captions

comment:1 Changed 2 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 2 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 2 years ago by Clayton Smith <argilo@…>

Force black background and white foreground

comment:3 Changed 2 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 2 years ago by danielk

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

comment:5 Changed 2 years ago by danielk

  • Status changed from new to closed
  • Resolution set to fixed

(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.

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.