Jump to:
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | image.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| image-large-480.patch | 662 bytes | Idle | PASSED: [[SimpleTest]]: [MySQL] 20,597 pass(es). | View details |
Comments
#1
#2
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.
#3
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.
#4
480px for the default works for me
#5
Yes please.
#6
Committed to CVS HEAD. Thanks.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.