Ticket #8555: ital_actions.patch

src/apps/aboutsystem/HyperTextView.cpp

@@ -40 +40 @@
 }
 
 
+void
+HyperTextAction::MouseEnter(HyperTextView* view, BPoint where,
+    int32 startOffset, int32 endOffset, BMessage* message)
+{
+    BFont font(be_plain_font);
+    font.SetFace(B_ITALIC_FACE);
+    view->SetFontAndColor(startOffset, endOffset, &font);
+}
+
+
+void
+HyperTextAction::MouseLeave(HyperTextView* view, BPoint where,
+    int32 startOffset, int32 endOffset, BMessage* message)
+{
+    BFont font(be_plain_font);
+    font.SetFace(B_REGULAR_FACE);
+    view->SetFontAndColor(startOffset, endOffset, &font);
+}       
+
+
 // #pragma mark - HyperTextView
 
 
@@ -114 +134 @@
 HyperTextView::MouseDown(BPoint where)
 {
     // We eat all mouse button events.
-
-    BTextView::MouseDown(where);
 }
 
 
@@ -127 +145 @@
     HyperTextAction* action = _ActionAt(where);
     if (action != NULL)
         action->Clicked(this, where, message);
-
-    BTextView::MouseUp(where);
 }
 
 
@@ -139 +155 @@
     BMessage* message = Window()->CurrentMessage();
 
     uint32 buttons;
-    HyperTextAction* action;
+    const ActionInfo* info = _ActionInfoAt(where);
     if (message->FindInt32("buttons", (int32*)&buttons) == B_OK
-        && buttons == 0 && (action = _ActionAt(where)) != NULL) {
+        && buttons == 0 && info != NULL) {
+        HyperTextAction* action = info->action;
         action->MouseOver(this, where, message);
+        if (fActiveInfo == NULL) {
+            fActiveInfo = info;
+            action->MouseEnter(this, where, info->startOffset, info->endOffset,
+                message);
+        } else if (fActiveInfo != info) {
+            // This is the scenario where mouse moves from one action directly
+            // into another.
+            action->MouseEnter(this, where, info->startOffset, info->endOffset,
+                message);
+            fActiveInfo->action->MouseLeave(this, where, fActiveInfo->startOffset,
+                fActiveInfo->endOffset, message);
+            fActiveInfo = info; 
+        }
         return;
+    } else if (fActiveInfo != NULL) {
+        fActiveInfo->action->MouseLeave(this, where, fActiveInfo->startOffset,
+            fActiveInfo->endOffset, message);
+        fActiveInfo = NULL;
     }
 
     BTextView::MouseMoved(where, transit, dragMessage);
@@ -193 +227 @@
 HyperTextAction*
 HyperTextView::_ActionAt(const BPoint& where) const
 {
+    const ActionInfo* info = _ActionInfoAt(where);
+    if (info != NULL) {
+        return info->action;
+    }
+
+    return NULL;
+}
+
+
+const HyperTextView::ActionInfo*
+HyperTextView::_ActionInfoAt(const BPoint& where) const
+{
     int32 offset = OffsetAt(where);
 
     ActionInfo pointer(offset, offset + 1, NULL);
@@ -204 +250 @@
         BRegion textRegion;
         GetTextRegion(action->startOffset, action->endOffset, &textRegion);
         if (textRegion.Contains(where))
-            return action->action;
+            return action;
     }
 
     return NULL;
 }
 
+

src/apps/aboutsystem/HyperTextView.h

@@ -24 +24 @@
                                     BMessage* message);
     virtual void                Clicked(HyperTextView* view, BPoint where,
                                     BMessage* message);
+    virtual void                MouseEnter(HyperTextView* view, BPoint where,
+                                    int32 startOffset, int32 endOffset,
+                                    BMessage* message);
+    virtual void                MouseLeave(HyperTextView* view, BPoint where,
+                                    int32 startOffset, int32 endOffset,
+                                    BMessage* message);
 };
 
 
@@ -59 +65 @@
             class ActionInfoList;
 
             ActionInfoList*     fActionInfos;
+            
+            const ActionInfo*   _ActionInfoAt(const BPoint& where) const;
+            
+            const ActionInfo*   fActiveInfo;
 };