Closed Bug 531397 Opened 15 years ago Closed 12 years ago

switching Layout modes breaks feed content display entirely until restart

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
All
defect
Not set
normal

Tracking

(blocking-thunderbird3.0 -, thunderbird3.0 wanted, thunderbird17 fixed)

RESOLVED FIXED
Thunderbird 18.0
Tracking Status
blocking-thunderbird3.0 --- -
thunderbird3.0 --- wanted
thunderbird17 --- fixed

People

(Reporter: matafagafo, Unassigned)

References

(Depends on 1 open bug)

Details

(Whiteboard: [fixed by bug 596234])

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2b4) Gecko/20091124 Firefox/3.6b4 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.5) Gecko/20091121 Thunderbird/3.0

If I set my layout to Wide View, the content of RSS feeds don't are showed.

Reproducible: Always

Steps to Reproduce:
1.With layout Classic
2.Get new RSS feeds
3.select the feed to see the content
3.The content are showed
4.Change the Layout to Wide
5.Select the feed again
6.The content are missing 
Actual Results:  
The content of RSS feeds aren't showed if You are using Wide View Layout

Expected Results:  
The contents of the RSS feed properly showed when the user use the Wide View Layout

Happens in TB 3.0 RC1
Flags: blocking-thunderbird3?
Version: unspecified → 3.0
confirmed on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.5) Gecko/20091121 Thunderbird/3.0
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows Vista → All
This bug actually happens with any change in layout, not just Classic -> Wide.  Fortunately, restarting puts Thunderbird back into a working state.
This is a rotten bug, but we can live with relnoting this for 3.0 since all that's required is a restart.  It does, however, block 3.0.1.
blocking-thunderbird3.0: --- → .1
Flags: blocking-thunderbird3? → blocking-thunderbird3-
Keywords: relnote
Summary: RSS feed summary ou Web content missing if Wide View Layout is used → switching Layout modes breaks feed content display entirely until restart
For the records, to me, the restart only works when I'm in Classic View. If I set the Wide View and restart TB, the RSS contents don't are showed.
With Wide View the content never are showed.
(In reply to comment #3)
> For the records, to me, the restart only works when I'm in Classic View. If I
> set the Wide View and restart TB, the RSS contents don't are showed.
> With Wide View the content never are showed.

I bet you have Lightning installed. I've just done a test with and without Lightning enabled and without Lightning enabled I can restart after switching layouts and the wide view works fine.

I've raised bug 532071 on the Lightning specific issue.
Restart works for me on linux (classic -> vertical caused the issue)
I've been taking a look at the code. This is roughly what I've worked out so far:

- When switching messages, feeds items, etc, in normal conditions messageHeaderSink (in msgHdrViewOverlay.js) gets called into with at least the following functions: 

onStartHeaders, onEndMsgHeaders, onEndMsgDownload

- after we switch layouts, the onEndMsgDownload callback is lost. This in turn appears to not call some of the code that seems vital for display and set up of feeds (I'm not saying not calling that function is totally responsible, but it does break some things).

From what I can tell onEndMsgDownload is only called from nsMsgStatusFeedback::OnStateChange, which implies that after we've changed layouts, nsMsgStatusFeedback is no longer being called from when urls are processed.

AFAICT we're not dropping and re-initialising msgWindow (which could cause the status feedback to drop as well), so at the moment why nsMsgStatusFeedback::OnStateChange isn't getting called I don't know.
In UpdateMailPaneConfig in msgMail3PaneWindow we do call messenger.setWindow(null, null) then we re-establish it...

There's also XBL direct poking due to the DOM node transplantation.

I blame that.
Bug 492216.  also, a Tb3 checkin many months ago caused the MoreLayoutsforThunderbird extension to break, under which dynamic layout changes were no issue for rss and layout changes specifically required no restart.  but i can't figure out how to fix it..
blocking-thunderbird3.0: .1+ → needed
blocking-thunderbird3.0: needed → -
Depends on: 492216
(In reply to alta88 from comment #8)
> Bug 492216.  also, a Tb3 checkin many months ago caused the
> MoreLayoutsforThunderbird extension to break, under which dynamic layout
> changes were no issue for rss and layout changes specifically required no
> restart.  but i can't figure out how to fix it..

Alta and you think you could now ?
hehe.  yes, i finally did figure it out.  i blame the tabs implementation, it misses the fact that each tab needs to set the msgWindow and also reroot DOM things (among other things it misses).

installing MoreLayouts fixes bug 492216, bug 531397, bug 636874.  perhaps it should simply be uplifted.
(In reply to alta88 from comment #12)
> hehe.  yes, i finally did figure it out.  i blame the tabs implementation,
> it misses the fact that each tab needs to set the msgWindow and also reroot
> DOM things (among other things it misses).
> 
> installing MoreLayouts fixes bug 492216, bug 531397, bug 636874.  perhaps it
> should simply be uplifted.

Alta, does your addon also fix this?
Bug 265393 - Changing View Layout does not preserve the mail message encoding

Alta, is it possible to convert relevant parts of your addon into a patch for TB?
Would you give that a try? According to your comment 12, your addon code seems to fix a lot of bugs... ;)
Hi. Sorry, just a tad confused with the commentary. I rely on lightning for its reminder characteristics; upgraded thunderbird once before and the calendar was broken for some time, don't want to repeat.

(1) Does Lighting still work if I upgrade to TB 15.0 ?
(1a) I may have to choose to disable it for RSS -- I would choose not, I can live without RSS better than I can live without the calendar/reminder.

Thunderbird's announcement to upgrade when you start it has a link for more information. Perhaps in the 'unresolved' section, which suggests lightning problems, should include something to my concerns expressed here.

Thanks.
-doug
Yes, lightning works with tb15. (It auto-updates to a compatible version.)
the patch in bug 596234 fixes this bug and bug 532071, for 3pane.  meaning a layout change does not require restart and summary/web page is displayed upon change.

tabs are still broken on layout change,  bug 636874.  use MoreLayouts.
Blocks: 596234
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
No longer blocks: 596234
Depends on: 596234
No longer depends on: 492216
Whiteboard: [relnote until TB 17]
blocking-kilimanjaro: --- → ?
Flags: blocking-thunderbird3-
Please be careful about altering flags, this has nothing to do with killmanjaro.
blocking-kilimanjaro: ? → ---
ok  i  will be carful  about altering  flags
(In reply to Mark Banner (:standard8) from comment #19)
> Please be careful about altering flags, this has nothing to do with
> killmanjaro.
blocking-basecamp: --- → ?
blocking-kilimanjaro: --- → ?
blocking-basecamp: ? → ---
blocking-kilimanjaro: ? → ---
(In reply to Jcoughlanardpatrick from comment #21)
> (In reply to Mark Banner (:standard8) from comment #19)
> > Please be careful about altering flags, this has nothing to do with
> > killmanjaro.

how do i fix  what wrong  thank you
(In reply to Jcoughlanardpatrick from comment #22)
> (In reply to Jcoughlanardpatrick from comment #21)
> > (In reply to Mark Banner (:standard8) from comment #19)
> > > Please be careful about altering flags, this has nothing to do with
> > > killmanjaro.
> 
> how do i fix  what wrong  thank you

Patrick, thanks for trying to assist. The flags are alright now, you do not have to fix anything, just do not change them in the future unless you know that you are entitled to do so. Setting the wrong flags as you did will not move this bug forward, it will just be confusing.

As you can see from the Status of this bug, it is already "Resolved FIXED".
Per comment 18, it was fixed by patches for bug 596234, which apply to TB17 and above. So if you are on the regular "release update channel", you have to wait just a little for the automatical update to TB17, where this bug 531397 will be fixed.
Whiteboard: [relnote until TB 17] → [relnote until TB 17][fixed by bug 596234]
Per comment 18, fixed by patches for bug 596234.
The patches landed in bug 596234 (which will fix this bug) are:

(from Bug 596234 Comment 67, Bug 596234 Comment 77)
https://hg.mozilla.org/comm-central/rev/4b79ea516c52
https://hg.mozilla.org/comm-central/rev/dbd153b0fa9c
-> Target Milestone: TB18

(from Bug 596234 Comment 81)
https://hg.mozilla.org/releases/comm-aurora/rev/328fbe81793e
-> status-thunderbird17: fixed

So I understand the same applies to this bug, setting flags accordingly.
Pls correct me if I'm wrong.
Target Milestone: --- → Thunderbird 18.0
(In reply to alta88 from comment #18)
> tabs are still broken on layout change,  bug 636874.  use MoreLayouts.
Depends on: 636874
Keywords: relnote
Whiteboard: [relnote until TB 17][fixed by bug 596234] → [fixed by bug 596234]
You need to log in before you can comment on or make changes to this bug.