The latest version has a missing "span" and is still missing (in my opinion) a
to make the line breaks look right... here's a diff on amazon.module that I think fixes it. Thanks!!!

CommentFileSizeAuthor
#3 amazontools_fix_linebreak.patch831 byteswayland76
diff_4.txt286 bytesbdillahu

Comments

wayland76’s picture

wayland76 commented
Project: Amazon Filter » Amazon associate tools

It would help if this bug were assigned to the correct project

wayland76’s picture

wayland76 commented
Version: 4.7.x-1.x-dev » 5.x-1.5
wayland76’s picture

wayland76 commented
Status: Active » Needs review
StatusFileSize
new amazontools_fix_linebreak.patch831 bytes

Ok, after looking at this, I came to the following conclusions:
1. The patch is in non-unified format (which is bad)
2. The "span" problem has already been fixed in head
3. The line break problem is now amazingly simple. Patch attached, but it's barely necessary.

MGParisi’s picture

MGParisi commented

I am consolidating the patches for this and will either submit it and get the original author to post it as a dev version, or I will fork it (not what I want).

Im questioning, if you want a
added then why is it a span and not a div?

MGParisi’s picture

MGParisi commented

I see, fixed...

MGParisi’s picture

MGParisi commented

The patch is applied to http://drupal.org/node/289780#comment-945539

MGParisi’s picture

MGParisi commented

fixed, should be in dev

MGParisi’s picture

MGParisi commented
Status: Needs review » Fixed

oops

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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