I noticed that there was always at least one blank line of text after the displayed address, and found that at the end of theme_cck_address there are two extra HTML line breaks added on: (line number 1230 in cck_address.module v5.x-2.1)

    $output .= '<br /><br />';

I know it's easy to override that theme function and remove the line, but I can't figure out why it's even there to begin with. If it was for spacing, then that should be done by the CSS, not by empty lines of text. Hard-coding in empty lines like that isn't very themer friendly, so I'm suggesting that the line be removed.

thanks, -ashtonium

CommentFileSizeAuthor
#4 Addition of line break error.jpg46.42 KBjmangion

Comments

rconstantine’s picture

Category: bug » feature

It seems like there was a good reason at the time, but I don't recall what. It may be that I was experimenting and left it in. More specifically, I recall experimenting with allowing 'multiple' addresses like some other cck fields allow multiple values. It turned out to be very impractical and would have more than doubled the code.

However, I would not call this a bug (since it can be overridden and does not impair the operation of the module at all), but a feature request that I'm happy to add to the pile. So unless I can come up with a very good reason to keep it, I'll remove it from the next release.

ashtonium’s picture

Sounds good; thanks for the consideration.

NewZeal’s picture

Status: Active » Closed (fixed)

fixed

jmangion’s picture

Title: extra line breaks in theme_cck_address » extra line breaks in Zen theme in class "block block-block first last odd"
Project: Address field for CCK » Project
Version: 5.x-2.1 » 7.x-2.x-dev
Component: Code » Projects
Category: feature » bug
Priority: Minor » Normal
Status: Closed (fixed) » Needs work
StatusFileSize
new46.42 KB

The addition of "
" to the Zen theme is wreaking havoc on my site's design. For some reason, it has inserted itself 11 times when the page is displayed, and I don't yet know how to override this "feature". So, now my header (where this block was/is to be displayed) now takes up most of my page, making the site look ridiculous.

I'm attaching a picture of the resulting code that shows up when the page is displayed.

Please help me figure out how to remove those line breaks.

Thanks!

dww’s picture

Project: Project » Zen
Version: 7.x-2.x-dev » 7.x-6.x-dev
Component: Projects » CSS/SASS/HTML markup

What does this have to do with the Project module?

echoz’s picture

Version: 7.x-6.x-dev » 7.x-5.x-dev
Category: bug » support
Status: Needs work » Closed (cannot reproduce)

This looks like code from some very poor wysiwyg setup, not produced by Zen.