richtext: underline all wrapped hyperlink segments on hover (#6227)

* richtext: underline all wrapped hyperlink segments on hover

When a HyperlinkSegment wraps across multiple rows, each row gets its
own Hyperlink widget instance. Hovering one now propagates the hovered
state to all sibling instances so the underline appears across the full
link, not just the segment under the cursor.

Siblings are linked inline during textRenderer.Refresh using the
existing visualCache: reuse equals the current visual's cache offset and
the count of prior row visuals, so earlier instances are fetched at
offsets 0..reuse-1 and wired into an all-to-all sibling list with no
extra allocations per refresh.

Fixes #6226

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* update test mouse position

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* simplify comment

* move to separate func

* Update richtext.go

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: 3 people

widget/hyperlink.go

@@ -44,6 +44,8 @@ type Hyperlink struct {
 	textSize         fyne.Size // updated in syncSegments
 	focused, hovered bool
 	provider         RichText
+
+	siblings []*Hyperlink // other visual instances of the same HyperlinkSegment when wrapped in RichText
 }
 
 // NewHyperlink creates a new hyperlink widget with the set text content
@@ -111,6 +113,9 @@ func (hl *Hyperlink) MouseMoved(e *desktop.MouseEvent) {
 	hl.hovered = hl.isPosOverText(e.Position)
 	if hl.hovered != oldHovered {
 		hl.BaseWidget.Refresh()
+		for _, s := range hl.siblings {
+			s.setHovered(hl.hovered)
+		}
 	}
 }
 
@@ -120,7 +125,19 @@ func (hl *Hyperlink) MouseOut() {
 	hl.hovered = false
 	if changed {
 		hl.BaseWidget.Refresh()
+		for _, s := range hl.siblings {
+			s.setHovered(false)
+		}
+	}
+}
+
+// setHovered updates the hovered state without propagating to siblings, to avoid recursion.
+func (hl *Hyperlink) setHovered(hovered bool) {
+	if hl.hovered == hovered {
+		return
 	}
+	hl.hovered = hovered
+	hl.BaseWidget.Refresh()
 }
 
 func (hl *Hyperlink) focusWidth() float32 {

widget/richtext.go

@@ -700,7 +700,7 @@ func (r *textRenderer) Refresh() {
 	for _, bound := range bounds {
 		for i, seg := range bound.segments {
 			_, isText := seg.(*TextSegment)
-			_, isHyperlink := seg.(*HyperlinkSegment)
+			hlSeg, isHyperlink := seg.(*HyperlinkSegment)
 			if !isText && !isHyperlink {
 				obj := r.obj.cachedSegmentVisual(seg, 0)
 				seg.Update(obj)
@@ -739,8 +739,10 @@ func (r *textRenderer) Refresh() {
 			if isText {
 				obj.(*canvas.Text).Text = txt
 			} else if isHyperlink {
-				obj.(*fyne.Container).Objects[0].(*Hyperlink).Text = txt
-				obj.(*fyne.Container).Objects[0].(*Hyperlink).Refresh()
+				hl := obj.(*fyne.Container).Objects[0].(*Hyperlink)
+				hl.Text = txt
+				r.associateSiblings(hl, hlSeg, reuse)
+				hl.Refresh()
 			}
 			objs = append(objs, obj)
 		}
@@ -763,6 +765,15 @@ func (r *textRenderer) Refresh() {
 	r.obj.cleanVisualCache()
 }
 
+func (r *textRenderer) associateSiblings(hl *Hyperlink, hlSeg *HyperlinkSegment, reuse int) {
+	hl.siblings = hl.siblings[:0]
+	for prev := 0; prev < reuse; prev++ {
+		prevHL := r.obj.cachedSegmentVisual(hlSeg, prev).(*fyne.Container).Objects[0].(*Hyperlink)
+		prevHL.siblings = append(prevHL.siblings, hl)
+		hl.siblings = append(hl.siblings, prevHL)
+	}
+}
+
 func (r *textRenderer) layoutRow(texts []fyne.CanvasObject, align fyne.TextAlign, xPos, yPos, lineWidth float32) (float32, float32) {
 	initialX := xPos
 	if len(texts) == 1 {

widget/richtext_objects_test.go

@@ -1,13 +1,15 @@
 package widget
 
 import (
+	"net/url"
 	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
 
 	"fyne.io/fyne/v2"
 	"fyne.io/fyne/v2/canvas"
+	"fyne.io/fyne/v2/driver/desktop"
 	"fyne.io/fyne/v2/storage"
 	"fyne.io/fyne/v2/test"
 )
@@ -71,6 +73,41 @@ func TestRichText_OrderedList(t *testing.T) {
 	assert.Equal(t, "Two", texts[3].(*canvas.Text).Text)
 }
 
+func TestRichText_HyperLink_WrappedHoverSynced(t *testing.T) {
+	u, _ := url.Parse("https://fyne.io")
+	seg := &HyperlinkSegment{Text: "this is a long hyperlink that wraps", URL: u}
+	rt := NewRichText(seg)
+	rt.Wrapping = fyne.TextWrapWord
+	// Render narrow enough to force wrapping into at least 2 rows.
+	rt.Resize(fyne.NewSize(100, 200))
+
+	objs := test.TempWidgetRenderer(t, rt).Objects()
+	// Collect all Hyperlink visuals for the segment (one per wrapped row).
+	var links []*Hyperlink
+	for _, obj := range objs {
+		if c, ok := obj.(*fyne.Container); ok {
+			if hl, ok := c.Objects[0].(*Hyperlink); ok {
+				links = append(links, hl)
+			}
+		}
+	}
+	assert.GreaterOrEqual(t, len(links), 2, "expected hyperlink to wrap into multiple segments")
+
+	// MouseIn on the first segment — choose a position inside the hyperlink's bounds.
+	center := fyne.NewPos(links[0].Size().Width/2, links[0].Size().Height/2)
+	inside := &desktop.MouseEvent{PointEvent: fyne.PointEvent{Position: center}}
+	links[0].MouseIn(inside)
+	for i, hl := range links {
+		assert.True(t, hl.hovered, "expected link[%d] to be hovered after MouseIn on link[0]", i)
+	}
+
+	// MouseOut on the first segment — all siblings should be unhovered too.
+	links[0].MouseOut()
+	for i, hl := range links {
+		assert.False(t, hl.hovered, "expected link[%d] to be unhovered after MouseOut on link[0]", i)
+	}
+}
+
 func TestRichText_OrderedListDifferentIndex(t *testing.T) {
 	for name, tt := range map[string]struct {
 		index        int