Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Eric Claeys' changes to allsky-website #47

Merged
merged 8 commits into from
Aug 6, 2021
Merged

Eric Claeys' changes to allsky-website #47

merged 8 commits into from
Aug 6, 2021

Conversation

EricClaeys
Copy link
Collaborator

Changes discussed with Thomas the end of July, 2021, primarily related to creating and displaying thumbnails.

An image to be used when no thumbnail is found for archived timelapses.  This replaces the aurora thumbnail.
I discussed this with Thomas late July, 2021.
Use the image as a thumbnail if unable to create a thumbnail.
Use the image as a thumbnail if unable to create a thumbnail.
Fixed issue where thumbnails were stretched horizontally on large screens.
If no thumbnail exists, it now displays an image that says there is no thumbnail, rather than displaying an aurora image.
I discussed this with Thomas the end of July, 2021.

Fixed issue where thumbnails were stretched horizontally on large screens.

Removed duplicate image name code.

Left in the legacy "onmouseenter" and "onmouseleave" code, but it's not used.  Thomas can determine if it should be removed (remove $y and <script> block).
Removed "-resize" because the new allsky scripts (which will be submitted soon) no longer use that suffix.
Added "showOverlayAtStartup" which displays the overlay when the page first loads, rather than requiring the user to click on the icon for it.
Uses config.showOverlayAtStartup
Checks if imagecreatefromjpeg() exists and displays an appropriate message if it doesn't, then returns false.   Returns true if it could make the thumbnail.
Changed default imageName.
Added showOverlayAtStartup.
@thomasjacquin thomasjacquin merged commit f581923 into AllskyTeam:master Aug 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants