Helioviewer-Project / helioviewer.org

Main web application for solar data visualization
Other
18 stars 15 forks source link

Fix some instances of embedded helioviewer #537

Closed dgarciabriseno closed 5 months ago

dgarciabriseno commented 5 months ago

Summary

For #536

This fixes the "embed" portion of Helioviewer being embedded in pages. The minimal view is still having issues being embedded.

Helioviewer was failing to load because it attempted to access the localStorage object on the window, which was denied.

Helioviewer was already design to work without localStorage support, but it failed to catch this instance of it not being accessible.

There were also issues positioning the earth scale since some HTML elements that it used as a reference don't exist.

Testing

Enter the browser versions this change was tested on below.

browser version
firefox 115.7.0esr (64-bit)
chrome 122.0.6261.69 (Official Build) (arm64)
safari 17.2.1 (18617.1.17.11.12, 18617)
edge untested