Jetpack: Remove devicepx-jetpack.js

[enejb]

Currently 'devicepx-jetpack.js' called the zoomImages function every second.
How can we improve this function?

[blobaugh]

@enejb take a gander at https://plugins.trac.wordpress.org/ticket/2031 for an in-depth post by @tollmanz

[jeherve]

I closed the old trac ticket. Let's continue the discussion here.

[kasparsd]

This is a very serious issue because it's forcing the browser to recalculate and redraw the whole layout in case of Safari. Here is the sequence of events:

and here is the offending part:

which is called every second as @enejb mentioned:

[kraftbj]

I'd like us to look at devicepx-jetpack.js overall. Now that Photon supports responsive imaging, how useful is devicepx? Can we reduce/drop usage of devicepx in addition to making this more efficient.

[stale[bot]]

This issue has been marked as stale. This happened because:

• It has been inactive in the past 6 months.
• It hasn’t been labeled `[Pri] Blocker`, `[Pri] High`.

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.

[stefenphelps]

I agree with @kraftbj, there is no need for devicepx.js anymore now that there is broad support for responsive images.

[BrookeDot]

I'll go ahead and re-name this issue "Remove Device devicepx-jetpack.js" Since that sounds like the desired result for this issue.

I would recommend perhaps making the script still filterable but not loaded by default. For example something like add_filter( 'Jetpack_devicepx', true, 10 ); would still load the script.

[jeherve]

I would recommend perhaps making the script still filterable but not loaded by default.

That's the approach we discussed in #1850. I'm going to close this issue as duplicate, since they're mostly the same now.