Similar to the "fade" effect added by https://drupal.org/node/1234680, we should also have a "slide" effect.

Patch forthcoming.

Comments

brettbirschbach’s picture

StatusFileSize
new688 bytes
brettbirschbach’s picture

Status: Active » Needs review
brettbirschbach’s picture

Note that I purposely removed the t() around 'fade' since it is an untranslated machine name.

m4olivei’s picture

Assigned: brettbirschbach » m4olivei
Status: Needs review » Patch (to be ported)

Assigning to myself for later inclusion.

m4olivei’s picture

StatusFileSize
new937 bytes

Re-roll against the latest dev.

  • m4olivei committed 04c3555 on 7.x-1.x
    Issue #2129567 by HitmanInWis, m4olivei: Add a jQuery slide effect
    
m4olivei’s picture

Status: Patch (to be ported) » Closed (fixed)

Committed to 7.x-1.x. Thanks!

brettbirschbach’s picture

Just out of curiousity...since I submitted the original patch, and it simply needed to be re-rolled against latest codebase but with no actual code changes, should I be credited as the author of the fix? Not 100% sure how attribution works, but I'd like for my company to get visibility for the fixes I made/helped make.

m4olivei’s picture

Humm, looks like I messed up the commit message :(. Sorry about that. I'm not sure hot to go back and fix..

brettbirschbach’s picture

Ya, not 100% sure how to do it...appreciate if you are able to figure it out. Thanks!