Project

General

Profile

Actions
Copy link

Defect #26379

closed

Fix thumbnail rendering for images with height >> width

Added by Gabriel Cardoso almost 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Jean-Philippe Lang
Category:
Attachments
Target version:
3.4.2
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:
3.4.0

Description

The ImageMagick format rule being `100x100>` (given 100 as thumbnail size in settings), when uploading images with a height much greater than the width (like websites templates) the thumbnail will have a height of 100px and a computed width so homothety is respected.

In the `thumbnail_tag` helper from `ApplicationHelper`, the width attribute is set width `Setting.thumbnails_size`. I suggest to remove this attribute from the image tag since the source thumbnail has the right size.

Am I missing some cases where this change could have an important impact ?

Files

Download all files
very high image example.jpg (89.6 KB) very high image example.jpg Gabriel Cardoso, 2017-07-07 12:02
26379-3_3@2x.png (15.7 KB) 26379-3_3@2x.png Go MAEDA, 2017-07-08 10:17
26379-3_4@2x.png (64.6 KB) 26379-3_4@2x.png Go MAEDA, 2017-07-08 10:17
defect-26379.diff (775 Bytes) defect-26379.diff Go MAEDA, 2017-07-09 08:37
Actions
Copy link
 #1

Updated by Pavel Rosický almost 7 years ago

you can disable imagemagick and then you won't be able to resize thumbnails

bundle install --without rmagick

Actions
Copy link
 #2

Updated by Go MAEDA almost 7 years ago

Gabriel Cardoso wrote:

In the `thumbnail_tag` helper from `ApplicationHelper`, the width attribute is set width `Setting.thumbnails_size`. I suggest to remove this attribute from the image tag since the source thumbnail has the right size.

The width attribute is workaround for strange behavior of Firefox. Please see #24927#note-6 and #24927#note-8 for details.

Actions
Copy link
 #3

Updated by Go MAEDA almost 7 years ago

  • Tracker changed from Patch to Defect
  • Affected version set to 3.4.0
Actions
Copy link
 #4

Updated by Go MAEDA almost 7 years ago

Redmine 3.3 (displayed with 25px * 100px):

Redmine 3.4 (displayed with 100px * 392px):

Actions
Copy link
 #5

Updated by Go MAEDA almost 7 years ago

Here is a patch to fix this issue.
Thumbnails are displayed in the same way as in Redmine 3.3 if the patch applied.

Actions
Copy link
 #6

Updated by Jean-Philippe Lang almost 7 years ago

  • Target version changed from 3.4.1 to 3.4.2

Waiting for some feedback before merging it.

Actions
Copy link
 #7

Updated by Gabriel Cardoso almost 7 years ago

  • Status changed from New to Resolved

Perfect ! Thank you !

Indeed, your approach is better and solve the proble. Hope it doesn't break #24927#note-6 and #24927#note-8 on Firefox

Actions
Copy link
 #8

Updated by Go MAEDA almost 7 years ago

  • Status changed from Resolved to Confirmed

Gabriel Cardoso wrote:

Indeed, your approach is better and solve the proble. Hope it doesn't break #24927#note-6 and #24927#note-8 on Firefox

No problem. Setting both max-width and max-height resolved #24927#note-6.
Thank you for your feedback!

"Resolved" status is used by core developers, I am setting the status to "Confirmed".

Actions
Copy link
 #9

Updated by Jean-Philippe Lang almost 7 years ago

  • Subject changed from Fix thumbnail rendering to Fix thumbnail rendering for images with height >> width
  • Status changed from Confirmed to Resolved
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Patch committed, thanks.

Actions
Copy link
 #10

Updated by Jean-Philippe Lang almost 7 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF