Closed (won't fix)
Project:
Mailchimp
Version:
6.x-2.0-rc5
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
8 Jun 2010 at 21:05 UTC
Updated:
11 Jun 2010 at 16:47 UTC
Simple patch adds a text input in the admin settings of each list. Then uses that setting as the submit button value on the subscription form. Defaults to "Sign Up!".
| Comment | File | Size | Author |
|---|---|---|---|
| mailchimp-submittext.patch | 1.87 KB | spatical |
Comments
Comment #1
levelos commentedThanks, I appreciate the patch, but I don't think it's a good idea to further clutter the already very busy configuration screen for this settings. It can already be changed either using translations or via form_alter(). I'm open to a better default, though.
Comment #2
spatical commentedYou are absolutely right. I got overly excited about making this mailchimp plugin perfect for everything I need on my website. I am going to update my forms using the form_alter method. That makes a lot more sense then adding to the clutter, and should stay more compatible with future mailchimp updates rather than modifying the mailchimp plugin.