That's what I was just about saying, in addition try to add the HTML entities to the regular expression as well: $string = preg_replace('/(?:& #34;|")(.*?)(?:& #34;|")/', '<font color="color">"\\1"</font>', $string); Also if you don't to get caught by the HTML validator you better surround the color name with double quotes ;) HTH, Nitsan On Sat, Apr 25, 2009 at 9:42 PM, Daniel Brown <danbrown@xxxxxxx> wrote: > On Sat, Apr 25, 2009 at 13:50, Marc Steinert <lists@xxxxxxxxxx> wrote: > > > > $string = preg_replace('/"(.*?)"/', '<font color=color>"\\1"</font>', > > $string); > > Close, but I'd also recommend dropping in a 'Us' modifier so that > it is `U`ngreedy and `s`pans lines. > > <?php > $regexp = '/"(.*?)"/Us'; > ?> > > -- > </Daniel P. Brown> > daniel.brown@xxxxxxxxxxxx || danbrown@xxxxxxx > http://www.parasane.net/ || http://www.pilotpig.net/ > 50% Off All Shared Hosting Plans at PilotPig: Use Coupon DOW10000 > > -- > PHP General Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > >