In your .js file you use .replaceWith to add the span and classes. This methodology doesn't take into account the 'pullquote-left' class concept.

Instead you could remove the .replaceWith line and just use an .addClass to add the 'pullquote-quote' class which would leave the 'pullquote-left' class intact.

This would make the $text = $span.text(); line unnecessary as well.

      // the following line will no longer be needed and can be removed
      //$text = $span.text();      
      $span.clone()
        .prependTo($parent)
        // add this next line
        .addClass('pullquote-quote');
        // remove the following line
        //.replaceWith('<span class="pullquote-processed pullquote-quote">' + $text + '</span>');

There is one change needed in the CSS file as well...

This declaration:
.pullquote-quote .pullquote-left {...}

should be re-arranged (and no space between the classes)
.pullquote-left.pullquote-quote {...}

Comments

Andrzej7’s picture

I have the same problem, but (not a programmer) can't implement these changes ;-(

lliss’s picture

Anyone see a reason not to do this?

daggerhart’s picture

@lliss, This is absolutely correct. This module can't possibly do left pullquotes the way it is written in 1.2.

nancydru’s picture

Worked for me.

sclm’s picture

Status: Active » Needs review
StatusFileSize
new848 bytes

I've done this as a patch and found that I only needed a CSS change.

nancydru’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

rjl’s picture

Status: Reviewed & tested by the community » Needs review

It looks like the patch only addresses the CSS, but without the change to the JS, the pullquote-left class isn't included properly.

lliss’s picture

I've updated this for the 7.x-2.x version but we still need to address the JS.

lliss’s picture

Ok, so the JS is fixed in the 7.x-2.x version but a backport is still needed for the 6.x version.

lliss’s picture

Assigned: Unassigned » lliss
lliss’s picture

Issue summary: View changes

Fixed in D7. Closing since D6 has reached its end-of-life.

lliss’s picture

Status: Needs review » Closed (fixed)