search

jazzhistorydatabase / jhdb-cms

Content management system for jazzhistorydatabase.com
3 stars 0 forks source link

Shadowbox bug #127

Closed matticoli closed 4 years ago

matticoli commented 4 years ago

fix shadowbox bug and remove the overlay. take a basic shadowbox style from template. not from a portfolio page like as is now, where they have to click the center of an empty circle to get a larger image.

ideally we would disable shadowbox for source images less than 800px max dim

drnek commented 4 years ago

yes we def need to disable shadowbox for images under 800px, otherwise it's very disappointing lol.. should we add this as a new bug?

matticoli commented 4 years ago

yes we def need to disable shadowbox for images under 800px, otherwise it's very disappointing lol.. should we add this as a new bug?

How should the smaller images be displayed if not with shadowbox? The template doesn't know the size of the image and ux should be consistent