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 512623 - Message list issues on trunk
Message list issues on trunk
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.22.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 272342 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-01-28 17:59 UTC by Srinivasa Ragavan
Modified: 2013-09-13 00:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed evo patch (2.66 KB, patch)
2008-01-28 19:10 UTC, Milan Crha
needs-work Details | Review
proposed evo patch ][ (2.76 KB, patch)
2008-01-29 10:55 UTC, Milan Crha
committed Details | Review

Description Srinivasa Ragavan 2008-01-28 17:59:37 UTC
1. Shift Left/Right on message threads doesn't collapse/expand. Neither +/-
2. Lots of messages tend not to reflect the actual message status.
For Example, if a message (expanded) is marked as read, the bold status remains. This happens for marking mails as Important also. The view/list doesn't reflect it. But when you right click, it shows the right status in the menu items. This doesn't happen on all mails all times. But at time. I have seen that when I read a mail it remains bold independent of click on the unread icon, Ctrk+k everything but on restart it works all fine.
Comment 1 Milan Crha 2008-01-28 19:10:49 UTC
Created attachment 103908 [details] [review]
proposed evo patch

for evolution;

as srinidhi said on IRC, it added GDK_MODE2_MASK to key->state when Num Lock is on, but because you said it didn't help you to turn off it, then I added all GDK_MODEx_MASK-s, just in case. Can you check, please?
Comment 2 Srinivasa Ragavan 2008-01-29 07:35:54 UTC
I think it is very close now. It should work only on Shift Left/Right. It now works with out that also. Also +/- should work. In my keyboard, it has [- _] [= +]
- [WITHOUTSHIFT WITHSHIFT]
It works on -/= but it should be -/+ 
Comment 3 André Klapper 2008-01-29 10:43:17 UTC
i wonder if this will have impact on bug 272342
Comment 4 Milan Crha 2008-01-29 10:50:59 UTC
Right, it should fix it (is a duplicate of such old bug) ;)
Comment 5 Milan Crha 2008-01-29 10:55:56 UTC
Created attachment 103947 [details] [review]
proposed evo patch ][

for evolution;

another attempt. Works for me, but you can think it's wrong idea to do it in this way. Anyway, it works similar to folder tree.
Comment 6 Srinivasa Ragavan 2008-01-30 18:04:30 UTC
Milan it works all fine. Just one small glitch.

It currently accepts '_' and not '-'. Which means.. it works for 'SHIFT'AND'-' means '_' but it should be simplly '-'. It works fine for 'SHIFT' AND'=' means '+'

Otherwise, looks great. 
Comment 7 Milan Crha 2008-01-30 18:49:42 UTC
I changed it because my keyboard returns underscore, does yours return something else?
Comment 8 Srinivasa Ragavan 2008-01-31 04:28:12 UTC
oh!. Milan, shouldn't we just listen to +/- and ignore any type modifiers? I'm lot confused here. I'm happy that it started working for me again.
Comment 9 Milan Crha 2008-01-31 17:25:00 UTC
So, it depends, if you are fine with that, then probably we can, but it will be either ('+' and '_') or ('=' and '-'), the first for the variant with Shift, the second for variant without Shift. Or both of them, it depends. But notice the '+' and '-' on numeric keyboard would work even without a modifier.

I also think the returned values are correct for English keyboards respecting the shift button. So maybe this is not such a big workaround as I thought before. If you want to drop the modifiers check, then it will work regardless of that, so say Ctrl+Shift+'_' will does same as Shift+'_', which may be confusing.
Comment 10 Srinivasa Ragavan 2008-02-01 07:41:08 UTC
Ah!, I'm confused. Lets commit Left/Right first. I would say that then just +/- and don't bother about anything else. 
Comment 11 Milan Crha 2008-02-01 08:46:14 UTC
Committed to trunk. Committed revision 34944.
Comment 12 Milan Crha 2008-02-01 08:48:48 UTC
*** Bug 272342 has been marked as a duplicate of this bug. ***