It's not possible to set the padding to zero. If you set the padding to zero in PHP, the following line in graphael.jquery.js read it as FALSE and uses the default instead:

params.padding = params.padding || 10;

Patch to follow.

CommentFileSizeAuthor
#2 921002_1.patch689 bytesWill White

Comments

mikl’s picture

mikl
Location Møn
commented
Status: Active » Needs work

Ah, the joys of dynamic typing. I’ll get it fixed for the next beta :)

Will White’s picture

Will White commented
Status: Needs work » Needs review
StatusFileSize
new 921002_1.patch689 bytes

The solution to this is to expect the padding param as a string. If the string is truly NULL, it uses the default. But if the string has a value (even "0"), it will use that and parse a float from it.

googletorp’s picture

googletorp commented
Status: Needs review » Fixed

I've committed the patch into dev, thanx.

Status: Fixed » Closed (fixed)

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