Skip to content

Commit 09f0939

Browse files
julianknutsenclaude
andcommitted
fix: use span instead of div inside hover card to fix DOM nesting
React warns about div inside span. The CSS display properties (block, flex) handle layout regardless of the HTML element. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 7f61c9f commit 09f0939

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

web/src/components/BrowseList.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -235,9 +235,9 @@ function PendingIndicator({ count, items }: { count: number; items?: PendingItem
235235
{count > 1 && <span className={styles.pendingCount}>&times;{count}</span>}
236236
{items && items.length > 0 && (
237237
<span className={styles.pendingCard}>
238-
<div className={styles.pendingCardTitle}>Competing submissions</div>
238+
<span className={styles.pendingCardTitle}>Competing submissions</span>
239239
{items.map((p, i) => (
240-
<div key={i} className={styles.pendingCardRow}>
240+
<span key={i} className={styles.pendingCardRow}>
241241
<span className={styles.pendingCardHandle}>{p.rig_handle}</span>
242242
{p.status && <span className={styles.pendingCardStatus}>{p.status.replace("_", " ")}</span>}
243243
{p.pr_url && (
@@ -250,7 +250,7 @@ function PendingIndicator({ count, items }: { count: number; items?: PendingItem
250250
branch
251251
</a>
252252
)}
253-
</div>
253+
</span>
254254
))}
255255
</span>
256256
)}

0 commit comments

Comments
 (0)