Skip to content

Commit

Permalink
Remove useless and invalid pagination color overrides. (#4177)
Browse files Browse the repository at this point in the history
- The settings were not needed because the default is already $primary.
- The settings in bootstrap5 variables.scss were invalid as $brand-primary (a BS3 thing) isn't actually defined (but !default masked the mistake).
  • Loading branch information
EreMaijala authored Jan 3, 2025
1 parent 8fb19dc commit f5f9ec2
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 21 deletions.
6 changes: 0 additions & 6 deletions themes/bootstrap3/less/variables.less
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@
// Cookie consent dialog should have a z-index low enough to stay below dropdowns and modals
@zindex-cookie-consent: (@zindex-dropdown - 10);

@pagination-active-bg: @brand-primary;
@pagination-active-border: @brand-primary;

// Variables used in Sidebar Menu Lists
@myresearch-menu-ul-margin-bottom: 0;
@myresearch-menu-ul-padding-left: 0;
Expand All @@ -13,9 +10,6 @@
// Cookie consent dialog should have a z-index low enough to stay below dropdowns and modals
$zindex-cookie-consent: (@zindex-dropdown - 10) !default;
$pagination-active-bg: @brand-primary !default;
$pagination-active-border: @brand-primary !default;
$myresearch-menu-ul-margin-bottom: 0 !default;
$myresearch-menu-ul-padding-left: 0 !default;
<#SCSS */
6 changes: 0 additions & 6 deletions themes/bootstrap3/scss/variables.scss
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@
// Cookie consent dialog should have a z-index low enough to stay below dropdowns and modals
$zindex-cookie-consent: ($zindex-dropdown - 10);
$pagination-active-bg: $brand-primary;
$pagination-active-border: $brand-primary;
// Variables used in Sidebar Menu Lists
$myresearch-menu-ul-margin-bottom: 0;
$myresearch-menu-ul-padding-left: 0;
Expand All @@ -13,9 +10,6 @@ $myresearch-menu-ul-padding-left: 0;
// Cookie consent dialog should have a z-index low enough to stay below dropdowns and modals
$zindex-cookie-consent: ($zindex-dropdown - 10) !default;

$pagination-active-bg: $brand-primary !default;
$pagination-active-border: $brand-primary !default;

$myresearch-menu-ul-margin-bottom: 0 !default;
$myresearch-menu-ul-padding-left: 0 !default;
/* <#SCSS */
3 changes: 0 additions & 3 deletions themes/bootstrap5/scss/bootstrap-variable-overrides.scss
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,6 @@ $form-switch-bg-image-dark: none !default;
$navbar-dark-toggler-icon-bg: none !default;
$navbar-light-toggler-icon-bg: none !default;

$pagination-active-bg: $primary !default;
$pagination-active-border: $primary !default;

$link-hover-decoration: underline !default;
$icon-link-underline-offset: inherit !default;

Expand Down
6 changes: 0 additions & 6 deletions themes/bootstrap5/scss/variables.scss
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@
// Cookie consent dialog should have a z-index low enough to stay below dropdowns and modals
$zindex-cookie-consent: ($zindex-dropdown - 10);
$pagination-active-bg: $brand-primary;
$pagination-active-border: $brand-primary;
// Variables used in Sidebar Menu Lists
$myresearch-menu-ul-margin-bottom: 0;
$myresearch-menu-ul-padding-left: 0;
Expand All @@ -13,9 +10,6 @@ $myresearch-menu-ul-padding-left: 0;
// Cookie consent dialog should have a z-index low enough to stay below dropdowns and modals
$zindex-cookie-consent: ($zindex-dropdown - 10) !default;

$pagination-active-bg: $brand-primary !default;
$pagination-active-border: $brand-primary !default;

$myresearch-menu-ul-margin-bottom: 0 !default;
$myresearch-menu-ul-padding-left: 0 !default;
/* <#SCSS */

0 comments on commit f5f9ec2

Please sign in to comment.