Hi all,

I've spent a bit of time looking through the source code for
sfTextReplacementPlugin.  This plugin hasn't changed since last summer
and still needs a bit of work doing on it.  I'd like to offer my own
contribution if that would be useful.

The specific issues are:

1. There is a bug in which the plugin attempts to send the generated
PNG to the browser. Inspection of the HTTP traffic shows that the
content length is not correctly set and some inappropriate headers are
sent. It's not surprising therefore that browsers sometimes refuse to
render the image.  I don't have a solution to this yet, although a
workaround is to enable the cache-in-web-dir feature and send the
browser a redirect to the static (cached) image instead of attempting
to stream the PNG bytes to the browser.

2. I would like to extend the plugin with a feature that allows the
user to specify a fixed size for the PNG in order to fit it into the
general layout of the webpage.  I have written code to do this,
allowing configuration over whether images will shrink if the text is
too big (by reducing the font size accordingly) and, separately,
configuration over whether the image will expand to fill a specified
size or be padded with the background colour.

3. I would also like it to be possible to create text-replacement
images manually and for the plugin to find these and, when present,
server them instead of generating images on the fly.  I haven't done
this yet although it should be quite easy.

What's the best way to offer my work?  Can I commit directly to SVN or
should I zip up everything and put it on a Trac issue?

Rick
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"symfony developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/symfony-devs?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to