From 461ee84d2a26009449211d1cb296236d1081cda4 Mon Sep 17 00:00:00 2001
From: Jose Olarte III <jose.anomalist.design@gmail.com>
Date: Sun, 12 Oct 2025 23:30:03 +0800
Subject: [PATCH] WIP: meeting members adjustments

---
 src/App.vue                             |   2 +-
 src/components/MembersList.vue          | 207 ++++++++++++------------
 src/views/OnboardMeetingMembersView.vue |  16 +-
 src/views/OnboardMeetingSetupView.vue   |  40 ++---
 4 files changed, 131 insertions(+), 134 deletions(-)

diff --git a/src/App.vue b/src/App.vue
index 245c5d5a..84d2b203 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -354,7 +354,7 @@
           </div>
 
           <div
-            v-if="notification.type === 'set-visibility-to-meeting-members'"
+            v-if="notification.type === 'set-visibility-meeting-members'"
             class="absolute inset-0 h-screen flex flex-col items-center justify-center bg-slate-900/50"
           >
             <div
diff --git a/src/components/MembersList.vue b/src/components/MembersList.vue
index d556418f..53970408 100644
--- a/src/components/MembersList.vue
+++ b/src/components/MembersList.vue
@@ -11,7 +11,7 @@
     <!-- Members List -->
 
     <div v-else>
-      <div class="text-center text-red-600 py-4">
+      <div class="text-center text-red-600 my-4">
         {{ decryptionErrorMessage() }}
       </div>
 
@@ -23,131 +23,125 @@
         to set it.
       </div>
 
-      <div>
-        <span
+      <ul class="list-disc text-sm ps-4 space-y-2 mb-4">
+        <li
           v-if="membersToShow().length > 0 && showOrganizerTools && isOrganizer"
-          class="inline-flex items-center flex-wrap"
         >
-          <span class="inline-flex items-center">
-            &bull; Click
-            <span
-              class="mx-2 min-w-[24px] min-h-[24px] w-6 h-6 flex items-center justify-center rounded-full bg-blue-100 text-blue-600"
-            >
-              <font-awesome icon="plus" class="text-sm" />
-            </span>
-            /
-            <span
-              class="mx-2 min-w-[24px] min-h-[24px] w-6 h-6 flex items-center justify-center rounded-full bg-blue-100 text-blue-600"
-            >
-              <font-awesome icon="minus" class="text-sm" />
-            </span>
-            to add/remove them to/from the meeting.
+          Click
+          <span
+            class="inline-block w-5 h-5 rounded-full bg-blue-100 text-blue-600 text-center"
+          >
+            <font-awesome icon="plus" class="text-sm" />
           </span>
-        </span>
-      </div>
-      <div>
-        <span
-          v-if="membersToShow().length > 0"
-          class="inline-flex items-center"
-        >
-          &bull; Click
+          /
+          <span
+            class="inline-block w-5 h-5 rounded-full bg-blue-100 text-blue-600 text-center"
+          >
+            <font-awesome icon="minus" class="text-sm" />
+          </span>
+          to add/remove them to/from the meeting.
+        </li>
+        <li v-if="membersToShow().length > 0">
+          Click
           <span
-            class="mx-2 w-8 h-8 flex items-center justify-center rounded-full bg-green-100 text-green-600"
+            class="inline-block w-5 h-5 rounded-full bg-green-100 text-green-600 text-center"
           >
-            <font-awesome icon="circle-user" class="text-xl" />
+            <font-awesome icon="circle-user" class="text-sm" />
           </span>
           to add them to your contacts.
-        </span>
-      </div>
+        </li>
+      </ul>
 
-      <div class="flex justify-center">
+      <div class="flex justify-between">
         <!-- 
         always have at least one refresh button even without members in case the organizer 
          changes the password 
          -->
         <button
-          class="btn-action-refresh"
+          class="text-sm bg-gradient-to-b from-slate-400 to-slate-700 shadow-[inset_0_-1px_0_0_rgba(0,0,0,0.5)] text-white px-3 py-1.5 rounded-md"
           title="Refresh members list"
           @click="fetchMembers"
         >
           <font-awesome icon="rotate" :class="{ 'fa-spin': isLoading }" />
+          Refresh <span class="text-xs">(10s)</span>
+        </button>
+
+        <button
+          class="text-sm bg-gradient-to-b from-slate-400 to-slate-700 shadow-[inset_0_-1px_0_0_rgba(0,0,0,0.5)] text-white px-3 py-1.5 rounded-md"
+          title="Set visibility for meeting members"
+          @click="showAddMembersNotification"
+        >
+          <font-awesome icon="eye" />
+          Set Visibility
         </button>
       </div>
