INTERIM-138 Made wordmark alt text more flexible

isubit · Sep 20, 2017 · af1aafc · af1aafc
2 parents 61a3be6 + cf4f16a
commit af1aafc
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 2 deletions.
diff --git a/preprocess/preprocess-region.inc b/preprocess/preprocess-region.inc
@@ -18,6 +18,7 @@ function suitcase_interim_alpha_preprocess_region(&$vars) {
 
     // Get the level 1 url
     $vars['level_1_url'] = variable_get('suitcase_interim_config_level_1_url');
+    $vars['wordmark_alt_text'] = theme_get_setting('suitcase_interim_config_isu_nameplate_alt_text');
 
     $vars['suitcase_interim_config_level_2_url'] = variable_get('suitcase_interim_config_level_2_url');
 

diff --git a/suitcase_interim.info b/suitcase_interim.info
@@ -52,6 +52,7 @@ settings[toggle_secondary_menu] = '1'
 settings[default_logo] = '1'
 settings[logo_path] = 'sites/all/themes/suitcase_interim/images/isu.svg'
 settings[logo_upload] = ''
+settings[suitcase_interim_config_isu_nameplate_alt_text] = 'Iowa State University'
 settings[default_favicon] = '0'
 settings[favicon_path] = 'sites/all/themes/suitcase_interim/images/favicon.ico'
 settings[favicon_upload] = ''

diff --git a/templates/region--branding.tpl.php b/templates/region--branding.tpl.php
@@ -4,9 +4,9 @@
 
         <?php if ($show_isu_nameplate): ?>
             <?php if (theme_get_setting('default_logo')): ?>
-              <a id="isu_header_wordmark" href="<?php print $level_1_url; ?>" title="Iowa State University Homepage"><img src="<?php print $wordmark_image; ?>" alt="Iowa State University"></a>
+              <a id="isu_header_wordmark" href="<?php print $level_1_url; ?>" title="Iowa State University Homepage"><img src="<?php print $wordmark_image; ?>" alt="<?php print $wordmark_alt_text; ?>"></a>
             <?php else: ?>
-              <a id="isu_header_wordmark" href="<?php print $level_1_url; ?>" title="<?php print $site_name; ?>"><img src="<?php print $wordmark_image; ?>" alt="Iowa State University - <?php print $site_name; ?>"></a>
+              <a id="isu_header_wordmark" href="<?php print $level_1_url; ?>" title="<?php print $site_name; ?>"><img src="<?php print $wordmark_image; ?>" alt="<?php print $wordmark_alt_text; ?>"></a>
             <?php endif; ?>
         <?php endif; ?>
 

diff --git a/theme-settings.php b/theme-settings.php
@@ -85,6 +85,14 @@ function suitcase_interim_form_system_theme_settings_alter(&$form, &$form_state)
       '#weight' => 2,
     );
 
+    $form['suitcase_interim_config']['suitcase_interim_config_logo']['settings']['suitcase_interim_config_isu_nameplate_alt_text'] = array(
+      '#type' => 'textfield',
+      '#title' => t('Wordmark Alt Text'),
+      '#description' => t('Alt text for the site\'s wordmark. Defaults to Iowa State University.'),
+      '#default_value' => variable_get('suitcase_interim_config_isu_nameplate_alt_text', 'Iowa State University'),
+      '#weight' => 3,
+    );
+
     $form['suitcase_interim_config']['suitcase_interim_config_logo']['suitcase_interim_config_level_2_url'] = array(
       '#type' => 'hidden',
       '#attributes' => array(
@@ -174,6 +182,8 @@ function suitcase_interim_config_form_submit($form, &$form_state) {
 
   variable_set('suitcase_interim_config_isu_nameplate_display', $form_state['values']['suitcase_interim_config_isu_nameplate_display']);
 
+  variable_set('suitcase_interim_config_isu_nameplate_alt_text', $form_state['values']['suitcase_interim_config_isu_nameplate_alt_text']);
+
   variable_set('suitcase_interim_config_show_advanced_settings', $form_state['values']['suitcase_interim_config_show_advanced_settings']);
 
   // Decide which level the site name should be set to