From 333cb919a8056ece6fa030ff7535d1e8e14e1551 Mon Sep 17 00:00:00 2001
From: Fred Chasen <fchasen@gmail.com>
Date: Tue, 17 Oct 2017 15:43:14 -0700
Subject: [PATCH] Set width to 100% for columns, fixes for pre-paginated

---
 src/contents.js              | 54 +++++++++++++++++-------------------
 src/layout.js                |  4 ---
 src/managers/views/iframe.js |  8 ++++--
 3 files changed, 31 insertions(+), 35 deletions(-)

diff --git a/src/contents.js b/src/contents.js
index b22078b..81c7894 100644
--- a/src/contents.js
+++ b/src/contents.js
@@ -1,5 +1,5 @@
 import EventEmitter from "event-emitter";
-import {isNumber, prefixed, borders} from "./utils/core";
+import {isNumber, prefixed, borders, defaults} from "./utils/core";
 import EpubCFI from "./epubcfi";
 import Mapping from "./mapping";
 import {replaceLinks} from "./utils/replacements";
@@ -193,7 +193,7 @@ class Contents {
 
 	viewport(options) {
 		var _width, _height, _scale, _minimum, _maximum, _scalable;
-		var width, height, scale, minimum, maximum, scalable;
+		// var width, height, scale, minimum, maximum, scalable;
 		var $viewport = this.document.querySelector("meta[name='viewport']");
 		var parsed = {
 			"width": undefined,
@@ -204,6 +204,7 @@ class Contents {
 			"scalable": undefined
 		};
 		var newContent = [];
+		var settings = {};
 
 		/*
 		* check for the viewport size
@@ -211,12 +212,13 @@ class Contents {
 		*/
 		if($viewport && $viewport.hasAttribute("content")) {
 			let content = $viewport.getAttribute("content");
-			let _width = content.match(/width\s*=\s*([^,]*)/g);
-			let _height = content.match(/height\s*=\s*([^,]*)/g);
-			let _scale = content.match(/initial-scale\s*=\s*([^,]*)/g);
-			let _minimum = content.match(/minimum-scale\s*=\s*([^,]*)/g);
-			let _maximum = content.match(/maximum-scale\s*=\s*([^,]*)/g);
-			let _scalable = content.match(/user-scalable\s*=\s*([^,]*)/g);
+			let _width = content.match(/width\s*=\s*([^,]*)/);
+			let _height = content.match(/height\s*=\s*([^,]*)/);
+			let _scale = content.match(/initial-scale\s*=\s*([^,]*)/);
+			let _minimum = content.match(/minimum-scale\s*=\s*([^,]*)/);
+			let _maximum = content.match(/maximum-scale\s*=\s*([^,]*)/);
+			let _scalable = content.match(/user-scalable\s*=\s*([^,]*)/);
+
 			if(_width && _width.length && typeof _width[1] !== "undefined"){
 				parsed.width = _width[1];
 			}
@@ -237,22 +239,24 @@ class Contents {
 			}
 		}
 
+		settings = defaults(options || {}, parsed);
+
 		if (options) {
-			if (options.width || parsed.width) {
-				newContent.push("width=" + (options.width || parsed.width));
+			if (settings.width) {
+				newContent.push("width=" + settings.width);
 			}
 
-			if (options.height || parsed.height) {
-				newContent.push("height=" + (options.height || parsed.height));
+			if (settings.height) {
+				newContent.push("height=" + settings.height);
 			}
 
-			if (options.scale || parsed.scale) {
-				newContent.push("initial-scale=" + (options.scale || parsed.scale));
+			if (settings.scale) {
+				newContent.push("initial-scale=" + settings.scale);
 			}
-			if (options.scalable || parsed.scalable) {
-				newContent.push("minimum-scale=" + (options.scale || parsed.minimum));
-				newContent.push("maximum-scale=" + (options.scale || parsed.maximum));
-				newContent.push("user-scalable=" + (options.scalable || parsed.scalable));
+			if (settings.scalable) {
+				newContent.push("minimum-scale=" + settings.minimum);
+				newContent.push("maximum-scale=" + settings.maximum);
+				newContent.push("user-scalable=" + settings.scalable);
 			}
 
 			if (!$viewport) {
@@ -267,10 +271,7 @@ class Contents {
 		}
 
 
-		return {
-			width: parseInt(width),
-			height: parseInt(height)
-		};
+		return settings;
 	}
 
 
@@ -764,7 +765,7 @@ class Contents {
 		var COLUMN_WIDTH = prefixed("column-width");
 		var COLUMN_FILL = prefixed("column-fill");
 
-		this.width(width);
+		this.width("100%");
 		this.height(height);
 
 		// Deal with Mobile trying to scale to viewport
@@ -801,8 +802,8 @@ class Contents {
 
 	fit(width, height){
 		var viewport = this.viewport();
-		var widthScale = width / viewport.width;
-		var heightScale = height / viewport.height;
+		var widthScale = width / parseInt(viewport.width);
+		var heightScale = height / parseInt(viewport.height);
 		var scale = widthScale < heightScale ? widthScale : heightScale;
 
 		var offsetY = (height - (viewport.height * scale)) / 2;
@@ -811,9 +812,6 @@ class Contents {
 		this.height(height);
 		this.overflow("hidden");
 
-		// Deal with Mobile trying to scale to viewport
-		this.viewport({ width: width, height: height, scale: 1.0 });
-
 		// Scale to the correct size
 		this.scaler(scale, 0, offsetY);
 
diff --git a/src/layout.js b/src/layout.js
index c734210..9581802 100644
--- a/src/layout.js
+++ b/src/layout.js
@@ -171,10 +171,6 @@ class Layout {
 	format(contents){
 		var formating;
 
-		if (this.settings.direction) {
-			contents.direction(this.settings.direction);
-		}
-		
 		if (this.name === "pre-paginated") {
 			formating = contents.fit(this.columnWidth, this.height);
 		} else if (this._flow === "paginated") {
diff --git a/src/managers/views/iframe.js b/src/managers/views/iframe.js
index 63c7b14..1332f33 100644
--- a/src/managers/views/iframe.js
+++ b/src/managers/views/iframe.js
@@ -247,12 +247,14 @@ class IframeView {
 
 		if(!this.iframe || this._expanding) return;
 
-		if(this.layout.name === "pre-paginated") return;
-
 		this._expanding = true;
 
+		if(this.layout.name === "pre-paginated") {
+			width = this.layout.columnWidth;
+			height = this.layout.height;
+		}
 		// Expand Horizontally
-		if(this.settings.axis === "horizontal") {
+		else if(this.settings.axis === "horizontal") {
 			// Get the width of the text
 			width = this.contents.textWidth();