From c9819c74592a68caeedddd0548ff3cf77845e97a Mon Sep 17 00:00:00 2001
From: Liam Morland <liam@openplus.ca>
Date: Fri, 2 May 2025 09:24:13 -0400
Subject: [PATCH 1/4] Issue #3522436: Make automated coding standards fixes

---
 docs/index.md                                       |  2 --
 .../facets_exposed_filters.module                   |  8 +++++---
 .../src/Plugin/views/filter/FacetsFilter.php        |  5 ++---
 .../tests/src/Functional/ExposedFiltersTest.php     |  6 +++---
 .../facets_summary/processor/ShowCountProcessor.php |  2 +-
 .../Plugin/views/FacetsSummaryViewsPluginTrait.php  |  2 +-
 src/Entity/Facet.php                                |  6 +++++-
 .../facets/processor/CombineFacetProcessor.php      |  3 +++
 .../facets/processor/DependentFacetProcessor.php    |  3 +++
 .../processor/HideInactiveSiblingsProcessor.php     |  3 +++
 .../ShowOnlyDeepestLevelItemsProcessor.php          |  5 ++++-
 .../facets/processor/ShowSiblingsProcessor.php      |  3 +++
 src/Plugin/facets/query_type/SearchApiDate.php      |  4 ++++
 src/Plugin/facets/query_type/SearchApiString.php    |  2 +-
 src/QueryType/QueryTypeRangeBase.php                |  5 ++++-
 tests/src/FunctionalJavascript/JsBase.php           | 13 +++++++++----
 16 files changed, 51 insertions(+), 21 deletions(-)

diff --git a/docs/index.md b/docs/index.md
index 19a6bc5d..ed2a41fc 100644
--- a/docs/index.md
+++ b/docs/index.md
@@ -1,5 +1,3 @@
 # Introduction
 
 The Facets module allows site builders to easily create and manage faceted search interfaces.
-
-
diff --git a/modules/facets_exposed_filters/facets_exposed_filters.module b/modules/facets_exposed_filters/facets_exposed_filters.module
index 2012b8cd..7ff91c14 100644
--- a/modules/facets_exposed_filters/facets_exposed_filters.module
+++ b/modules/facets_exposed_filters/facets_exposed_filters.module
@@ -4,6 +4,7 @@
  * @file
  */
 
+use Drupal\views\Plugin\Block\ViewsExposedFilterBlock;
 use Drupal\Core\Block\BlockPluginInterface;
 use Drupal\search_api\Entity\Index;
 use Drupal\search_api\Plugin\views\query\SearchApiQuery;
@@ -57,7 +58,8 @@ function facets_exposed_filters_search_api_query_alter(QueryInterface $query) {
         'operator' => $filter->options["facet"]["query_operator"],
         'min_count' => $filter->options["facet"]["min_count"],
         'missing' => FALSE,
-        'query_type' => 'search_api_string', // TODO: investigate if this property (query_type) is used.
+      // @todo investigate if this property (query_type) is used.
+        'query_type' => 'search_api_string',
       ];
     }
   }
