From 53c209ff120afd6d127784116bade66713527ade Mon Sep 17 00:00:00 2001
From: alicegoarnisson <alice.goarnisson@pix.fr>
Date: Fri, 15 Nov 2024 15:29:09 +0100
Subject: [PATCH] feat(pix-ui): mutualized pix select list

---
 addon/components/pix-select-list.hbs | 83 +++++++++++++++++++++++
 addon/components/pix-select.hbs      | 98 ++++------------------------
 app/components/pix-select-list.js    |  1 +
 3 files changed, 96 insertions(+), 86 deletions(-)
 create mode 100644 addon/components/pix-select-list.hbs
 create mode 100644 app/components/pix-select-list.js

diff --git a/addon/components/pix-select-list.hbs b/addon/components/pix-select-list.hbs
new file mode 100644
index 000000000..57cbd55ba
--- /dev/null
+++ b/addon/components/pix-select-list.hbs
@@ -0,0 +1,83 @@
+<ul role="listbox" id={{@listId}} class="pix-select__options">
+  <li
+    class="pix-select-options-category__option{{unless
+        @value
+        ' pix-select-options-category__option--selected'
+      }}{{unless @displayDefaultOption ' pix-select-options-category__option--hidden'}}"
+    role="option"
+    tabindex={{if @isDefaultOptionHidden "-1" "0"}}
+    aria-selected={{if @value "false" "true"}}
+    {{on "click" (fn @onChange @defaultOption)}}
+    {{on-enter-action (fn @onChange @defaultOption)}}
+    {{on-space-action (fn @onChange @defaultOption)}}
+  >
+    {{@placeholder}}
+  </li>
+  {{#if @results}}
+    {{#if @displayCategory}}
+      {{#each @results as |element index|}}
+        <ul
+          class="pix-select-options__category"
+          role="group"
+          aria-labelledby={{if @displayCategory (concat "cat-" @selectId "-" index)}}
+        >
+          {{#if @displayCategory}}
+            <li
+              class="pix-select-options-category__name"
+              role="presentation"
+              id={{concat "cat-" @selectId "-" index}}
+            >
+              {{element.category}}
+            </li>
+          {{/if}}
+
+          {{#each element.options as |option|}}
+            {{! template-lint-disable require-context-role }}
+            {{!https://www.w3.org/WAI/ARIA/apg/example-index/listbox/listbox-grouped.html}}
+            <li
+              class="pix-select-options-category__option{{if
+                  (eq option.value @value)
+                  ' pix-select-options-category__option--selected'
+                }}"
+              role="option"
+              tabindex={{if @isExpanded "0" "-1"}}
+              aria-selected={{if (eq option.value @value) "true" "false"}}
+              {{on "click" (fn @onChange option)}}
+              {{on-enter-action (fn @onChange option)}}
+              {{on-space-action (fn @onChange option)}}
+            >
+              {{option.label}}
+
+              {{#if (eq option.value @value)}}
+                <PixIcon @name="check" role="presentation" />
+              {{/if}}
+            </li>
+          {{/each}}
+        </ul>
+      {{/each}}
+    {{else}}
+      {{#each @results as |option|}}
+        <li
+          class="pix-select-options-category__option{{if
+              (eq option.value @value)
+              ' pix-select-options-category__option--selected'
+            }}"
+          role="option"
+          tabindex={{if @isExpanded "0" "-1"}}
+          aria-selected={{if (eq option.value @value) "true" "false"}}
+          {{on "click" (fn @onChange option)}}
+          {{on-enter-action (fn @onChange option)}}
+          {{on-space-action (fn @onChange @defaultOption)}}
+        >
+          {{option.label}}
+
+          {{#if (eq option.value @value)}}
+            <PixIcon @name="check" role="presentation" />
+          {{/if}}
+        </li>
+      {{/each}}
+    {{/if}}
+  {{else}}
+    <li class="pix-select__empty-search-message">{{@emptySearchMessage}}</li>
+  {{/if}}
+</ul>
\ No newline at end of file
diff --git a/addon/components/pix-select.hbs b/addon/components/pix-select.hbs
index db94f27c6..4a3c63535 100644
--- a/addon/components/pix-select.hbs
+++ b/addon/components/pix-select.hbs
@@ -68,92 +68,18 @@
             />
           </div>
         {{/if}}
-        <ul role="listbox" id={{this.listId}} class="pix-select__options">
-          <li
-            class="pix-select-options-category__option{{unless
-                @value
-                ' pix-select-options-category__option--selected'
-              }}{{unless this.displayDefaultOption ' pix-select-options-category__option--hidden'}}"
-            role="option"
-            tabindex={{if this.isDefaultOptionHidden "-1" "0"}}
-            aria-selected={{if @value "false" "true"}}
-            {{on "click" (fn this.onChange this.defaultOption)}}
-            {{on-enter-action (fn this.onChange this.defaultOption)}}
-            {{on-space-action (fn this.onChange this.defaultOption)}}
-          >
-            {{@placeholder}}
-          </li>
-          {{#if this.results}}
-            {{#if this.displayCategory}}
-              {{#each this.results as |element index|}}
-                <ul
-                  class="pix-select-options__category"
-                  role="group"
-                  aria-labelledby={{if
-                    this.displayCategory
-                    (concat "cat-" this.selectId "-" index)
-                  }}
-                >
-                  {{#if this.displayCategory}}
-                    <li
-                      class="pix-select-options-category__name"
-                      role="presentation"
-                      id={{concat "cat-" this.selectId "-" index}}
-                    >
-                      {{element.category}}
-                    </li>
-                  {{/if}}
-
-                  {{#each element.options as |option|}}
-                    {{! template-lint-disable require-context-role }}
-                    {{!https://www.w3.org/WAI/ARIA/apg/example-index/listbox/listbox-grouped.html}}
-                    <li
-                      class="pix-select-options-category__option{{if
-                          (eq option.value @value)
-                          ' pix-select-options-category__option--selected'
-                        }}"
-                      role="option"
-                      tabindex={{if this.isExpanded "0" "-1"}}
-                      aria-selected={{if (eq option.value @value) "true" "false"}}
-                      {{on "click" (fn this.onChange option)}}
-                      {{on-enter-action (fn this.onChange option)}}
-                      {{on-space-action (fn this.onChange option)}}
-                    >
-                      {{option.label}}
-
-                      {{#if (eq option.value @value)}}
-                        <PixIcon @name="check" role="presentation" />
-                      {{/if}}
-                    </li>
-                  {{/each}}
-                </ul>
-              {{/each}}
-            {{else}}
-              {{#each this.results as |option|}}
-                <li
-                  class="pix-select-options-category__option{{if
-                      (eq option.value @value)
-                      ' pix-select-options-category__option--selected'
-                    }}"
-                  role="option"
-                  tabindex={{if this.isExpanded "0" "-1"}}
-                  aria-selected={{if (eq option.value @value) "true" "false"}}
-                  {{on "click" (fn this.onChange option)}}
-                  {{on-enter-action (fn this.onChange option)}}
-                  {{on-space-action (fn this.onChange this.defaultOption)}}
-                >
-                  {{option.label}}
-
-                  {{#if (eq option.value @value)}}
-                    <PixIcon @name="check" role="presentation" />
-                  {{/if}}
-                </li>
-              {{/each}}
-            {{/if}}
-          {{else}}
-            <li class="pix-select__empty-search-message">{{@emptySearchMessage}}</li>
-          {{/if}}
-        </ul>
+        <PixSelectList
+          @listId={{this.listId}}
+          @value={{@value}}
+          @displayDefaultOption={{this.displayDefaultOption}}
+          @isDefaultOptionHidden={{this.isDefaultOptionHidden}}
+          @results={{this.results}}
+          @onChange={{this.onChange}}
+          @defaultOption={{this.defaultOption}}
+          @displayCategory={{this.displayCategory}}
+          @selectId={{this.selectId}}
+          @isExpanded={{this.isExpanded}}
+        />
       </div>
     </PopperJS>
     {{#if @errorMessage}}
diff --git a/app/components/pix-select-list.js b/app/components/pix-select-list.js
new file mode 100644
index 000000000..1ebed758c
--- /dev/null
+++ b/app/components/pix-select-list.js
@@ -0,0 +1 @@
+export { default } from '@1024pix/pix-ui/components/pix-select-list';