The Twitter Post fields should use the Form API #states logic instead of custom JS to hide & show the fields.

This only affects the D7 branches as D6 didn't have the #states system.

CommentFileSizeAuthor
#4 twitter-n2546686-3-7.x-6.x.patch717 bytesdamienmckenna
#2 twitter-n2546686-2-7.x-5.x.patch1.91 KBdamienmckenna

Comments

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new twitter-n2546686-2-7.x-5.x.patch1.91 KB

For the 7.x-5.x branch.

  • DamienMcKenna committed 21796a7 on 
    Issue #2546686 by DamienMcKenna: Use FormAPI states logic to hide/show...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new twitter-n2546686-3-7.x-6.x.patch717 bytes

For the 7.x-6.x branch.

Status: Needs review » Needs work

The last submitted patch, 4: twitter-n2546686-3-7.x-6.x.patch, failed testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-5.x-dev » 7.x-6.x-dev

Status: Needs work » Needs review

DamienMcKenna queued 4: twitter-n2546686-3-7.x-6.x.patch for re-testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Git commit.

Status: Fixed » Closed (fixed)

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