The label of the submit button for mailchimp_lists_user_subscribe_form() is "Subscribe" which depending on the situation can be inaccurate, for example in the case of a single newsletter that one wants to unsubscribe: untick the name of the newsletter and click "Subscribe" is not intuite.

Solution: either rename the label to something more generic like "Save", or change the label with jQuery based on the state of the checkbox. Note however that there might be situation where some are checked and some others aren't, so "Subscribe" still does not make sense. Maybe "Save" or "Save subscriptions" is better.

Comments

antipex’s picture

antipex CreditAttribution: antipex commented
Status: Active » Fixed

Good idea - changed to "Save" in 740ed17.

BarwonHack’s picture

BarwonHack CreditAttribution: BarwonHack commented

add to your themes template.php file ...

function THEMENAME_form_alter(&$form, &$form_state, $form_id) {

if ($form_id == "mailchimp_lists_user_subscribe_form_LIST_MACHINE_NAME") {
$form['submit'] = array(
'#type' => 'submit',
'#value' => t('Sign Uppppxx'),
);
}
}

Status: Fixed » Closed (fixed)

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

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Status: Closed (fixed) » Active

I think this is fixed & in the module now.

@smilne23 that's just not a good enough answer. Yes, absolutely you can hack it in. However, the standard in D7 is to fix accessibility issues like this and not make everyone go through and manually add the label.

EDIT: Frig.. Didn't read the start of that message well enough.. Arg, multitasking.. Sorry all.

BarwonHack’s picture

BarwonHack CreditAttribution: BarwonHack commented
Status: Closed (fixed) » Active

Blinks ... really?!

levelos’s picture

levelos CreditAttribution: levelos commented
Status: Active » Closed (fixed)

I'm closing this. We will not be adding a configuration setting for the text on the submit button at this time. It can be changed through the Drupal core localization system, using form_alter, writing your own module, theme override, etc. Please do not reopen.

atiba’s picture

atiba CreditAttribution: atiba commented
Status: Active » Closed (fixed)

Thanks smilne23!
Post #2 works just perfectly.

I just want to thank you for that. :)