-      <div
-        v-for="member in membersToShow()"
-        :key="member.member.memberId"
-        class="mt-2 p-4 bg-gray-50 rounded-lg"
+      <ul
+        v-if="membersToShow().length > 0"
+        class="border-t border-slate-300 my-2"
       >
-        <div class="flex items-center justify-between">
-          <div class="flex items-center">
-            <h3 class="text-lg font-medium">
-              {{ member.name || unnamedMember }}
-            </h3>
-            <div
-              v-if="!getContactFor(member.did) && member.did !== activeDid"
-              class="flex justify-end"
-            >
-              <button
-                class="btn-add-contact"
-                title="Add as contact"
-                @click="addAsContact(member)"
+        <li
+          v-for="member in membersToShow()"
+          :key="member.member.memberId"
+          class="border-b border-slate-300 py-1.5"
+        >
+          <div class="flex items-center justify-between">
+            <div class="flex items-center overflow-hidden">
+              <h3 class="font-semibold truncate">
+                {{ member.name || unnamedMember }}
+              </h3>
+              <div
+                v-if="!getContactFor(member.did) && member.did !== activeDid"
+                class="flex justify-end"
               >
-                <font-awesome icon="circle-user" class="text-xl" />
-              </button>
+                <button
+                  class="btn-add-contact"
+                  title="Add as contact"
+                  @click="addAsContact(member)"
+                >
+                  <font-awesome icon="circle-user" class="text-sm" />
+                </button>
+              </div>
             </div>
-            <button
-              v-if="member.did !== activeDid"
-              class="btn-info-contact"
-              title="Contact info"
-              @click="
-                informAboutAddingContact(
-                  getContactFor(member.did) !== undefined,
-                )
-              "
-            >
-              <font-awesome icon="circle-info" class="text-base" />
-            </button>
-          </div>
-          <div class="flex">
-            <span
-              v-if="
-                showOrganizerTools && isOrganizer && member.did !== activeDid
-              "
-              class="flex items-center"
-            >
-              <button
-                class="btn-admission"
-                :title="
-                  member.member.admitted ? 'Remove member' : 'Admit member'
+            <div class="flex">
+              <span
+                v-if="
+                  showOrganizerTools && isOrganizer && member.did !== activeDid
                 "
-                @click="checkWhetherContactBeforeAdmitting(member)"
-              >
-                <font-awesome
-                  :icon="member.member.admitted ? 'minus' : 'plus'"
-                  class="text-sm"
-                />
-              </button>
-              <button
-                class="btn-info-admission"
-                title="Admission info"
-                @click="informAboutAdmission()"
+                class="flex items-center"
               >
-                <font-awesome icon="circle-info" class="text-base" />
-              </button>
-            </span>
+                <button
+                  class="btn-admission"
+                  :title="
+                    member.member.admitted ? 'Remove member' : 'Admit member'
+                  "
+                  @click="checkWhetherContactBeforeAdmitting(member)"
+                >
+                  <font-awesome
+                    :icon="member.member.admitted ? 'minus' : 'plus'"
+                    class="text-sm"
+                  />
+                </button>
+              </span>
+            </div>
           </div>
-        </div>
-        <p class="text-sm text-gray-600 truncate">
-          {{ member.did }}
-        </p>
-      </div>
-      <div v-if="membersToShow().length > 0" class="flex justify-center mt-4">
+          <p class="text-xs text-gray-600 truncate">
+            {{ member.did }}
+          </p>
+        </li>
+      </ul>
+
+      <div v-if="membersToShow().length > 0" class="flex justify-between">
+        <!-- 
+        always have at least one refresh button even without members in case the organizer 
+         changes the password 
+         -->
         <button
-          class="btn-action-refresh"
+          class="text-sm bg-gradient-to-b from-slate-400 to-slate-700 shadow-[inset_0_-1px_0_0_rgba(0,0,0,0.5)] text-white px-3 py-1.5 rounded-md"
           title="Refresh members list"
           @click="fetchMembers"
         >
           <font-awesome icon="rotate" :class="{ 'fa-spin': isLoading }" />
+          Refresh <span class="text-xs">(10s)</span>
         </button>
       </div>
 
@@ -508,6 +502,21 @@ export default class MembersList extends Vue {
       this.notify.error(message, TIMEOUTS.LONG);
     }
   }
+
+  showAddMembersNotification() {
+    this.$notify(
+      {
+        group: "modal",
+        type: "set-visibility-meeting-members",
+        title: "Set Visibility for Meeting Members",
+        onYes: async () => {
+          // Handle the "Add Selected" action - you can implement the actual logic here
+          console.log("User confirmed adding selected members as contacts");
+        },
+      },
+      -1,
+    ); // -1 means no auto-dismiss, stays open until user acts
+  }
 }
 </script>
 
