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

Thumbnails are displayed even though display thumbnails is off in redmine settings #29

Open
jdkemme opened this issue Jul 14, 2016 · 3 comments

Comments

@jdkemme
Copy link

jdkemme commented Jul 14, 2016

Redmine 3.3

setting

thumbnail

@tofi86
Copy link
Collaborator

tofi86 commented Jul 14, 2016

Is this a new 3.3 setting? never noticed this before...

Anyways, you're right, this setting should be taken into account...

@jdkemme
Copy link
Author

jdkemme commented Jul 14, 2016

This doesnt appear to be a new setting in 3.3. The behavior of the setting in redmine, when enabled, displays an image thumbnail preview. When the setting is unchecked, it removes the preview entirely.

Lightbox behavior honors this setting- when checked it displays the image at the thumbnail size. However when unchecked, unlike core redmine, it displays the image at hard coded width => '180' in _links.html.erb in the 'else' condition. To mimic core redmine behavior, the else condition that adds <div class='images'> should be removed, or at least made optional via a setting.

@tofi86
Copy link
Collaborator

tofi86 commented Jul 17, 2016

Oh, yeah, you're right... This has been the behaviour ever since I took over the redmine_lightbox project and forked it to redmine_lightbox2.

While I agree with you that this setting should be taken into account, the current behaviour is still a great solution for those who cannot use imagemagik/rmagick on their system on which the thumbnail function depends...

Will think about that for the next release...

dani-sanchez-dorado added a commit to dani-sanchez-dorado/redmine_lightbox2 that referenced this issue May 11, 2018
dani-sanchez-dorado added a commit to dani-sanchez-dorado/redmine_lightbox2 that referenced this issue May 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants