refactor(highlights): consolidate sync state into relay indicator

Show automatic rebroadcast/sync state in the relay indicator instead of separate meta spinner:
- Relay indicator shows spinner during both manual rebroadcast and auto-sync
- Update tooltip to distinguish between manual rebroadcast and auto-sync
- Remove redundant syncing indicator from meta area
- Clean up unused CSS for syncing indicator

This provides a single, consistent visual indicator for all relay broadcast states.
This commit is contained in:
Gigi
2025-10-09 16:11:33 +01:00
parent d7320c4bc8
commit 0c5c4b6c23
2 changed files with 4 additions and 20 deletions

View File

@@ -159,15 +159,16 @@ export const HighlightItem: React.FC<HighlightItemProps> = ({
// Determine relay indicator icon and tooltip
const getRelayIndicatorInfo = () => {
if (isRebroadcasting) {
// Show spinner if manually rebroadcasting OR auto-syncing
if (isRebroadcasting || isSyncing) {
return {
icon: faSpinner,
tooltip: 'Rebroadcasting to all relays...',
tooltip: isRebroadcasting ? 'Rebroadcasting to all relays...' : 'Auto-syncing to remote relays...',
spin: true
}
}
const isLocalOrOffline = highlight.isLocalOnly || (showOfflineIndicator && !isSyncing)
const isLocalOrOffline = highlight.isLocalOnly || showOfflineIndicator
if (isLocalOrOffline) {
return {
@@ -236,15 +237,6 @@ export const HighlightItem: React.FC<HighlightItemProps> = ({
{formatDistanceToNow(new Date(highlight.created_at * 1000), { addSuffix: true })}
</span>
{isSyncing && (
<>
<span className="highlight-meta-separator"></span>
<span className="highlight-syncing-indicator" title="Syncing to remote relays...">
<FontAwesomeIcon icon={faSpinner} spin />
</span>
</>
)}
{sourceLink && (
<a
href={sourceLink}

View File

@@ -1653,14 +1653,6 @@ body {
color: #888;
}
.highlight-syncing-indicator {
display: inline-flex;
align-items: center;
color: #3b82f6;
font-size: 0.8rem;
opacity: 0.9;
}
.highlight-source {
display: flex;
align-items: center;