Opened 16 years ago
Closed 12 years ago
#2446 closed enhancement (fixed)
Add Cc: text field to mails shown when it's not empty (easy)
Reported by: | axeld | Owned by: | czeidler |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/Mail | Version: | R1/pre-alpha1 |
Keywords: | gsoc2011 | Cc: | jalopeura@… |
Blocked By: | Blocking: | ||
Platform: | All |
Description
It would be nice to see the other recipients of a mail automatically, without needing to turn on the "show header" option.
To save space, the field should only be visible in case there actually are any Cc addressees.
Attachments (2)
Change History (13)
comment:1 by , 15 years ago
comment:2 by , 15 years ago
I have attached my work on this ticket. I have also tested and can confirm it to be working.
comment:3 by , 14 years ago
patch: | 0 → 1 |
---|
comment:4 by , 14 years ago
I have added a patch to implement this. I do not know whether LoadMessage can be called multiple times, so I assumed it could be and coded to that (although I was unable to test whether loading multiple messages would work).
NOTE: I was unable to get to previous patch to work. Even applying the changes manually did not work, as it relied on already knowing, at the time the header controls are created, whether the CC field will be required. (Perhaps this was the case when the patch was created, but is no longer the case now.)
comment:5 by , 14 years ago
Accidentally hit the back button on my browser and it resubmiited the patch. The last two patches are the same.
comment:6 by , 14 years ago
Keywords: | gsoc2011 added |
---|
comment:7 by , 14 years ago
Cc: | added |
---|
comment:8 by , 13 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Let's see if Clemens is interested.
comment:9 by , 12 years ago
To note, the patch attachment:maildiff.txt applies cleanly on current the master (hrev45296)
comment:10 by , 12 years ago
If it applies and works, I say commit it. The code looks decent enough, beyond the fact that Mail has a terrible old-school non-layout-aware GUI.
comment:11 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Applied in hrev45418. Thanks for the patch!
Trying out a patch for this issue today. Will attach patch if it is good.