The "large" image preset stretches to 640 px width. While this may be fine for some themes, it is definitely too big for many.
http://screencast.com/t/ZDgxZjBlZDct
Stretching it to the full width of the content area may be a good goal, but in doubt it should be rather smaller than wider.

So how about reducing it to 540x540 width. An image of the size 640x640 will not be the rule on most sites anyway, it is just too ginormous, and maybe an even smaller size, but somewhat more sqare like 480x480 might be more adequate.

So here is how it looks with 480px width and one sidebar:
http://screencast.com/t/ZGVhMWY0YzEt
And like this with 540, which should be enough for any big-image-lover:
http://screencast.com/t/MDY3ZDhkM2It

We had ugly Html image resizing, but this is just too bad #803934: Remove 300px width limit of images

CommentFileSizeAuthor
image-large-480.patch662 byteseigentor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eigentor’s picture

Status: Active » Needs review
eojthebrave’s picture

Status: Needs review » Reviewed & tested by the community

I think changing the default size to 480px is a good plan. 640px is huge, and rarely used outside of an image gallery or similar use case.

Patch looks good to me.

Dries’s picture

I'm OK with that. I agree that 640x640 will be too large for most sites. I can commit this later on -- want to leave a bit more time for feedback.

mikeytown2’s picture

480px for the default works for me

yoroy’s picture

Yes please.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.