Change the settings form to use a table to list the ads.

CommentFileSizeAuthor
#3 broadstreet_ads-n2672474-3.patch10.25 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Related issues: +#2202441: Fix the table on the admin page
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
broadstreet_ads-n2672474-3.patch10.25 KB

This handles it.

Status: Needs review » Needs work

The last submitted patch, 3: broadstreet_ads-n2672474-3.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2225139: META: Plan for Broadstreet Ads 7.x-1.0 release

  • DamienMcKenna committed 223d531 on 7.x-1.x 
    Issue #2672474 by DamienMcKenna, joshua.boltz: Restructured the settings...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Fixed

Committed.

Status: Fixed » Closed (fixed)

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