Page 4 of 5 FirstFirst ... 2345 LastLast
Results 31 to 40 of 41

Thread: What WordPress plugin would you like to see made?

  1. #31
    chipbennett's Avatar
    chipbennett is offline WordPress Legend
    Join Date
    Feb 2009
    Location
    St. Louis, MO
    Posts
    1,993

    Default

    Quote Originally Posted by Otto View Post
    That gravatar plugin is now in the Extend repository: http://wordpress.org/extend/plugins/gravatar-box/

    Okay, so, that's one plugin solved. Next? ;)
    Otto,

    I've installed the plugin, but it doesn't seem to be working. Nothing is displayed, at all.

    Interestingly, it appears that the DIV is getting assigned class="gravbox" instead of id="gravbox".

    It looks like gravbox.php is outputting DIV class="gravbox", while gravbox.js is looking explicitly for #gravatar.

    I changed gravbox.php to output DIV id="gravbox", and I'm still not getting anything. I get the DIV output, but no background image (for which I added an appropriate CSS declaration), and nothing happens when inputting a valid/invalid email address.

    I'm sure it's probably something on my end; I just don't know what...
    WP TurnKey - Turn-Key WordPress installation and maintenance services
    WordPress user since 2005 | @chip_bennett | chipbennett.net | cbnet Plugins

  2. #32
    Otto's Avatar
    Otto is offline On The Rocks
    Join Date
    Apr 2009
    Location
    Memphis, TN
    Posts
    862

    Default

    I think I checked in an older version, actually, because I corrected two bugs early on that somehow made it back in there.

    The class/id thing was one, the other was that "init" should have been "wp_print_scripts".

    Anyway, fixed it. Give it 15 and the repo will update the download file. For instant satisfaction, updated files are here: http://svn.wp-plugins.org/gravatar-box/trunk/
    Last edited by Otto; 08-26-2009 at 03:48 PM.

  3. #33
    chipbennett's Avatar
    chipbennett is offline WordPress Legend
    Join Date
    Feb 2009
    Location
    St. Louis, MO
    Posts
    1,993

    Default

    Quote Originally Posted by Otto View Post
    The class/id thing was an early bug that I corrected later. Guess that somehow got uncorrected in the check in.

    The rest of the problem was caused by another problem I fixed too, but somehow didn't make it into the repo. "init" should have been wp_print_scripts.

    Anyway, fixed it. Give it 15 and the repo will update the download file.
    Awesome!

    I thought I had managed to screw something up. :)
    WP TurnKey - Turn-Key WordPress installation and maintenance services
    WordPress user since 2005 | @chip_bennett | chipbennett.net | cbnet Plugins

  4. #34
    chipbennett's Avatar
    chipbennett is offline WordPress Legend
    Join Date
    Feb 2009
    Location
    St. Louis, MO
    Posts
    1,993

    Default

    Quote Originally Posted by Otto View Post
    I think I checked in an older version, actually, because I corrected two bugs early on that somehow made it back in there.

    The class/id thing was one, the other was that "init" should have been "wp_print_scripts".

    Anyway, fixed it. Give it 15 and the repo will update the download file. For instant satisfaction, updated files are here: http://svn.wp-plugins.org/gravatar-box/trunk/
    Updated, and everything is working perfectly now!

    Slick!

    Feature request:

    How difficult would it be to link the plugin into the blog's settings for Wavatar/MonsterID, so that if no gravatar is returned, the IMG is replaced with whatever the user has set in the comment/gravatar settings?
    Last edited by chipbennett; 08-26-2009 at 04:57 PM.
    WP TurnKey - Turn-Key WordPress installation and maintenance services
    WordPress user since 2005 | @chip_bennett | chipbennett.net | cbnet Plugins

  5. #35
    Otto's Avatar
    Otto is offline On The Rocks
    Join Date
    Apr 2009
    Location
    Memphis, TN
    Posts
    862

    Default

    Well, if you did that, then the sign up link wouldn't display any more, since there'd be no way to detect that the user has no avatar.

    However, if you edit the gravbox.js file, and change the two instances of d=404 to d=whatever, where whatever is one of:
    'mystery' => __('Mystery Man'),
    'blank' => __('Blank'),
    'gravatar_default' => __('Gravatar Logo'),
    'identicon' => __('Identicon (Generated)'),
    'wavatar' => __('Wavatar (Generated)'),
    'monsterid' => __('MonsterID (Generated)')

    Then you'll get that icon instead.

    Actually, it'd be possible for you to force the background CSS to the new image via javascript and retain the sign up link. Bit more tricky code though.

  6. #36
    chipbennett's Avatar
    chipbennett is offline WordPress Legend
    Join Date
    Feb 2009
    Location
    St. Louis, MO
    Posts
    1,993

    Default

    Quote Originally Posted by Otto View Post
    Well, if you did that, then the sign up link wouldn't display any more, since there'd be no way to detect that the user has no avatar.

    However, if you edit the gravbox.js file, and change the two instances of d=404 to d=whatever, where whatever is one of:
    'mystery' => __('Mystery Man'),
    'blank' => __('Blank'),
    'gravatar_default' => __('Gravatar Logo'),
    'identicon' => __('Identicon (Generated)'),
    'wavatar' => __('Wavatar (Generated)'),
    'monsterid' => __('MonsterID (Generated)')

    Then you'll get that icon instead.

    Actually, it'd be possible for you to force the background CSS to the new image via javascript and retain the sign up link. Bit more tricky code though.
    That's more what I had in mind: still display the P with the gravatar signup link, but change the DIV background accordingly.
    WP TurnKey - Turn-Key WordPress installation and maintenance services
    WordPress user since 2005 | @chip_bennett | chipbennett.net | cbnet Plugins

  7. #37
    Otto's Avatar
    Otto is offline On The Rocks
    Join Date
    Apr 2009
    Location
    Memphis, TN
    Posts
    862

    Default

    Adding something along these lines in the error section of gravbox.js would do the trick:

    Code:
    $('#gravbox').css('background-image', 'url('+gravatar(email)+'?s='+size+'&d=identicon)');
    That's off the top of my head, mind you. May need adjustment.

  8. #38
    chipbennett's Avatar
    chipbennett is offline WordPress Legend
    Join Date
    Feb 2009
    Location
    St. Louis, MO
    Posts
    1,993

    Default

    Quote Originally Posted by Otto View Post
    Adding something along these lines in the error section of gravbox.js would do the trick:

    Code:
    $('#gravbox').css('background-image', 'url('+gravatar(email)+'?s='+size+'&d=identicon)');
    That's off the top of my head, mind you. May need adjustment.
    Is this how the core comments works? If so, wow; that's succinct!

    (Or, does this just display the identicon (as opposed to wavatar/monsterid)?

    Would this be something that would benefit from an options/settings admin page? I've never tried writing one, but I'll offer to do so - even if just to learn how to do it.
    WP TurnKey - Turn-Key WordPress installation and maintenance services
    WordPress user since 2005 | @chip_bennett | chipbennett.net | cbnet Plugins

  9. #39
    Otto's Avatar
    Otto is offline On The Rocks
    Join Date
    Apr 2009
    Location
    Memphis, TN
    Posts
    862

    Default

    Quote Originally Posted by chipbennett View Post
    Is this how the core comments works? If so, wow; that's succinct!

    (Or, does this just display the identicon (as opposed to wavatar/monsterid)?

    Would this be something that would benefit from an options/settings admin page? I've never tried writing one, but I'll offer to do so - even if just to learn how to do it.
    This just displays the identicon. Change the word "identicon" to something else to get that something else.

    The option to choose same as in the core would require having that javascript be generated dynamically instead of putting it in a separate file. Could be done, but it's a bit overkill.

  10. #40
    chipbennett's Avatar
    chipbennett is offline WordPress Legend
    Join Date
    Feb 2009
    Location
    St. Louis, MO
    Posts
    1,993

    Default

    Quote Originally Posted by Otto View Post
    This just displays the identicon. Change the word "identicon" to something else to get that something else.

    The option to choose same as in the core would require having that javascript be generated dynamically instead of putting it in a separate file. Could be done, but it's a bit overkill.
    Ah, I didn't realize that it would be so diffucult.
    WP TurnKey - Turn-Key WordPress installation and maintenance services
    WordPress user since 2005 | @chip_bennett | chipbennett.net | cbnet Plugins

Page 4 of 5 FirstFirst ... 2345 LastLast

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •