After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 354650 - Reply with blank line selected truncates text
Reply with blank line selected truncates text
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.8.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 354976 358145 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-09-06 16:34 UTC by Daniel Gryniewicz
Modified: 2013-09-10 14:04 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Check for blank selection when replying (606 bytes, patch)
2006-09-06 19:38 UTC, Daniel Gryniewicz
committed Details | Review

Description Daniel Gryniewicz 2006-09-06 16:34:26 UTC
Please describe the problem:
When you reply to a message with a blank line selected (double-click on a blank line, for example), the reply comes up with nothing but the attribute line.  This is arguably what the user wanted, but it most often happens by accident, and (since selected blank lines don't show up highlighted), there's not feedback as to what's causing the problem.

Talking this over on IRC, several of us came to the conclusion that this is probably not the correct behavor.  Attached is a patch that fixes this, by ignoring empty selections.

Steps to reproduce:
1. In the preview mode, double click on a blank line
2. reply to the message



Actual results:
Reply is empty except for attribution line.

Expected results:
Reply contains full message text.

Does this happen every time?
Yes.

Other information:
Comment 1 Daniel Gryniewicz 2006-09-06 19:38:58 UTC
Created attachment 72333 [details] [review]
Check for blank selection when replying

Check the selection to make sure it's not empty before using it in the reply.
Comment 2 André Klapper 2006-09-11 12:45:48 UTC
*** Bug 354976 has been marked as a duplicate of this bug. ***
Comment 3 André Klapper 2006-09-11 12:46:30 UTC
patch! :-)
Comment 4 André Klapper 2006-09-29 13:17:25 UTC
*** Bug 358145 has been marked as a duplicate of this bug. ***
Comment 5 Veerapuram Varadhan 2006-09-29 14:32:02 UTC
Patch looks good.  Please commit with a changelog.
Comment 6 Daniel Gryniewicz 2006-09-29 16:17:48 UTC
Just for the record: I don't have gnome commit (yet?), so someone else will have to commit this...
Comment 7 André Klapper 2006-09-29 18:08:30 UTC
committed to CVS HEAD ans gnome-2-14 branch on behalf of daniel:
http://cvs.gnome.org/viewcvs/evolution/mail/em-folder-view.c?r1=1.161&r2=1.162
http://cvs.gnome.org/viewcvs/evolution/mail/em-folder-view.c?r1=1.145.2.1&r2=1.145.2.2

patch will be included in evolution 2.8.1. thanks a lot!