I propose that the "Image size" option in the Gravatar settings is unnecessary. We already have the 'user_picture_dimensions' variable which we should use instead of duplicating an option. The following code
would return a $size parameter, no larger than 512px (the maximum allowed by the gravatar service):
$size = min(explode('x', variable_get('user_picture_dimensions', '85x85')), 512);
Comments
Comment #1
Narno commentedI vote for: settings will be more simple and logic.
Comment #2
dave reidFor now I have added a default option for the size setting to use the existing user picture dimensions, looking at options for removal.
Comment #3
dave reidHave removed the gravatar_size variable from 6.x-1.x. Will be a change in the next release.