@mdguggenbichler/slugbase-core 0.0.29 → 0.0.30

package/frontend/src/components/sharing/ShareResourceDialog.tsx

@@ -14,7 +14,7 @@ import { ScrollArea } from '../ui/scroll-area';
 import Button from '../ui/Button';
 import Tooltip from '../ui/Tooltip';
 import { useToast } from '../ui/Toast';
-import { UserPlus, User, Users, X } from 'lucide-react';
+import { User, Users, X } from 'lucide-react';
 import { cn } from '@/lib/utils';
 
 interface SharedUser {
@@ -146,9 +146,10 @@ export default function ShareResourceDialog({
       const msg = err.response?.data?.error || t('common.error');
       setError(msg);
       showToast(msg, 'error');
-    } finally {
       setSaving(false);
+      throw err;
     }
+    setSaving(false);
   }
 
   function handleRemoveUser(userId: string) {
@@ -162,11 +163,19 @@ export default function ShareResourceDialog({
   }
 
   function handleAddUser(userId: string) {
+    if (sharedUsers.some((u) => u.id === userId)) return;
+    const userToAdd = allUsers.find((u) => u.id === userId);
     const newUserIds = [...sharedUsers.map((u) => u.id), userId];
-    if (newUserIds.includes(userId)) return;
-    updateShares(newUserIds, sharedTeams.map((t) => t.id), false);
+    if (userToAdd) {
+      setSharedUsers((prev) => [...prev, userToAdd]);
+    }
     setPeopleDropdownOpen(false);
     setEmailInput('');
+    updateShares(newUserIds, sharedTeams.map((t) => t.id), false).catch(() => {
+      if (userToAdd) {
+        setSharedUsers((prev) => prev.filter((u) => u.id !== userId));
+      }
+    });
   }
 
   function handleAddUserByEmail() {
@@ -181,9 +190,17 @@ export default function ShareResourceDialog({
   }
 
   function handleAddTeam(teamId: string) {
+    if (sharedTeams.some((t) => t.id === teamId)) return;
+    const teamToAdd = teams.find((t) => t.id === teamId);
     const newTeamIds = [...sharedTeams.map((t) => t.id), teamId];
-    if (newTeamIds.includes(teamId)) return;
-    updateShares(sharedUsers.map((u) => u.id), newTeamIds, false);
+    if (teamToAdd) {
+      setSharedTeams((prev) => [...prev, teamToAdd]);
+    }
+    updateShares(sharedUsers.map((u) => u.id), newTeamIds, false).catch(() => {
+      if (teamToAdd) {
+        setSharedTeams((prev) => prev.filter((t) => t.id !== teamId));
+      }
+    });
   }
 
   const searchQuery = emailInput.trim().toLowerCase();
@@ -326,41 +343,29 @@ export default function ShareResourceDialog({
               {allowShareToUsers && (activeTab === 'people' || !allowShareToTeams) && (
                 <div className="space-y-2">
                   <div className="relative">
-                    <div className="flex gap-2">
-                      <Input
-                        placeholder={t('admin.searchUsers')}
-                        value={emailInput}
-                        onChange={(e) => {
-                          setEmailInput(e.target.value);
+                    <Input
+                      placeholder={t('admin.searchUsers')}
+                      value={emailInput}
+                      onChange={(e) => {
+                        setEmailInput(e.target.value);
+                        setPeopleDropdownOpen(true);
+                      }}
+                      onFocus={() => {
+                        if (searchQuery.length >= MIN_CHARS_FOR_USER_DROPDOWN)
                           setPeopleDropdownOpen(true);
-                        }}
-                        onFocus={() => {
-                          if (searchQuery.length >= MIN_CHARS_FOR_USER_DROPDOWN)
-                            setPeopleDropdownOpen(true);
-                        }}
-                        onBlur={() => {
-                          setTimeout(() => setPeopleDropdownOpen(false), 150);
-                        }}
-                        onKeyDown={(e) => {
-                          if (e.key === 'Enter') {
-                            e.preventDefault();
-                            handleAddUserByEmail();
-                          }
-                        }}
-                        className="flex-1"
-                        autoComplete="off"
-                      />
-                      <Button
-                        type="button"
-                        variant="outline"
-                        size="sm"
-                        disabled={saving}
-                        onClick={() => handleAddUserByEmail()}
-                      >
-                        <UserPlus className="h-4 w-4" />
-                        {t('sharing.add')}
-                      </Button>
-                    </div>
+                      }}
+                      onBlur={() => {
+                        setTimeout(() => setPeopleDropdownOpen(false), 150);
+                      }}
+                      onKeyDown={(e) => {
+                        if (e.key === 'Enter') {
+                          e.preventDefault();
+                          handleAddUserByEmail();
+                        }
+                      }}
+                      className="w-full"
+                      autoComplete="off"
+                    />
                     {peopleDropdownOpen && showUserDropdown && (
                       <div
                         className="absolute top-full left-0 right-0 z-10 mt-1 max-h-48 overflow-auto rounded-md border bg-popover shadow-md"

package/frontend/src/pages/Folders.tsx

@@ -368,8 +368,6 @@ export default function Folders() {
                       )}
                     </div>
                   </div>
-                  {/* TODO: Add bookmark_count when backend supports it */}
-                  <p className="text-xs text-muted-foreground">—</p>
                 </div>
               </Link>
               {folder.folder_type === 'own' && (

package/frontend/src/pages/Tags.tsx

@@ -287,8 +287,6 @@ export default function Tags() {
                       </h3>
                     </div>
                   </div>
-                  {/* TODO: Add bookmark_count when backend supports it */}
-                  <p className="text-xs text-muted-foreground">—</p>
                 </div>
               </Link>
               <div className={`flex gap-1.5 pt-2.5 mt-auto shrink-0 border-t border-border ${compactMode ? 'pt-2' : ''}`}>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mdguggenbichler/slugbase-core",
-  "version": "0.0.29",
+  "version": "0.0.30",
   "description": "SlugBase core: backend and frontend entrypoints for self-hosted and cloud apps",
   "type": "module",
   "exports": {