Skip to content

Commit eabdf12

Browse files
committed
Refactor proposal status rendering and improve code consistency in AdminActionBar, ProposalActionModal, and CompactProposalList components
1 parent 6c945c0 commit eabdf12

3 files changed

Lines changed: 19 additions & 16 deletions

File tree

src/components/admin/AdminActionBar.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,8 @@ export function AdminActionBar({
160160
Status:
161161
</span>
162162
<span
163-
className={`inline-flex items-center rounded-full px-2 py-1 text-xs font-medium ${proposal.status === 'accepted'
163+
className={`inline-flex items-center rounded-full px-2 py-1 text-xs font-medium ${
164+
proposal.status === 'accepted'
164165
? 'bg-green-100 text-green-800 dark:bg-green-900 dark:text-green-200'
165166
: proposal.status === 'waitlisted'
166167
? 'bg-orange-100 text-orange-800 dark:bg-orange-900 dark:text-orange-200'
@@ -169,7 +170,7 @@ export function AdminActionBar({
169170
: proposal.status === 'submitted'
170171
? 'bg-yellow-100 text-yellow-800 dark:bg-yellow-900 dark:text-yellow-200'
171172
: 'bg-gray-100 text-gray-800 dark:bg-gray-700 dark:text-gray-200'
172-
}`}
173+
}`}
173174
>
174175
{proposal.status.charAt(0).toUpperCase() +
175176
proposal.status.slice(1)}

src/components/admin/ProposalActionModal.tsx

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -198,15 +198,15 @@ export function ProposalActionModal({
198198
<span className="font-semibold">{proposal.title}</span> by{' '}
199199
<span className="font-semibold">
200200
{proposal.speakers &&
201-
Array.isArray(proposal.speakers) &&
202-
proposal.speakers.length > 0
201+
Array.isArray(proposal.speakers) &&
202+
proposal.speakers.length > 0
203203
? proposal.speakers
204-
.map((speaker) =>
205-
typeof speaker === 'object' && 'name' in speaker
206-
? (speaker as Speaker).name
207-
: 'Unknown',
208-
)
209-
.join(', ')
204+
.map((speaker) =>
205+
typeof speaker === 'object' && 'name' in speaker
206+
? (speaker as Speaker).name
207+
: 'Unknown',
208+
)
209+
.join(', ')
210210
: 'Unknown author'}
211211
</span>
212212
?

src/components/cfp/CompactProposalList.tsx

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,8 @@ export function CompactProposalList({
107107
const audienceFeedback = proposal.audienceFeedback
108108
const totalFeedback = audienceFeedback
109109
? audienceFeedback.greenCount +
110-
audienceFeedback.yellowCount +
111-
audienceFeedback.redCount
110+
audienceFeedback.yellowCount +
111+
audienceFeedback.redCount
112112
: 0
113113
const showFeedback =
114114
conferenceHasEnded &&
@@ -133,10 +133,11 @@ export function CompactProposalList({
133133
return (
134134
<div
135135
key={proposal._id}
136-
className={`flex flex-col gap-3 rounded-lg border px-3 py-2.5 text-sm transition-colors sm:flex-row sm:items-center sm:justify-between ${isMuted
136+
className={`flex flex-col gap-3 rounded-lg border px-3 py-2.5 text-sm transition-colors sm:flex-row sm:items-center sm:justify-between ${
137+
isMuted
137138
? 'border-gray-200/50 bg-gray-50/50 opacity-60 hover:opacity-80 dark:border-gray-700/50 dark:bg-gray-800/50'
138139
: 'border-gray-200 bg-white hover:border-gray-300 hover:shadow-sm dark:border-gray-700 dark:bg-gray-800 dark:hover:border-gray-600'
139-
}`}
140+
}`}
140141
>
141142
<div className="flex min-w-0 flex-1 items-center gap-3 overflow-hidden">
142143
{proposal.speakers && proposal.speakers.length > 0 && (
@@ -158,10 +159,11 @@ export function CompactProposalList({
158159
? `/cfp/proposal?id=${proposal._id}${queryString ? `&${queryString.slice(1)}` : ''}`
159160
: `/cfp/proposal/${proposal._id}${queryString}`
160161
}
161-
className={`flex-1 truncate font-medium hover:text-brand-cloud-blue dark:hover:text-blue-400 ${isMuted
162+
className={`flex-1 truncate font-medium hover:text-brand-cloud-blue dark:hover:text-blue-400 ${
163+
isMuted
162164
? 'text-gray-600 dark:text-gray-400'
163165
: 'text-gray-900 dark:text-white'
164-
}`}
166+
}`}
165167
>
166168
{proposal.title}
167169
</Link>

0 commit comments

Comments
 (0)