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 655568 - Cursor lost after Collapse all threads called
Cursor lost after Collapse all threads called
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.32.x (obsolete)
Other Linux
: Normal minor
: ---
Assigned To: Milan Crha
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2011-07-29 10:39 UTC by Patryk Benderz
Modified: 2011-09-07 06:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
evo patch (1.09 KB, patch)
2011-09-05 17:06 UTC, Milan Crha
committed Details | Review

Description Patryk Benderz 2011-07-29 10:39:17 UTC
Overview: Summary is pretty self explanatory, but as you wish... Currently selected message in threaded view is not highlighted, when thread is collapsed using Shift+b shortcut. 

Steps to Reproduce:
1.Switch to threaded view
2.Click/Select a message deep in a thread
3.Press ^B to collapse all threads in current folder
4.Observe as selection vanishes :)

Actual results:
Top message of current thread is not highlighted.

Expected results:
I expect Evolution to highlight top message of current thread. On a thread example like below:
MSG1
|-MSG2
  |-MSG3
when MSG3 is selected and it's contents are listed in mail preview, during collapse of the thread, color of MSG1 highlight should be different than normal selection, to avoid confusion that MSG3 is displayed in preview, but MSG1 is highlighted.

Reproducible: Always.

Other information:
-When you take focus out of Evolution to another app, and focus back, Evolution selects top message in current folder.
-Let me know if you need additional debug info.
-This bug is NOT a duplicate of:
#514359
#589249
-I wasn't able to find any duplicates.

Regards, LeadMan
Comment 1 Milan Crha 2011-09-05 17:06:34 UTC
Created attachment 195719 [details] [review]
evo patch

for evolution;

Confirming, I can reproduce it too. This patch fixes the issue by ensuring that restored selected node is currently visible too.
Comment 2 Milan Crha 2011-09-05 17:08:03 UTC
Created commit 4035b6f in evo master (3.1.92+)
Comment 3 Patryk Benderz 2011-09-06 15:29:14 UTC
[cut]
> Confirming, I can reproduce it too. This patch fixes the issue by ensuring that
> restored selected node is currently visible too.
Hi Milan,
good to hear that. What is estimated time needed for this patch to reach
stable version of Evolution?
Comment 4 Milan Crha 2011-09-07 06:46:10 UTC
It's part of 3.2.0 stable release, which will be done approximately at the end of this month.