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 200215 - Disable functions based on selection in the message list
Disable functions based on selection in the message list
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
unspecified
Other All
: Normal normal
: ---
Assigned To: Peter Williams
Evolution QA team
: 202095 202758 203309 205085 205086 205088 207180 (view as bug list)
Depends on: 205353 206795
Blocks:
 
 
Reported: 2000-06-01 13:22 UTC by Jeffrey Stedfast
Modified: 2005-11-15 02:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch created with cvs diff (8.12 KB, patch)
2000-10-29 13:49 UTC, Jos Dehaes
none Details | Review
Candidate patch (10.63 KB, patch)
2001-08-09 20:37 UTC, Peter Williams
none Details | Review

Description Jeffrey Stedfast 2000-06-01 13:22:37 UTC
* Disable Reply, Reply-to-All, Forward, Delete, and Print when no messages are
selected or available.

* When multiple messages are selected:
A) Disable Reply and Reply-to-All when multiple messages are selected.
B) Print should queue all the messages for printing
C) Forward should attach all selected messages
D) Delete should mark selected messages as deleted
Comment 1 Jos Dehaes 2000-08-26 11:43:05 UTC
- added placeholder for signal selection_changed signal, waiting for http://bugzilla.ximian.com/show_bug.cgi?id=196
- disable reply, reply-to-all in popupmenu works
- forward all seemed to be allready working...
- same for delete all
- print all is beyond my (current) capabilities :-(
Comment 2 Jos Dehaes 2000-10-29 13:49:50 UTC
Created attachment 39876 [details] [review]
patch created with cvs diff
Comment 3 Jos Dehaes 2000-10-29 13:54:48 UTC
done with most of it.
I'm wondering whether some more items in the message menu should be disabled, if
no message is selected, since they don't seem to work anyway then...

!!print should queue all is not done!! I feel for this I would need to change
too much, I'd like that done by someone with more insight in the code...

maybe this should go in a new bug??? I'll just set resolution to fixed for now...
Comment 4 Dan Winship 2001-03-30 21:14:11 UTC
This code never acatually got committed. And it's probably way out of date
now, so it will need to be updated.
Comment 5 Dan Winship 2001-04-03 15:23:25 UTC
*** bug 202095 has been marked as a duplicate of this bug. ***
Comment 6 Dan Winship 2001-05-11 15:52:38 UTC
*** bug 202758 has been marked as a duplicate of this bug. ***
Comment 7 Dan Winship 2001-06-12 18:05:07 UTC
*** bug 203309 has been marked as a duplicate of this bug. ***
Comment 8 Luis Villa 2001-06-26 18:19:34 UTC
I apologize for the spam; re-setting all target milestones to 'future'
in preparation for evolution 1.0. If you have any questions, please feel
free to write louie@ximian.com.
Comment 9 Dan Winship 2001-07-23 13:44:03 UTC
*** bug 205085 has been marked as a duplicate of this bug. ***
Comment 10 Dan Winship 2001-07-23 13:44:38 UTC
(5085 has a possibly comprehensive list of relevant
menu commands)
Comment 11 Dan Winship 2001-07-23 13:45:06 UTC
*** bug 205086 has been marked as a duplicate of this bug. ***
Comment 12 Dan Winship 2001-07-23 13:45:40 UTC
*** bug 205088 has been marked as a duplicate of this bug. ***
Comment 13 Dan Winship 2001-07-23 13:46:04 UTC
(these others also have good info)
Comment 14 Luis Villa 2001-07-24 03:03:36 UTC
Adjusting the triage; if this is going to be fixed, it needs to be
fixed before the UI freeze.
Comment 15 Peter Williams 2001-07-26 16:03:34 UTC
The patch attached way at the top of this bug is mostly good, but
unfortunately the code now uses e_tree. which doesn't implement
get_selection_count. Fortunately this has already been filed as bug
205353 so I'll make this depend on that.
Comment 16 Peter Williams 2001-08-09 17:25:39 UTC
I'll work on updating the patch.
Comment 17 Peter Williams 2001-08-09 20:02:22 UTC
Ok, this is fairly close to completion, but I don't want to commit
until 6795 is fixed.
Comment 18 Peter Williams 2001-08-09 20:37:04 UTC
Created attachment 40137 [details] [review]
Candidate patch
Comment 19 Peter Williams 2001-08-09 20:37:45 UTC
The attached patch works well for me, but again, I don't want to
commit until 6795 is fixed.
Comment 20 Peter Williams 2001-08-13 17:47:15 UTC
*** bug 207180 has been marked as a duplicate of this bug. ***
Comment 21 Peter Williams 2001-08-14 15:27:08 UTC
Checked in, finally.
Comment 22 Ben Liblit 2001-09-19 09:06:14 UTC
The File -> Print menu item, File -> Print Preview menu item, and
Print toolbar button remain enabled even when no message is selected. 
But if there's nothing to print, they should be disabled.

Consistently repeatable as follows:

  1. Open a view of any mail folder.
  2. If a message is selected, use <Control>-<Mouse 1> to deselect it.
  3. Observe that while numerous other items have been (correctly)
     disabled, the printing controls remain available.

One might argue that the current behavior is reasonable if we take
printing to apply to the message currently being displayed in the
preview pane.  However, if the preview pane is closed, then how can
the user possibly know which message would be printed?

At the very least, these controls should be disabled when there is no
selected message and the preview pane is closed.

Note that bug 210346 seems at least somewhat related, though not quite
a duplicate.  Anyone attempting to fix this bug should at least be
aware of bug 210346 as well.
Comment 23 Jeffrey Stedfast 2001-10-28 10:40:50 UTC
fixed as far as I can tell