@@ -170,7 +172,7 @@ function facets_exposed_filters_get_processed_facet($view_id, $display_id, $filt
 function facets_exposed_filters_block_build_alter(array &$build, BlockPluginInterface $block) {
   // Exposed filter blocks can be placed on any page. We need to ensure that the view is executed when the block is
   // rendered and has Facet filters.
-  if($block instanceof \Drupal\views\Plugin\Block\ViewsExposedFilterBlock) {
+  if ($block instanceof ViewsExposedFilterBlock) {
     $view = $block->getViewExecutable();
     $view->initHandlers();
     foreach ($view->filter as $filter) {
@@ -178,7 +180,7 @@ function facets_exposed_filters_block_build_alter(array &$build, BlockPluginInte
         $filter_id = $filter->options["id"];
         $display = $view->current_display;
         $processed_facet = facets_exposed_filters_get_processed_facet($view->id(), $display, $filter_id);
-        if(!$processed_facet) {
+        if (!$processed_facet) {
           // The facet has not been processed.
           $view->execute($display);
         }
diff --git a/modules/facets_exposed_filters/src/Plugin/views/filter/FacetsFilter.php b/modules/facets_exposed_filters/src/Plugin/views/filter/FacetsFilter.php
index 80f5e9ed..22caf107 100644
--- a/modules/facets_exposed_filters/src/Plugin/views/filter/FacetsFilter.php
+++ b/modules/facets_exposed_filters/src/Plugin/views/filter/FacetsFilter.php
@@ -11,7 +11,6 @@ use Drupal\facets\FacetInterface;
 use Drupal\facets\Hierarchy\HierarchyPluginBase;
 use Drupal\facets\Processor\ProcessorInterface;
 use Drupal\facets\Processor\SortProcessorInterface;
-use Drupal\facets\Result\Result;
 use Drupal\views\Plugin\views\filter\FilterPluginBase;
 
 /**
@@ -135,7 +134,7 @@ class FacetsFilter extends FilterPluginBase {
       $query_type_plugin->build();
 
       // When no results are available, we do not need to process the facet or render the form item.
-      if(!$facet->getResults()) {
+      if (!$facet->getResults()) {
         return;
       }
 
@@ -549,7 +548,7 @@ class FacetsFilter extends FilterPluginBase {
    */
   private function getActiveFacetValues() {
     // Reset button in ajax request. We probably want a better way to detect if this was clicked.
-    if(isset($_GET["reset"])) {
+    if (isset($_GET["reset"])) {
       return [];
     }
     $exposed = $this->view->getExposedInput();
diff --git a/modules/facets_exposed_filters/tests/src/Functional/ExposedFiltersTest.php b/modules/facets_exposed_filters/tests/src/Functional/ExposedFiltersTest.php
index 902b554a..a4f299fb 100644
--- a/modules/facets_exposed_filters/tests/src/Functional/ExposedFiltersTest.php
+++ b/modules/facets_exposed_filters/tests/src/Functional/ExposedFiltersTest.php
@@ -48,18 +48,18 @@ class ExposedFiltersTest extends FacetsTestBase {
     $this->assertSession()->pageTextContains('strawberry');
 
     // Test filtered page.
-    $this->drupalGet('test-facets-exposed-filters',['query' => ['keywords[]' => 'apple']] );
+    $this->drupalGet('test-facets-exposed-filters', ['query' => ['keywords[]' => 'apple']]);
     $this->assertSession()->pageTextContains('Keywords');
     $this->assertSession()->pageTextNotContains('entity:entity_test_mulrev_changed/3:en');
     $this->assertSession()->pageTextContains('strawberry');
 
     // Test if facet in keywords disappears when non-matching category is selected.
-    $this->drupalGet('test-facets-exposed-filters',['query' => ['category[]' => 'item_category']] );
+    $this->drupalGet('test-facets-exposed-filters', ['query' => ['category[]' => 'item_category']]);
     $this->assertSession()->pageTextContains('Keywords');
     $this->assertSession()->pageTextNotContains('strawberry');
 
     // Test if facet in keywords stays when matching category is selected.
-    $this->drupalGet('test-facets-exposed-filters',['query' => ['category[]' => 'article_category']] );
+    $this->drupalGet('test-facets-exposed-filters', ['query' => ['category[]' => 'article_category']]);
     $this->assertSession()->pageTextContains('Keywords');
     $this->assertSession()->pageTextContains('strawberry');
   }
diff --git a/modules/facets_summary/src/Plugin/facets_summary/processor/ShowCountProcessor.php b/modules/facets_summary/src/Plugin/facets_summary/processor/ShowCountProcessor.php
index 79969eb3..5f2f1697 100644
--- a/modules/facets_summary/src/Plugin/facets_summary/processor/ShowCountProcessor.php
+++ b/modules/facets_summary/src/Plugin/facets_summary/processor/ShowCountProcessor.php
@@ -31,7 +31,7 @@ class ShowCountProcessor extends ProcessorPluginBase implements BuildProcessorIn
     $count = $facets_summary->getFacetSource()->getCount();
     $build_count = [
       '#theme' => 'facets_summary_count',
-      '#count' => $count === NULL ? 0 : $count,
+      '#count' => $count ?? 0,
     ];
     array_unshift($build['#items'], $build_count);
     return $build;
diff --git a/modules/facets_summary/src/Plugin/views/FacetsSummaryViewsPluginTrait.php b/modules/facets_summary/src/Plugin/views/FacetsSummaryViewsPluginTrait.php
index 6712e513..3004f17a 100644
--- a/modules/facets_summary/src/Plugin/views/FacetsSummaryViewsPluginTrait.php
+++ b/modules/facets_summary/src/Plugin/views/FacetsSummaryViewsPluginTrait.php
@@ -37,7 +37,7 @@ trait FacetsSummaryViewsPluginTrait {
       '#options' => $options,
       '#type' => 'radios',
       '#required' => TRUE,
-      '#default_value' => isset($this->options['facet_summary']) ? $this->options['facet_summary'] : [],
+      '#default_value' => $this->options['facet_summary'] ?? [],
     ];
 
     $form['label_display'] = [
diff --git a/src/Entity/Facet.php b/src/Entity/Facet.php
index f4cf01bb..72f22da3 100644
--- a/src/Entity/Facet.php
+++ b/src/Entity/Facet.php
@@ -498,7 +498,7 @@ class Facet extends ConfigEntityBase implements FacetInterface {
 
     // Allow Facets without widgets (e.g. for facets exposed filters, where views handles the widget part).
     $widgetQueryType = NULL;
-    if($this->widget != "<nowidget>") {
+    if ($this->widget != "<nowidget>") {
       // Get the widget configured for this facet.
       /** @var \Drupal\facets\Widget\WidgetPluginInterface $widget */
       $widget = $this->getWidgetInstance();
@@ -717,6 +717,7 @@ class Facet extends ConfigEntityBase implements FacetInterface {
   public function getName() {
     return $this->name;
   }
+
   /**
    * {@inheritdoc}
    */
@@ -1219,6 +1220,9 @@ class Facet extends ConfigEntityBase implements FacetInterface {
     return $this->cacheMaxAge;
   }
 
+  /**
+   *
+   */
   protected function calculateCacheDependencies(): void {
     if (!$this->cache_dependencies_calculated) {
       if ($facet_source = $this->getFacetSource()) {
diff --git a/src/Plugin/facets/processor/CombineFacetProcessor.php b/src/Plugin/facets/processor/CombineFacetProcessor.php
index e1c223a7..7290bc27 100644
--- a/src/Plugin/facets/processor/CombineFacetProcessor.php
+++ b/src/Plugin/facets/processor/CombineFacetProcessor.php
@@ -171,6 +171,9 @@ class CombineFacetProcessor extends ProcessorPluginBase implements BuildProcesso
     return $results;
   }
 
+  /**
+   *
+   */
   public function supportsFacet(FacetInterface $facet) {
     // Only support facets as entities, not e.g. facets_exposed_filters.
     return $facet->getFacetType() == 'facet_entity';
diff --git a/src/Plugin/facets/processor/DependentFacetProcessor.php b/src/Plugin/facets/processor/DependentFacetProcessor.php
index 92a9028d..eb067746 100644
--- a/src/Plugin/facets/processor/DependentFacetProcessor.php
+++ b/src/Plugin/facets/processor/DependentFacetProcessor.php
@@ -222,6 +222,9 @@ class DependentFacetProcessor extends ProcessorPluginBase implements BuildProces
     return $return;
   }
 
+  /**
+   *
+   */
   public function supportsFacet(FacetInterface $facet) {
     // Only support facets as entities, not e.g. facets_exposed_filters.
     return $facet->getFacetType() == 'facet_entity';
diff --git a/src/Plugin/facets/processor/HideInactiveSiblingsProcessor.php b/src/Plugin/facets/processor/HideInactiveSiblingsProcessor.php
index 6bc6ac0f..f203067f 100644
--- a/src/Plugin/facets/processor/HideInactiveSiblingsProcessor.php
+++ b/src/Plugin/facets/processor/HideInactiveSiblingsProcessor.php
@@ -62,6 +62,9 @@ class HideInactiveSiblingsProcessor extends ProcessorPluginBase implements Build
     return $results;
   }
 
+  /**
+   *
+   */
   public function supportsFacet(FacetInterface $facet) {
     return $facet->getFacetType() == 'facet_entity';
   }
diff --git a/src/Plugin/facets/processor/ShowOnlyDeepestLevelItemsProcessor.php b/src/Plugin/facets/processor/ShowOnlyDeepestLevelItemsProcessor.php
index 99694280..d7ad1d64 100644
--- a/src/Plugin/facets/processor/ShowOnlyDeepestLevelItemsProcessor.php
+++ b/src/Plugin/facets/processor/ShowOnlyDeepestLevelItemsProcessor.php
@@ -36,8 +36,11 @@ class ShowOnlyDeepestLevelItemsProcessor extends ProcessorPluginBase implements
     return $results;
   }
 
+  /**
+   *
+   */
   public function supportsFacet(FacetInterface $facet) {
-    // TODO: Support "facets_exposed_filter".
+    // @todo Support "facets_exposed_filter".
     return $facet->getFacetType() == 'facet_entity';
   }
 
diff --git a/src/Plugin/facets/processor/ShowSiblingsProcessor.php b/src/Plugin/facets/processor/ShowSiblingsProcessor.php
index 8c34d46c..45cb5b1f 100644
--- a/src/Plugin/facets/processor/ShowSiblingsProcessor.php
+++ b/src/Plugin/facets/processor/ShowSiblingsProcessor.php
@@ -68,6 +68,9 @@ class ShowSiblingsProcessor extends ProcessorPluginBase implements BuildProcesso
     return $build;
   }
 
+  /**
+   *
+   */
   public function supportsFacet(FacetInterface $facet) {
     return $facet->getFacetType() == 'facet_entity';
   }
diff --git a/src/Plugin/facets/query_type/SearchApiDate.php b/src/Plugin/facets/query_type/SearchApiDate.php
index 54283220..52f79f26 100644
--- a/src/Plugin/facets/query_type/SearchApiDate.php
+++ b/src/Plugin/facets/query_type/SearchApiDate.php
@@ -267,6 +267,10 @@ class SearchApiDate extends QueryTypeRangeBase {
       return $this->calculateResultFilterAbsolute($value);
     }
   }
+
+  /**
+   *
+   */
   public function getDisplayValue($raw_value) {
     $dateTime = new DrupalDateTime();
     switch ($this->getGranularity()) {
diff --git a/src/Plugin/facets/query_type/SearchApiString.php b/src/Plugin/facets/query_type/SearchApiString.php
index 63faf51e..b328ef8a 100644
--- a/src/Plugin/facets/query_type/SearchApiString.php
+++ b/src/Plugin/facets/query_type/SearchApiString.php
@@ -82,7 +82,7 @@ class SearchApiString extends QueryTypePluginBase {
           if ($result_filter[strlen($result_filter) - 1] === '"') {
             $result_filter = substr($result_filter, 0, -1);
           }
-          if (($key = array_search($result_filter, $unprocessed_active_items)) !== false) {
+          if (($key = array_search($result_filter, $unprocessed_active_items)) !== FALSE) {
             unset($unprocessed_active_items[$key]);
           }
           $count = $result['count'];
diff --git a/src/QueryType/QueryTypeRangeBase.php b/src/QueryType/QueryTypeRangeBase.php
index 89684874..de813c70 100644
--- a/src/QueryType/QueryTypeRangeBase.php
+++ b/src/QueryType/QueryTypeRangeBase.php
@@ -91,7 +91,7 @@ abstract class QueryTypeRangeBase extends QueryTypePluginBase {
           else {
             $facet_results[$result_filter['raw']] = new Result($this->facet, $result_filter['raw'], $result_filter['display'], $count);
           }
-          if (($key = array_search($result_filter['raw'], $unprocessed_active_items)) !== false) {
+          if (($key = array_search($result_filter['raw'], $unprocessed_active_items)) !== FALSE) {
             unset($unprocessed_active_items[$key]);
           }
         }
@@ -111,6 +111,9 @@ abstract class QueryTypeRangeBase extends QueryTypePluginBase {
     return $this->facet;
   }
 
+  /**
+   *
+   */
   public function getDisplayValue($raw_value) {
     return $raw_value;
   }
diff --git a/tests/src/FunctionalJavascript/JsBase.php b/tests/src/FunctionalJavascript/JsBase.php
index 97351338..6f357602 100644
--- a/tests/src/FunctionalJavascript/JsBase.php
+++ b/tests/src/FunctionalJavascript/JsBase.php
@@ -151,10 +151,15 @@ abstract class JsBase extends WebDriverTestBase {
    * @throws \Drupal\Component\Plugin\Exception\PluginNotFoundException
    * @throws \Drupal\Core\Entity\EntityStorageException
    */
-  protected function createFacet($id, $field = 'type', $widget_type = 'links', array $widget_settings = [
-    'show_numbers' => TRUE,
-    'soft_limit' => 0,
-  ]) {
+  protected function createFacet(
+    $id,
+    $field = 'type',
+    $widget_type = 'links',
+    array $widget_settings = [
+      'show_numbers' => TRUE,
+      'soft_limit' => 0,
+    ],
+  ) {
     $facet_storage = \Drupal::entityTypeManager()->getStorage('facets_facet');
     // Create and save a facet with a checkbox widget.
     $facet_storage->create([
-- 
GitLab


From 594d7759f2acf45191936f0925c5d0a434838faa Mon Sep 17 00:00:00 2001
From: Liam Morland <liam@openplus.ca>
Date: Fri, 2 May 2025 09:27:40 -0400
Subject: [PATCH 2/4] Issue #3522436: Make .gitlab-ci.yml follow template

---
 .gitlab-ci.yml | 99 ++++++++------------------------------------------
 1 file changed, 15 insertions(+), 84 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 40d5c5a6..78c5a372 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -1,99 +1,30 @@
 ################
-# DrupalCI GitLabCI template
+# GitLabCI template for Drupal projects.
 #
-# Gitlab-ci.yml to replicate DrupalCI testing for Contrib
-#
-# With thanks to:
-#   * The GitLab Acceleration Initiative participants
-#   * DrupalSpoons
-################
-
-################
-# Guidelines
-#
-# This template is designed to give any Contrib maintainer everything they need to test, without requiring modification. It is also designed to keep up to date with Core Development automatically through the use of include files that can be centrally maintained.
-#
-# However, you can modify this template if you have additional needs for your project.
-################
-
-################
-# Includes
-#
-# Additional configuration can be provided through includes.
-# One advantage of include files is that if they are updated upstream, the changes affect all pipelines using that include.
-#
-# Includes can be overridden by re-declaring anything provided in an include, here in gitlab-ci.yml
-# https://docs.gitlab.com/ee/ci/yaml/includes.html#override-included-configuration-values
+# This template is designed to give any Contrib maintainer everything they need to test, without requiring modification.
+# It is also designed to keep up to date with Core Development automatically through the use of include files that can be centrally maintained.
+# As long as you include the project, ref and three files below, any future updates added by the Drupal Association will be used in your
+# pipelines automatically. However, you can modify this template if you have additional needs for your project.
+# The full documentation is on https://project.pages.drupalcode.org/gitlab_templates/
 ################
 
+# For information on alternative values for 'ref' see https://project.pages.drupalcode.org/gitlab_templates/info/templates-version/
+# To test a Drupal 7 project, change the first include filename from .main.yml to .main-d7.yml
 include:
-  ################
-  # DrupalCI includes:
-  # As long as you include this, any future includes added by the Drupal Association will be accessible to your pipelines automatically.
-  # View these include files at https://git.drupalcode.org/project/gitlab_templates/
-  ################
   - project: $_GITLAB_TEMPLATES_REPO
     ref: $_GITLAB_TEMPLATES_REF
     file:
       - '/includes/include.drupalci.main.yml'
-      # EXPERIMENTAL: For Drupal 7, remove the above line and uncomment the below.
-      # - '/includes/include.drupalci.main-d7.yml'
       - '/includes/include.drupalci.variables.yml'
       - '/includes/include.drupalci.workflows.yml'
-
-################
-# Pipeline configuration variables
 #
-# These are the variables provided to the Run Pipeline form that a user may want to override.
-#
-# Docs at https://git.drupalcode.org/project/gitlab_templates/-/blob/1.0.x/includes/include.drupalci.variables.yml
+################
+# Pipeline configuration variables are defined with default values and descriptions in the file
+# https://git.drupalcode.org/project/gitlab_templates/-/blob/main/includes/include.drupalci.variables.yml
+# Uncomment the lines below if you want to override any of the variables. The following is just an example.
 ################
 variables:
+#   SKIP_ESLINT: '1'
+#   OPT_IN_TEST_NEXT_MAJOR: '1'
+#   _CURL_TEMPLATES_REF: 'main'
   _PHPUNIT_CONCURRENT: '1'
-
-
-###################################################################################
-#
-#                                        *
-#                                       /(
-#                                      ((((,
-#                                    /(((((((
-#                                   ((((((((((*
-#                                ,(((((((((((((((
-#                              ,(((((((((((((((((((
-#                            ((((((((((((((((((((((((*
-#                         *(((((((((((((((((((((((((((((
-#                       ((((((((((((((((((((((((((((((((((*
-#                    *((((((((((((((((((  .((((((((((((((((((
-#                  ((((((((((((((((((.       /(((((((((((((((((*
-#                /(((((((((((((((((            .(((((((((((((((((,
-#             ,((((((((((((((((((                 ((((((((((((((((((
-#           .((((((((((((((((((((                   .(((((((((((((((((
-#          (((((((((((((((((((((((                     ((((((((((((((((/
-#        (((((((((((((((((((((((((((/                    ,(((((((((((((((*
-#      .((((((((((((((/  /(((((((((((((.                   ,(((((((((((((((
-#     *((((((((((((((      ,(((((((((((((/                   *((((((((((((((.
-#    ((((((((((((((,          /(((((((((((((.                  ((((((((((((((,
-#   (((((((((((((/              ,(((((((((((((*                 ,(((((((((((((,
-#  *(((((((((((((                .(((((((((((((((                ,(((((((((((((
-#  ((((((((((((/                /((((((((((((((((((.              ,((((((((((((/
-# (((((((((((((              *(((((((((((((((((((((((*             *((((((((((((
-# (((((((((((((            ,(((((((((((((..(((((((((((((           *((((((((((((
-# ((((((((((((,          /((((((((((((*      /((((((((((((/         ((((((((((((
-# (((((((((((((        /((((((((((((/          (((((((((((((*       ((((((((((((
-# (((((((((((((/     /((((((((((((               ,((((((((((((,    *((((((((((((
-#  ((((((((((((((  *(((((((((((/                   *((((((((((((.  ((((((((((((/
-#  *((((((((((((((((((((((((((,                      /(((((((((((((((((((((((((
-#   (((((((((((((((((((((((((                         ((((((((((((((((((((((((,
-#   .(((((((((((((((((((((((/                         ,(((((((((((((((((((((((
-#     ((((((((((((((((((((((/                         ,(((((((((((((((((((((/
-#      *(((((((((((((((((((((                         (((((((((((((((((((((,
-#       ,(((((((((((((((((((((,                      ((((((((((((((((((((/
-#         ,(((((((((((((((((((((*                  /((((((((((((((((((((
-#            ((((((((((((((((((((((,           ,/((((((((((((((((((((,
-#              ,(((((((((((((((((((((((((((((((((((((((((((((((((((
-#                 .(((((((((((((((((((((((((((((((((((((((((((((
-#                     .((((((((((((((((((((((((((((((((((((,.
-#                          .,(((((((((((((((((((((((((.
-#
-###################################################################################
-- 
GitLab


From 148bdee3568bc6e109055e7e597d15b87ce3eddf Mon Sep 17 00:00:00 2001
From: Liam Morland <liam@openplus.ca>
Date: Fri, 2 May 2025 09:36:47 -0400
Subject: [PATCH 3/4] Issue #3522436: Fix issues raised by stylelint

---
 css/checkbox-widget.css | 4 ++--
 css/facets.admin.css    | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/css/checkbox-widget.css b/css/checkbox-widget.css
index a36c7249..ff50f154 100644
--- a/css/checkbox-widget.css
+++ b/css/checkbox-widget.css
@@ -1,5 +1,5 @@
 .facets-widget-links ul li.facet-item label .facet-item__value {
   cursor: pointer;
-  color:blue;
-  text-decoration:underline;
+  text-decoration: underline;
+  color: blue;
 }
diff --git a/css/facets.admin.css b/css/facets.admin.css
index 5cea7596..7887d6b8 100644
--- a/css/facets.admin.css
+++ b/css/facets.admin.css
@@ -34,11 +34,11 @@
  */
 .facets-processor-settings-sorting {
   margin-bottom: -7px;
-  margin-left: 20px;
   margin-top: -16px;
+  margin-left: 20px;
 }
 
 .facets-processor-settings-facet {
-  margin-left: 20px;
   margin-bottom: 20px;
+  margin-left: 20px;
 }
-- 
GitLab


From bb39e40637cc1d9c8612ee078dac0d2c804c77b1 Mon Sep 17 00:00:00 2001
From: Liam Morland <liam@openplus.ca>
Date: Fri, 2 May 2025 09:40:20 -0400
Subject: [PATCH 4/4] Issue #3522436: Fix issues raised by stylelint

---
 css/facets.admin.css | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/css/facets.admin.css b/css/facets.admin.css
index 7887d6b8..a8c2ff55 100644
--- a/css/facets.admin.css
+++ b/css/facets.admin.css
@@ -33,8 +33,8 @@
  * Facets Display page
  */
 .facets-processor-settings-sorting {
-  margin-bottom: -7px;
   margin-top: -16px;
+  margin-bottom: -7px;
   margin-left: 20px;
 }
 
-- 
GitLab