fix(chat): prevent display-name flicker and admin-badge duplication on hover

[novavale]

Problem

Fixes #248

When hovering over a chat message, two visual bugs appeared:

1. Display name disappeared — temporarily replaced by ...
2. Admin/mod badge duplicated momentarily in the header row

Root Cause

MessageDisplayState stored hover state as a bool field and called setState() on enter/exit. This triggered a full rebuild of the widget, including UserInfoBuilder. During that rebuild UserInfoBuilder briefly re-entered its loading state, making snapshot.data == null and causing the display name to fall back to '...'. The same rebuild caused Wrap to re-lay out its children, producing a momentary duplicate paint of the admin badge.

Fix

Replace the bool _isHovered field with a ValueNotifier<bool> and wrap only the background Container in a ValueListenableBuilder. The message content Row is passed as the static child argument of ValueListenableBuilder — Flutter reuses it unchanged when the notifier fires, so UserInfoBuilder is never re-triggered by a hover event.

// Before — setState triggers full rebuild
onEnter: (hover) => setState(() => _isHovered = true),

// After — only Container color re-evaluates
final _isHovered = ValueNotifier<bool>(false);
onEnter: (_) => _isHovered.value = true,

Testing

• Hovered over chat messages with admin/mod senders — name stays stable, no badge duplication
• Verified hover highlight still appears/disappears correctly
• Verified delete button still renders for permitted users

[katherineqian]

Hi @novavale, thank you so much for your contribution! I will test out these updates and provide a review early next week. Have a great weekend!

[Copilot]

Pull request overview

This PR addresses a Flutter chat UI hover regression where message header content (display name and admin/mod badge) visually flickered/duplicated due to hover-triggered full widget rebuilds.

Changes:

• Replaced hover tracking from setState-backed bool to a ValueNotifier<bool> to avoid rebuilding the full widget subtree on hover.
• Wrapped only the background Container in a ValueListenableBuilder, passing the message content as the static child to prevent UserInfoBuilder from being retriggered.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.