Closed Bug 368239 Opened 18 years ago Closed 18 years ago

forward/back in stand-alone message window doesn't cross folders

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

References

(Blocks 1 open bug)

Details

(Keywords: fixed1.8.1.2)

Attachments

(1 file)

Attached patch proposed fixSplinter Review
There are a couple issues with forward/back crossing folders using the stand-alone message window. Also, you need to do next unread twice to get it to cross folders w/ the standalone message window. The patch I'm about to attach fixes both problems, as well as an issue where the drop down menu for the back/forward buttons doesn't notice when you change folders so it shows the wrong folder in the menu, in that case.
Comment on attachment 252839 [details] [diff] [review]
proposed fix

this fixes the folder name n the drop down, when you've crossed folders:

+  msgWindow.openFolder = msgfolder;
+  

this fixes the need to do next unread twice to cross folders:

+      // accessing the db causes the folder loaded notification to get sent
+      // for local folders.
+      var db = msgfolder.getMsgDatabase(msgWindow);

and I added a method LoadNavigatedToMessage for messageWindow.js and msgMail3PaneWindow.js to differentiate the way they need to handle loading messages after back/forward navigation.
Attachment #252839 - Flags: superreview?(mscott)
Karsten, you'll probably be interested in this patch for SM if you have forward/back buttons.
Attachment #252839 - Flags: superreview?(mscott) → superreview+
fixed on trunk and branch
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1.2
Resolution: --- → FIXED
(In reply to comment #2)
> Karsten, you'll probably be interested in this patch for SM if you have
> forward/back buttons.

Moved to bug 399787.
Blocks: 399787
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: