Allow the reset button to be either above or below the rest of the results

There is currently no option to do this. I assume that it would be a good idea to make this configurable though.

Attaching initial patch with update.

Nice approach! Meantime I think the option should be string instead of int (more readable for config interdiffs)

Thanks for the tip! Attaching new patch and interdiff.

Now it needs post_update hook to set default value to existing config

NW for that

Hi, all here!

post_update hook was added.

+++ b/modules/facets_summary/facets_summary.post_update.php
@@ -0,0 +1,29 @@
+  $facets_summaries = \Drupal::entityTypeManager()->getStorage('facets_summary')->loadMultiple();
+  foreach ($facets_summaries as $facets_summary) {

Please use ConfigEntityUpdater for this kind of upgrades, see https://git.drupalcode.org/project/drupal/blob/8.8.x/core/modules/views/...

Usage of ConfigEntityUpdater was added.

Patch applies cleanly to 1.4 and works great!

* /admin/config/search/facets/facet-summary/[my_facets_summary]/edit
* Under the reset facets link settings there's a new dropdown for Position (see screenshot). I tested all three options Show reset link before facets link (default), Show reset link after facets link, and Show only reset link.

Thanks!

Rerolled against 2.x

1. +++ b/modules/facets_summary/facets_summary.post_update.php
   @@ -0,0 +1,32 @@
   +function facets_summary_post_update_set_reset_facets_position(&$sandbox = NULL) {
   

   Since a proper default value is provided, I assume that this post update is not needed.

   If it is needed an update hook should be used instead to modify the configs.

2. +++ b/modules/facets_summary/src/Plugin/facets_summary/processor/ResetFacetsProcessor.php
   @@ -23,6 +23,21 @@ use Drupal\facets_summary\Processor\ProcessorPluginBase;
   +   * Indicates, that reset link should be positioned after facet links.
   

   Comment is wrong.

Hi @mkalkbrenner, thanks for the review. Comment is fixed and update hook is added instead of post_update.