search

documentcloud / document-viewer

The NYTimes Document Viewer
http://open.blogs.nytimes.com/2010/03/27/a-new-view-introducing-doc-viewer-2-0/
Apache License 2.0
690 stars 119 forks source link

Padding is screwed up. #44

Open jashkenas opened 10 years ago

jashkenas commented 10 years ago

Example: http://www.nytimes.com/interactive/2014/07/30/world/africa/31kidnap-docviewer2.html

Padding-left on .DV-docViewer .p0, .DV-docViewer .p1, .DV-docViewer .p2 is currently 60px. It looks like it wants to be 40.

samuelclay commented 10 years ago

They should be using auto zoom instead. Looks like it's a hard-coded width.

DV.load("//www.documentcloud.org/documents/1236725-abubasirletter.js", {
  width: 970,
  height: 800,
  text: false,
  pdf: false,
  container: "#DV-viewer-1236725-abubasirletter"
});
jashkenas commented 10 years ago

The same problem appears if you simply narrow any document in a new window on DocumentCloud.org. It starts cutting off the right side instead of centering nicely.

knowtheory commented 10 years ago

Yeah, i see it. I gotta get my windows vms back up and running on my (new) machine so i can test against some IEs.