https://bugzilla.wikimedia.org/show_bug.cgi?id=31775
--- Comment #5 from Brion Vibber <[email protected]> 2011-10-21 23:26:23 UTC --- Made a few comments on the rev -- I'd recommend breaking this into several separate bugs and treating them separately. The page rotation looks like it requires patching third-party programs -- this would seem to mean that it won't work with a default poppler-utils install, and would make deploying it a lot harder. There are a bunch of introductions of register_globals vulnerabilities -- do not want. :) I'm a bit unsure what this multiple-thumbnail thing is for; it's not even mentioned in your descriptions. Not sure if this is a desirable thing, and if it is I'm not sure how much difficulty it would cause with compatibility. Thumbnails are rendered via a 404 handler on major installs like Wikimedia, and we need consistent naming that's understood on the back and front end; this may not work with that. What's the benefits of the direct rendering from ImageMagick? Are there any reasons that wasn't done in the first place, such as file size, memory usage, etc? What does "Thumbnails now link to correct individual pages of PDF." actually mean? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. You are on the CC list for the bug. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
