Skip to:

Opened 14 years ago

Closed 14 years ago

#2022 closed defect (bug) (fixed)

Default Blog Post Image in Activity Stream forced to 100px Height

Reported by: geoffm33's profile geoffm33 Owned by:
Milestone: 1.5 Priority: minor
Severity: Version:
Component: Core Keywords: dev-feedback


When I publish a blog post the image in the activity stream (when applicable) has it's height forced to 100px and scaled based on that. This works fine only when the actual image is 100px in height or taller. For images less than 100px in height the image is blown up.

Attachments (1)

imgheight.png (74.6 KB) - added by geoffm33 14 years ago.
Example of image being scaled up incorrectly

Download all attachments as: .zip

Change History (5)

14 years ago

Example of image being scaled up incorrectly

#1 @cnorris23
14 years ago

  • Component set to Core
  • Keywords dev-feedback added; image height img height activity stream removed
  • Resolution set to fixed
  • Status changed from new to closed

It's not incorrectly scaled up. This is likely done for layout purposes. You are, however, allowed to change this behavior using the 'bp_activity_thumbnail_content_images' filter located in the the bp_activity_thumbnail_content_images() function of bp-activity.php. Close as invalid/wontfix?

#2 @cnorris23
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#3 @boonebgorges
14 years ago

I ran some tests and I don't really think the layout requires a minimum height. I think that not scaling up will be better in most cases; where it's not, admins can use bp_activity_thumbnail_content_images as cnorris23 suggests.

#4 @boonebgorges
14 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [3492]) Don't upsize small blog images in activity stream. Fixes #2022

Note: See TracTickets for help on using tickets.