Skip to content

Update SiteConfiguration.php #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jcsparks merged 3 commits into from
Nov 12, 2024
Merged

Update SiteConfiguration.php #71

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
dfa9447
Update SiteConfiguration.php
jnicholCU Nov 10, 2024
eaf4164
Update SiteConfiguration.php
jnicholCU Nov 10, 2024
3c936ca
Update SiteConfiguration.php
jnicholCU Nov 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
67 changes: 67 additions & 0 deletions src/SiteConfiguration.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -262,6 +262,66 @@ public function buildThemeSettingsForm(array &$form, FormStateInterface &$form_s
'#description' => $this->t('Headings are bold by default, but can also be set to the same font weight as normal text.'),
];

$form['content']['ucb_above_content_region_color'] = [
'#type' => 'select',
'#title' => $this->t('Above Content background color'),
'#default_value' => theme_get_setting('ucb_above_content_region_color', $themeName),
'#options' => [
'white' => $this->t('None'),
'light-gray' => $this->t('Light Gray'),
'dark-gray' => $this->t('Dark Gray'),
'black' => $this->t('Black'),
'gold' => $this->t('Gold'),
'tan' => $this->t('Tan'),
'light-blue' => $this->t('Light Blue'),
'medium-blue' => $this->t('Medium Blue'),
'dark-blue' => $this->t('Dark Blue'),
'light-green' => $this->t('Light Green'),
'brick' => $this->t('Brick'),
],
'#description' => $this->t('The global background color for this Block Layout region.'),
];

$form['content']['ucb_after_content_one_region_color'] = [
'#type' => 'select',
'#title' => $this->t('After Content 1 background color'),
'#default_value' => theme_get_setting('ucb_after_content_one_region_color', $themeName),
'#options' => [
'white' => $this->t('None'),
'light-gray' => $this->t('Light Gray'),
'dark-gray' => $this->t('Dark Gray'),
'black' => $this->t('Black'),
'gold' => $this->t('Gold'),
'tan' => $this->t('Tan'),
'light-blue' => $this->t('Light Blue'),
'medium-blue' => $this->t('Medium Blue'),
'dark-blue' => $this->t('Dark Blue'),
'light-green' => $this->t('Light Green'),
'brick' => $this->t('Brick'),
],
'#description' => $this->t('The global background color for this Block Layout region.'),
];

$form['content']['ucb_after_content_two_region_color'] = [
'#type' => 'select',
'#title' => $this->t('After Content 2 background color'),
'#default_value' => theme_get_setting('ucb_after_content_two_region_color', $themeName),
'#options' => [
'white' => $this->t('None'),
'light-gray' => $this->t('Light Gray'),
'dark-gray' => $this->t('Dark Gray'),
'black' => $this->t('Black'),
'gold' => $this->t('Gold'),
'tan' => $this->t('Tan'),
'light-blue' => $this->t('Light Blue'),
'medium-blue' => $this->t('Medium Blue'),
'dark-blue' => $this->t('Dark Blue'),
'light-green' => $this->t('Light Green'),
'brick' => $this->t('Brick'),
],
'#description' => $this->t('The global background color for this Block Layout region.'),
];

if ($this->user->hasPermission('edit ucb site advanced')) {
$form['advanced'] = [
'#type' => 'details',
Expand All @@ -282,9 +342,16 @@ public function buildThemeSettingsForm(array &$form, FormStateInterface &$form_s
$this->t('None'),
$this->t('Footer'),
$this->t('Header'),
$this->t('Homepage Layout'),
],
'#description' => $this->t('Check this box if you would like to display the "Be Boulder" slogan in the header.'),
];
$form['advanced']['ucb_homepage_header'] = [
Copy link
Member

@jcsparks jcsparks Nov 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we get rid of the double-negative here, parsing this is confusing. Also it's unchecked but not displaying by default. We want it unchecked by default but displaying, currently this would wipe out the site title for every site even though it's not checked.

'#type' => 'checkbox',
'#title' => $this->t('Hide the normal header bar. For the CU Homepage only.'),
'#default_value' => theme_get_setting('ucb_homepage_header', $themeName),
'#description' => $this->t('Check this box if you would like to hide the header. This is meant for the CU Homepage layout only.'),
];
$form['advanced']['ucb_secondary_menu_default_links'] = [
'#type' => 'checkbox',
'#title' => $this->t('Display the standard Boulder secondary menu in the header navigation region.'),
Expand Down