@@ -522,7 +531,7 @@ export default class MembersList extends Vue {
 
 .btn-add-contact {
   /* stylelint-disable-next-line at-rule-no-unknown */
-  @apply ml-2 w-8 h-8 flex items-center justify-center rounded-full 
+  @apply ml-2 w-5 h-5 flex items-center justify-center rounded-full 
     bg-green-100 text-green-600 hover:bg-green-200 hover:text-green-800 
     transition-colors;
 }
@@ -536,14 +545,14 @@ export default class MembersList extends Vue {
 
 .btn-admission {
   /* stylelint-disable-next-line at-rule-no-unknown */
-  @apply mr-2 w-6 h-6 flex items-center justify-center rounded-full 
+  @apply w-5 h-5 flex items-center justify-center rounded-full 
     bg-blue-100 text-blue-600 hover:bg-blue-200 hover:text-blue-800 
     transition-colors;
 }
 
 .btn-info-admission {
   /* stylelint-disable-next-line at-rule-no-unknown */
-  @apply mr-2 mb-2 w-6 h-6 flex items-center justify-center rounded-full 
+  @apply w-6 h-6 flex items-center justify-center rounded-full 
     bg-slate-100 text-slate-500 hover:bg-slate-200 hover:text-slate-800 
     transition-colors;
 }
diff --git a/src/views/OnboardMeetingMembersView.vue b/src/views/OnboardMeetingMembersView.vue
index 6cb1bf8b..9dfba3d9 100644
--- a/src/views/OnboardMeetingMembersView.vue
+++ b/src/views/OnboardMeetingMembersView.vue
@@ -7,7 +7,7 @@
     <!-- Sub View Heading -->
     <div id="SubViewHeading" class="flex gap-4 items-start mb-8">
       <h1 class="grow text-xl text-center font-semibold leading-tight">
-        Meeting Member<a href="#" @click="showAddMembersNotification">s</a>
+        Meeting Members
       </h1>
 
       <!-- Spacer (no Back button) -->
@@ -259,19 +259,5 @@ export default class OnboardMeetingMembersView extends Vue {
   handleError(message: string) {
     this.errorMessage = message;
   }
-
-  showAddMembersNotification() {
-    this.$notify(
-      {
-        group: "modal",
-        type: "set-visibility-to-meeting-members",
-        onYes: async () => {
-          // Handle the "Add Selected" action - you can implement the actual logic here
-          console.log("User confirmed adding selected members as contacts");
-        },
-      },
-      -1,
-    ); // -1 means no auto-dismiss, stays open until user acts
-  }
 }
 </script>
diff --git a/src/views/OnboardMeetingSetupView.vue b/src/views/OnboardMeetingSetupView.vue
index 45b2d580..e70148f5 100644
--- a/src/views/OnboardMeetingSetupView.vue
+++ b/src/views/OnboardMeetingSetupView.vue
@@ -230,26 +230,28 @@
       class="mt-8 p-4 border rounded-lg bg-white shadow"
     >
       <div class="flex items-center justify-between">
-        <h2 class="text-2xl">Meeting Members</h2>
-      </div>
-      <div class="mt-4">
-        &bull; Page for Members
-        <span
-          class="ml-4 cursor-pointer text-blue-600"
-          title="Click to copy link for members"
-          @click="copyMembersLinkToClipboard"
-        >
-          <font-awesome icon="link" />
-        </span>
-        <a
-          v-if="!!currentMeeting.password"
-          :href="onboardMeetingMembersLink()"
-          class="ml-4 text-blue-600"
-          target="_blank"
-        >
-          <font-awesome icon="external-link" />
-        </a>
+        <h2 class="font-bold">Meeting Members</h2>
       </div>
+      <ul class="list-disc text-sm mt-4 mb-2 ps-4 space-y-2">
+        <li>
+          Page for Members:
+          <span
+            class="ml-4 cursor-pointer text-blue-600"
+            title="Click to copy link for members"
+            @click="copyMembersLinkToClipboard"
+          >
+            <font-awesome icon="link" />
+          </span>
+          <a
+            v-if="!!currentMeeting.password"
+            :href="onboardMeetingMembersLink()"
+            class="ml-4 text-blue-600"
+            target="_blank"
+          >
+            <font-awesome icon="external-link" />
+          </a>
+        </li>
+      </ul>
 
       <MembersList
         :password="currentMeeting.password || ''"