cinera_player_pre.js: Allow DOCTYPE html

d420513e · Matt Mascarenhas · e77d2086 · d420513e · d420513e · d420513e
Commit d420513e authored May 02, 2018 by Matt Mascarenhas
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 7 deletions

README.md README.md +0 -5

cinera.c cinera/cinera.c +1 -1

cinera_player_pre.js cinera/cinera_player_pre.js +1 -1

No files found.
--- a/README.md
+++ b/README.md
@@ -109,11 +109,6 @@ directory. Typical operation will involve setting these flags:
 Feel free to play with templates to your heart's content. If you do anything
 invalid, _Cinera_ will tell you what's wrong.

-Note: There is currently an issue sizing the annotation marker container if your
-HTML document contains `<!DOCTYPE html>`. To work around this for now, please do
-not declare a DOCTYPE and instead let the page be displayed in [quirks
-mode](https://www.w3.org/International/articles/serving-xhtml/#quirks).
-
 #### Arguments

    Usage: ./cinera [option(s)] filename(s)

--- a/cinera/cinera.c
+++ b/cinera/cinera.c
@@ -16,7 +16,7 @@ typedef struct
 version CINERA_APP_VERSION = {
    .Major = 0,
    .Minor = 5,
-    .Patch = 47
+    .Patch = 48
 };

 // TODO(matt): Copy in the DB 3 stuff from cinera_working.c

--- a/cinera/cinera_player_pre.js
+++ b/cinera/cinera_player_pre.js
@@ -115,7 +115,7 @@ Player.prototype.jumpToPrevMarker = function() {
 Player.prototype.updateSize = function() {
    var width = this.videoContainer.offsetWidth;
    var height = width / 16 * 9;
-    this.markersContainer.style.height = height;
+    this.markersContainer.style.height = height + "px";
    if (this.youtubePlayerReady) {
        this.youtubePlayer.setSize(Math.floor(width), Math.floor(height));
    }