Skip to content

C608 to WebVTT translation#599

Merged
cfsmp3 merged 5 commits intoCCExtractor:masterfrom
Izaron:webvtt-color
Jan 4, 2017
Merged

C608 to WebVTT translation#599
cfsmp3 merged 5 commits intoCCExtractor:masterfrom
Izaron:webvtt-color

Conversation

@Izaron
Copy link
Copy Markdown
Member

@Izaron Izaron commented Jan 3, 2017

Work in progress - translating CEA-608 subs to WebVTT

Done:

  • Support of the raw old format without colors
  • Added colors and fonts with segments (consecutive characters of the same font or color)

Todo:

  • Replace current "webvtt" to "webvtt-raw" and "webvtt-full" to "webvtt" (???)
  • Add XDS metadata what already computed by CCExtractor and written on screen; and add it in file
  • True positioning as in document
  • Anything else missing from the document

@cfsmp3 Write your suggestions

Evgeny added 5 commits January 2, 2017 20:22
@cfsmp3
Copy link
Copy Markdown
Contributor

cfsmp3 commented Jan 3, 2017

Aren't we missing the actual styling here? CSS or something :-)

@Izaron
Copy link
Copy Markdown
Member Author

Izaron commented Jan 3, 2017

Don't sure that there was CSS style need for CEA-608 (any hint???), by "Styling" I mean support of text color and fonts in output file in proper format. CSS styling is the post-CCExtractor work :)
Don't really think we should redefine default CSS style but if it's necessary for standart C608 look (black backgroun for example) we can just add the line in the header in the code

@cfsmp3
Copy link
Copy Markdown
Contributor

cfsmp3 commented Jan 4, 2017 via email

@cfsmp3 cfsmp3 merged commit 34a21a9 into CCExtractor:master Jan 4, 2017
hrideshmg pushed a commit to hrideshmg/ccextractor that referenced this pull request Mar 12, 2025
* Use rust build by default and release build

* Update windows script

* Update install/ci-vm/ci-windows/ci/runCI.bat

Co-authored-by: Willem <github@canihavesome.coffee>
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