sul-dlss / sul-embed

An oEmbed Service for Stanford University Libraries
Other
19 stars 6 forks source link

Consider removing fscreen as a dependency #2178

Open jmartin-sul opened 2 weeks ago

jmartin-sul commented 2 weeks ago

As of this ticket's creation, the most recent commit to the package is 3 years old. So it's not clear that it's maintained.

Unclear whether the browser APIs for full screen have converged enough to obviate the desire for this sort of cross-browser layer.

OTOH, it seems to still work just fine for us. So likely not a terribly high priority.

spawned from looking into #2157 and #2162