Maximum font size vice Minimum font size - simple code correction
bwv - June 17, 2008 - 16:44
| Project: | Node Cloud |
| Version: | 5.x-1.0 |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
Terrific module, thank you!
A minor oversight to correct: in node cloud settings under site configuration, Minimum node size appears over both fields. The second field header should read "Maximum node size."
Corrected code is below, fyi.
function nodecloud_settings() {
$form['nodecloud_min'] = array(
'#type' => 'textfield',
'#title' => t('Minimum font size (in "EM" units)'),
'#default_value' => variable_get('nodecloud_min', NODECLOUD_MIN)
);
$form['nodecloud_max'] = array(
'#type' => 'textfield',
'#title' => t('Maximum font size (in "EM" units)'),
'#default_value' => variable_get('nodecloud_max', NODECLOUD_MAX)
);
return system_settings_form($form);
}
#1
Apologies, those are lines 27 through 39 of the nodecloud.module.
#2
This is fixed in the development version. Thanks anyway for reporting.
#3
Automatically closed -- issue fixed for two weeks with no activity.