Dummy link in subject cell on messages list using IE8

[alec]

There is a dummy (invisible) link at the end of subject text. Move cursor to there and you'll see it is changing from text to pointer style. Clicking it changes message read/unread state. It happens only when Read/Unread? column is disabled. Doesn't happen in IE6 and IE7.

[thomasgraham]

Hi Alec, I've tried to reproduce this bug in IE8 on Windows XP but it doesn't seem to happen for me. Could you give more details?

What I do get (which seems a little counter intuitive) is that when the column is removed, the point immediately to the left of the message subject becomes the mark as read/unread link.

Strangely this field stops being active and is only displayed as a point when the read/unread column is used so you get the dot (which when the read/unread column isn't there would indicate unread) all the time, regardless of whether the message is unread or not. This behaviour is consistent across platforms/browsers (Mac/Windows? Firefox, Chrome, IE8).

[thomasgraham]

I re-read, the ticket does seems to relate to the code in trunk, I can't replicate this problem.

[alec]

The issue is there since ever I think. It is in 0.4 and in current svn-trunk. I'm talking about an invisible link in subject column at the end of the text. You can find it by moving mouse cursor over there. The cursor will change.

[thomasgraham]

Ah, got you. I see what you mean, thought you meant at the end of the subject text column.... I'll take a look, hopefully can't be too complicated to fix, right?

[alec]

The issue disapear when IE8 compatybility view is enabled.

[alec]

Here's the fix, however I'd like to see css solution for this issue.

--- app.js      (wersja 5201)
+++ app.js      (kopia robocza)
@@ -1772,8 +1772,10 @@
       else if (c == 'threads')
         html = expando;
       else if (c == 'subject') {
-        if (bw.ie)
+        if (bw.ie) {
           col.onmouseover = function() { rcube_webmail.long_subject_title_ie(this, message.depth+1); };
+          tree = '<span style="width:0px"></span>' + tree;
+        }
         html = tree + cols[c];
       }
       else if (c == 'priority') {

[alec]

The issue also exists in Larry.

[thomasb]

I have no idea how to solve this with CSS. Either we take your solution or ignore the issue.

[alec]

Fixed in [d960a01b].