Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Too much hand holding here, got to go.
Too much hand holding here, got to go.
Comments
Comment #1
Jeff Burnz CreditAttribution: Jeff Burnz commentedFixed: http://drupal.org/cvs?commit=369308
Comment #2
eigentor CreditAttribution: eigentor commentedStill, there is need to resize Teaser and Full node images. Did you remove any resizing?
The full node image can become to wide for the content region, especially since we have fluid layout.
Comment #3
Jeff Burnz CreditAttribution: Jeff Burnz commentedMaybe we should just do :
Setting a width on images with CSS feels way wrong to me and like nanny-hand-holding, I think theres always some configuration to do for users, we can make a documentation issue.
Comment #4
eigentor CreditAttribution: eigentor commentedO.K. for Full node this is o.k.
While for teasers things are different: Core does not provide limitless Image Styles (Imagecache presets) and we need to set Teaser images fitting from core.
For Full Node I am with you.
While I can see your point... How did Bartik solve this?
Comment #5
Jeff Burnz CreditAttribution: Jeff Burnz commentedBartik doesnt try to solve it, images just overflow if two sidebars are enabled.
Core does provide unlimited Imagecache presets, just like Imagecache always has, add a new style, go to the field display settings and select the style.
Comment #6
eigentor CreditAttribution: eigentor commentedActually I had another idea: the teaser image provided by core is just too big. So who, if not the core themes, should have it in a usable size.
So how about filing a core patch to scale it to a size that accomodates both Bartik and Corolla (and sure Garland).
We need pretty teaser images out of the box. No way to advide people to learn imagecache before they can post an image.
Comment #7
Jeff Burnz CreditAttribution: Jeff Burnz commentedI agree dude, post an issue, the default imagecache presets are very large indeed. I dont think either of the new themes handle these sizes very well.
Comment #8
Jeff Burnz CreditAttribution: Jeff Burnz commentedLets just postpone this guys, its a core issue since it actually affects all core themes.
Comment #9
eigentor CreditAttribution: eigentor commentedFiled a bug, filed a patch to reduce it to 480px width: #820020: Full node image is too large - reduce to 480px width
Comment #11
Jeff Burnz CreditAttribution: Jeff Burnz commented