Closed Bug 1176694 Opened 9 years ago Closed 2 years ago

Pressing the "Not Junk" in the new window does not update standalone message window, but updates in the main window.

Categories

(Thunderbird :: Message Reader UI, defect)

38 Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
101 Branch

People

(Reporter: ilkostikov, Assigned: TbSync)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/42.0.2311.152 YaBrowser/15.6.2311.3927 Safari/537.36

Steps to reproduce:

1. Select a message in the message list.
2. In the context menu, select "Open message in a New Window."
3. Press the "Not Junk."


Actual results:

1. Message reloaded in the main window (three pane window). 
2. Notification of junk has not hidden.


Expected results:

1. Message reloaded in the new window (one pane window). 
2. Notification of junk has hidden.
Did version 31 give different results?
Component: Untriaged → Message Reader UI
Flags: needinfo?(ilkostikov)
Summary: Pressing the "Not Junk" in the new window does not update letter in this window, but updates in the main window. → Pressing the "Not Junk" in the new window does not update standalone message window, but updates in the main window.
(In reply to Wayne Mery (:wsmwk, use Needinfo for questions) from comment #1)
> Did version 31 give different results?

In version 31.6.0 result is the same as in version 38. 

My research (maybe it will help):
When user press "not junk" in a new window:
1) Called the function "JunkSelectedMessages(setAsJunk)" from junkCommands.js. 
2) Changing the status of the message is listened in the folderListner ("OnItemEvent: function(folder, event)"). 
3) folderListner calls function "HandleJunkStatusChanged(folder)".
4) "HandleJunkStatusChanged(folder)" calls function "ReloadMessage()".

But folderListner defined in msgMail3PaneWindow.js and only works for main window (3 pane window).
Flags: needinfo?(ilkostikov)
some good diagnosis in comment 2
Do you click the "Not junk" in the new window? Or in the original message preview in 3-pane window?

If I click it in the new window, the message is actually moved from Junk folder to Inbox AND the new window is closed.

Do you see something else?
Ilya?

(In reply to :aceman from comment #4)
> Do you click the "Not junk" in the new window? Or in the original message
> preview in 3-pane window?
> 
> If I click it in the new window, the message is actually moved from Junk
> folder to Inbox AND the new window is closed.
> 
> Do you see something else?
Flags: needinfo?(ilkostikov)
38.6 has same behavior
Flags: needinfo?(ilkostikov)
Assignee: nobody → john
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #9271217 - Attachment description: Bug 1176694 - Reload message in all windows, when the junk status changes and reload is required. r=mkmelin → WIP: Bug 1176694 - Reload message in all windows, when the junk status changes and reload is required. r=mkmelin
Attachment #9271217 - Attachment description: WIP: Bug 1176694 - Reload message in all windows, when the junk status changes and reload is required. r=mkmelin → Bug 1176694 - Reload message in all windows, when the junk status changes and reload is required. r=mkmelin

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/6f1b7165ec11
Reload message in all windows, when the junk status changes and reload is required. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: