Opened 3 years ago

Closed 22 months ago

#1487128 closed Feature Patches (fixed)

Mail subject is cut

Reported by: tomamplius Owned by:
Priority: 5 Milestone: 0.7-beta
Component: User Interface Version: git-master
Severity: normal Keywords: subject
Cc: tomamplius@…

Description

You can show in the screen shot the Mail subject is cut. It's possible to add 'title' property with subject

svn diff

Index: roundcubemail/program/js/app.js
===================================================================
--- roundcubemail/program/js/app.js (révision 4231)
+++ roundcubemail/program/js/app.js (copie de travail)
@@ -1688,7 +1688,7 @@

var action = flags.mbox == this.env.drafts_mailbox ? 'compose' : 'show';
var uid_param = flags.mbox == this.env.drafts_mailbox ? '_draft_uid' : '_uid';
cols.subject = '<a href="./?_task=mail&_action='+action+'&_mbox='+urlencode(flags.mbox)+'&'+uid_param+'='+uid+'"'+

  • ' onclick="return rcube_event.cancel(event)">'+cols.subject+'</a>';

+ ' title="'+cols.subject+'" onclick="return rcube_event.cancel(event)">'+cols.subject+'</a>';

}


add each submitted col

Attachments (4)

a-with-title.png (52.7 KB) - added by tomamplius 3 years ago.
patch.diff (744 bytes) - added by tomamplius 3 years ago.
test_get_td_witdh.patch (1.0 KB) - added by tomamplius 3 years ago.
full_subject_if_needed.patch (2.1 KB) - added by tomamplius 3 years ago.

Download all attachments as: .zip

Change History (12)

Changed 3 years ago by tomamplius

Changed 3 years ago by tomamplius

comment:1 Changed 3 years ago by tomamplius

You can show in the screen shot the Mail subject is cut. It's possible to add 'title' property with subject

comment:2 Changed 3 years ago by alec

The title useless with short subjects. Maybe we could add some scripting to display it only when needed.

comment:3 Changed 3 years ago by tomamplius

if td width is not defined, javascript document.getElementById(<td_id>).width return empty. I don't know how get size of subject column. (see test_get_td_witdh.patch)

Changed 3 years ago by tomamplius

Changed 3 years ago by tomamplius

comment:4 Changed 3 years ago by tomamplius

  • Keywords subject added
  • Milestone changed from later to 0.5-stable
  • Severity changed from minor to normal

It's work now

Version 0, edited 3 years ago by tomamplius (next)

comment:5 Changed 2 years ago by tomamplius

  • Cc tomamplius@… added

important to use roudcube on netbook 9 or 10 pouce. Because many subject are cut.

Last edited 2 years ago by tomamplius (previous) (diff)

comment:6 Changed 2 years ago by thomasb

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [bc374503]

comment:7 Changed 2 years ago by alec

  • Resolution fixed deleted
  • Status changed from closed to reopened

It doesn't work with IE. We could add span for subject in IE, but then text-overflow doesn't work.

comment:8 Changed 22 months ago by alec

  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed in [065d7025].

Note: See TracTickets for help on using tickets.