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
Comment #1
Andrzej7 commentedI have the same problem, but (not a programmer) can't implement these changes ;-(
Comment #2
lliss commentedAnyone see a reason not to do this?
Comment #3
daggerhart commented@lliss, This is absolutely correct. This module can't possibly do left pullquotes the way it is written in 1.2.
Comment #4
nancydruWorked for me.
Comment #5
sclm commentedI've done this as a patch and found that I only needed a CSS change.
Comment #6
nancydruLooks good to me.
Comment #7
rjl commentedIt looks like the patch only addresses the CSS, but without the change to the JS, the pullquote-left class isn't included properly.
Comment #8
lliss commentedI've updated this for the 7.x-2.x version but we still need to address the JS.
Comment #9
lliss commentedOk, so the JS is fixed in the 7.x-2.x version but a backport is still needed for the 6.x version.
Comment #10
lliss commentedComment #11
lliss commentedFixed in D7. Closing since D6 has reached its end-of-life.
Comment #12
lliss commented