From 8fdd973a00857138909d21b86b8f102b49bd4c26 Mon Sep 17 00:00:00 2001
From: Stanislav Romanov <kaermorchen@gmail.com>
Date: Sat, 15 Oct 2022 14:07:29 +0300
Subject: [PATCH] Refactored listbox

---
 app/components/form/listbox.hbs                   | 15 +++++++++++----
 app/components/icons/menu-swap.js                 |  1 +
 .../admin/indexes/item/documents/index.hbs        |  4 ++--
 app/templates/admin/indexes/item/settings.hbs     |  4 ++--
 app/templates/admin/tasks/index.hbs               |  8 ++++----
 5 files changed, 20 insertions(+), 12 deletions(-)
 create mode 100644 app/components/icons/menu-swap.js

diff --git a/app/components/form/listbox.hbs b/app/components/form/listbox.hbs
index 3a991b3..768b485 100644
--- a/app/components/form/listbox.hbs
+++ b/app/components/form/listbox.hbs
@@ -7,23 +7,30 @@
 >
   <Listbox @value={{@value}} @onChange={{@onChange}} as |listbox|>
     <listbox.Button
-      class='inline-flex items-center px-3 py-2 border rounded cursor-pointer'
+      class='relative w-full inline-flex items-center py-2 pl-3 pr-10 border rounded cursor-pointer'
     >
       {{#if (has-block)}}
         {{yield @value}}
       {{else}}
-        {{@listboxLabel}}
+        {{@prefixValue}}
         {{@value}}
       {{/if}}
+
+      <span
+        class='absolute inset-y-0 right-0 flex items-center pr-2 pointer-events-none'
+      >
+        <Icons::MenuSwap class='fill-base-3' />
+      </span>
     </listbox.Button>
     <listbox.Options
-      class='absolute z-dropdown right-0 w-28 min-w-min mt-2 py-1 origin-top-right bg-base-7 border rounded shadow-sm outline-none'
+      class='absolute w-full z-dropdown right-0 min-w-min mt-2 py-1 origin-top-right bg-base-7 border rounded shadow-sm outline-none'
       as |options|
     >
       {{#each @values as |item|}}
         <options.Option @value={{item}} class='outline-none'>
           <div
-            class='flex items-center px-4 py-2 text-sm hover:bg-base-6 hover:text-base-1 cursor-pointer'
+            class='flex items-center px-4 py-2 text-sm hover:bg-base-6 hover:text-base-1 cursor-pointer
+              {{if (eq item @value) "bg-base-6 text-base-1"}}'
           >
             {{#if (has-block)}}
               {{yield item}}
diff --git a/app/components/icons/menu-swap.js b/app/components/icons/menu-swap.js
new file mode 100644
index 0000000..dca727a
--- /dev/null
+++ b/app/components/icons/menu-swap.js
@@ -0,0 +1 @@
+export { MenuSwap as default } from 'ember-mdi';
diff --git a/app/templates/admin/indexes/item/documents/index.hbs b/app/templates/admin/indexes/item/documents/index.hbs
index 0df2815..fdad16a 100644
--- a/app/templates/admin/indexes/item/documents/index.hbs
+++ b/app/templates/admin/indexes/item/documents/index.hbs
@@ -19,7 +19,7 @@
   {{! sort }}
   {{#if this.sortValues.length}}
     <Form::Listbox
-      @listboxLabel={{'Sort: '}}
+      @prefixValue={{'Sort: '}}
       @value={{nu-co this.sort 'none'}}
       @values={{this.sortValues}}
       @onChange={{set this 'sort'}}
@@ -30,7 +30,7 @@
 
   {{! limit }}
   <Form::Listbox
-    @listboxLabel={{'Limit: '}}
+    @prefixValue={{'Limit: '}}
     @value={{nu-co this.limit 'none'}}
     @values={{array 20 50 100 500}}
     @onChange={{set this 'limit'}}
diff --git a/app/templates/admin/indexes/item/settings.hbs b/app/templates/admin/indexes/item/settings.hbs
index 2d74990..219e828 100644
--- a/app/templates/admin/indexes/item/settings.hbs
+++ b/app/templates/admin/indexes/item/settings.hbs
@@ -3,9 +3,9 @@
   @values={{this.model.settings}}
   @onChange={{set this.model 'currentSetting'}}
   class="w-72"
-  as |setting|
+  as |item|
 >
-  {{if setting.name (namelize setting.name) 'All settings'}}
+  {{if item.name (namelize item.name) 'All settings'}}
 </Form::Listbox>
 
 <IndexSetting
diff --git a/app/templates/admin/tasks/index.hbs b/app/templates/admin/tasks/index.hbs
index 390ee27..1bd2627 100644
--- a/app/templates/admin/tasks/index.hbs
+++ b/app/templates/admin/tasks/index.hbs
@@ -3,7 +3,7 @@
 <div class='sm:flex sm:flex-nowrap'>
   {{! indexUid }}
   <Form::Listbox
-    @listboxLabel={{'Index: '}}
+    @prefixValue={{'Index: '}}
     @value={{nu-co this.indexUid 'none'}}
     @values={{map-by 'uid' @model.indexes.results}}
     @onChange={{set this 'indexUid'}}
@@ -12,7 +12,7 @@
 
   {{! status }}
   <Form::Listbox
-    @listboxLabel={{'Status: '}}
+    @prefixValue={{'Status: '}}
     @value={{nu-co this.status 'none'}}
     @values={{this.taskStatuses}}
     @onChange={{set this 'status'}}
@@ -22,7 +22,7 @@
 
   {{! status }}
   <Form::Listbox
-    @listboxLabel={{'Type: '}}
+    @prefixValue={{'Type: '}}
     @value={{nu-co this.type 'none'}}
     @values={{this.taskTypes}}
     @onChange={{set this 'type'}}
@@ -32,7 +32,7 @@
 
   {{! limit }}
   <Form::Listbox
-    @listboxLabel={{'Limit: '}}
+    @prefixValue={{'Limit: '}}
     @value={{nu-co this.limit 'none'}}
     @values={{array 20 50 100 500}}
     @onChange={{set this 'limit'}}