mudlet-makers team mailing list archive

[Vadim Peretokin <vperetokin@xxxxxxxxx>]

I can confirm that this patch works.

Have you noticed that it extends beyond a single character, though?
Attached a video demo.

** Attachment added: "out-5.ogv"
   https://bugs.launchpad.net/mudlet/+bug/734455/+attachment/1908974/+files/out-5.ogv

-- 
You received this bug notification because you are a member of Mudlet
Makers, which is subscribed to Mudlet.
https://bugs.launchpad.net/bugs/734455

Title:
  mouse cursor does not change to pointing hand correctly

Status in Mudlet the MUD client:
  New

Bug description:
  Mudlet 2.0-dev built: January 19, 2011
  running Kubuntu 10.10

  in TTextEdit.cpp

  in function 
  void TTextEdit::mouseMoveEvent( QMouseEvent * event )

  the line (approx 892)
      int x = event->x() / mFontWidth -1;

  should be
     int x = event->x() / mFontWidth ;

  on a link, with the -1, the mouse pointer changes to the pointing hand 1 character later and extends 1 character beyond 
  the link

  in void TTextEdit::mousePressEvent( QMouseEvent * event ) the same line reads 
  int x = event->x() / mFontWidth; which is correct