Seth commited on
Commit
7f301e6
·
1 Parent(s): 04c41a8
Files changed (1) hide show
  1. frontend/src/pages/Deals.jsx +21 -10
frontend/src/pages/Deals.jsx CHANGED
@@ -458,7 +458,7 @@ function DealRow({
458
  </Select>
459
  </td>
460
  <td className="px-3 py-2 align-top" onClick={(e) => e.stopPropagation()}>
461
- {isAdmin && tableEditRowId === deal.id ? (
462
  <Select
463
  value={
464
  deal.owner_user_id != null && deal.owner_user_id !== ''
@@ -473,14 +473,25 @@ function DealRow({
473
  }
474
  }}
475
  >
476
- <SelectTrigger className="h-8 min-w-[10rem] border-slate-200 shadow-none text-xs">
477
- <span className="truncate">
478
- {deal.owner_user_id == null
479
- ? 'Unassigned'
480
- : deal.owner_display_name ||
481
- deal.owner_initials ||
482
- `User ${deal.owner_user_id}`}
483
- </span>
 
 
 
 
 
 
 
 
 
 
 
484
  </SelectTrigger>
485
  <SelectContent className="min-w-[12rem]">
486
  <SelectItem value={EMPTY_OWNER_KEY}>Unassigned</SelectItem>
@@ -977,7 +988,7 @@ export default function Deals() {
977
  return (
978
  <AppShell
979
  title="Deals"
980
- subtitle="Pipeline by stage and owner. Members see their deals and unassigned deals; admins see all and can assign owners (edit row Owner)."
981
  >
982
  <MainTableWorkspace
983
  tabs={[
 
458
  </Select>
459
  </td>
460
  <td className="px-3 py-2 align-top" onClick={(e) => e.stopPropagation()}>
461
+ {isAdmin ? (
462
  <Select
463
  value={
464
  deal.owner_user_id != null && deal.owner_user_id !== ''
 
473
  }
474
  }}
475
  >
476
+ <SelectTrigger
477
+ className={cn(
478
+ 'h-9 min-w-[10.5rem] max-w-[14rem] border-slate-200 shadow-none px-2 py-1',
479
+ 'gap-2 [&>svg]:shrink-0'
480
+ )}
481
+ title="Change owner"
482
+ >
483
+ <div className="flex min-w-0 flex-1 items-center gap-2 text-left">
484
+ <div className="flex h-8 w-8 shrink-0 items-center justify-center rounded-full border border-violet-200 bg-violet-100 text-xs font-semibold text-violet-800">
485
+ {(deal.owner_initials || '?').toString().slice(0, 2).toUpperCase()}
486
+ </div>
487
+ <span className="truncate text-xs font-medium text-slate-800">
488
+ {deal.owner_user_id == null
489
+ ? 'Unassigned'
490
+ : deal.owner_display_name ||
491
+ deal.owner_initials ||
492
+ `User ${deal.owner_user_id}`}
493
+ </span>
494
+ </div>
495
  </SelectTrigger>
496
  <SelectContent className="min-w-[12rem]">
497
  <SelectItem value={EMPTY_OWNER_KEY}>Unassigned</SelectItem>
 
988
  return (
989
  <AppShell
990
  title="Deals"
991
+ subtitle="Pipeline by stage and owner. Members see their deals and unassigned deals; admins see all and can assign owners from the Owner column."
992
  >
993
  <MainTableWorkspace
994
  tabs={[