Remove the |el| property in PDFPageView and PDFThumbnailView

[Snuffleupagus]

Obviously it would have been nicer to just remove it all together, but in the event that some third-party implementation depends on it, I figured that we might want to keep it around for GENERIC builds for now.

[CodingFabian]

how will a developer know that this has been deprecated?
right now i would even prefer to remove this (its not really public api). Developers building on the code would quickly see that el no longer exist and could easily rewrite to use the div element.

[timvandermeij]

I agree with @CodingFabian. I'm fairly sure no-one is using this, and if so they will notice soon enough that it has been deprecated if we remove it. If we keep enabling it for generic builds, most people wouldn't notice that it's deprecated I think.

[Snuffleupagus]

I only left el for GENERIC builds as a matter of consistency, since we've done similar things with other code, e.g. this and also this. But in this particular case, I don't have a strong opinion either way.

[brendandahl]

We're not currently versioning the viewer API and we warn about this in the getting started code. Let's just wipe this out.

[Snuffleupagus]

It's gone now!

/botio-windows preview

[pdfjsbot]

From: Bot.io (Windows)

---

Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.22.172.223:8877/4b909cc4e3db544/output.txt

[pdfjsbot]

From: Bot.io (Windows)

---

Success

Full output at http://107.22.172.223:8877/4b909cc4e3db544/output.txt

Total script time: 1.15 mins

Published

• Viewer: http://107.22.172.223:8877/4b909cc4e3db544/web/viewer.html
• B2G Viewer: http://107.22.172.223:8877/4b909cc4e3db544/extensions/b2g/content/web/viewer.html
• Extension: http://107.22.172.223:8877/4b909cc4e3db544/extensions/firefox/pdf.js.xpi

[yurydelendik]

Thank you