Remove GIF format for screen capture export (render)

[ariya]

One notable feature of PhantomJS is its ability to take the screenshot of a web page and save it as an image (see WebPage's render function). For the upcoming version 2.0, the only support formats for the image will be JPEG and PNG.

The reason is do remove GIF support is to reduce maintenance burden. A GIF representation of a web page can be produced by converting its PNG capture. There are numerous third-party tools which can do the job better (optimized palette, minimized loss, etc).

[JamesMGreene]

Sounds fine to me. 👍

[zackw]

As long as GIFs in the webpage can still be rendered, +1 from me.

[ariya]

Implemented in master.

[apodgorny]

Why not update docs at http://phantomjs.org/api/webpage/method/render.html?

[mysterycommand]

👍 to @apodgorny 's comment. Please keep your docs up to date! Oh geez, I just noticed the time stamp … it's been almost 2 years since this was implemented, but searching for "phantomjs render gif" leads to (first result) the screen capture docs "Beside PNG format, PhantomJS supports JPEG, GIF, and PDF." … and (second result) the render docs "GIF support depends on the build of Qt used"!

[apodgorny]

@mysterycommand I'vd ditched phantom a while back for reasons known to all who touched it. My favorite now is "nightmarejs w/ electron" – it makes nicer quality snapshots and documentation up to date, it is a lot more intuitive, and much fewer bugs. That day I had to celebrate. )))