Closed Bug 472076 Opened 17 years ago Closed 16 years ago

Changing the layout view when seeing a single message on a tab cause weird behaviors

Categories

(Thunderbird :: Toolbars and Tabs, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0rc1

People

(Reporter: ShareBird, Assigned: rain1)

References

Details

(Whiteboard: [no l10n impact])

Attachments

(1 file)

When I'm seeing a single message on a tab, if I change the layout view: "View" > "Layout", I see very strange behaviors. If I've choose originally the "standard" view and change to "vertical" the message will "disappear" and the main tab will become unusable. Steps to reproduce: 1. Open Thunderbird and be sure you are using the "classic" view. 2. Open a single message on a tab. 3. With this tab open, change the view to "vertical". Results: The content from this tab will "disappear". The initial tab will be unusable. Expect: I'd expect no changes on this tab and changes on the main tab (the 3 pane window). Maybe we could disable changes on view in this situation (one single message on the tab).
No longer blocks: smtabmail
Component: Mail Window Front End → Toolbars and Tabs
QA Contact: front-end → toolbars-tabs
Target Milestone: --- → Thunderbird 3.0rc1
Assignee: nobody → bugmail
Flags: blocking-thunderbird3+
This still happens even with the bug 474701 changes. We should indeed just disable the layout menu on message tabs.
Whiteboard: [no l10n impact]
Assignee: bugmail → sid.bugzilla
I also noticed that there is also problem with codepage detection/setting in Message pane window when you change layout in 3-pane view. I reproduced it with message with following encoding: Content-Type: text/plain; charset="koi8-r" Content-Transfer-Encoding: 8bit Steps to reproduce: 1. Open Thunderbird in 3-pane view 2. Select message to see content in message pane in 3-pane view 3. Change Layout to any different 4. Now you should see message with crazy encoding.
Whiteboard: [no l10n impact] → [no l10n impact][patch in progress]
Attached patch patch, v1Splinter Review
I think we should move to separate controllers for each tab mode in the future, but for now I think this is reasonable and not too risky.
Attachment #404674 - Flags: review?(bugmail)
Attachment #404674 - Flags: review?(bienvenu)
Whoever gets to this first.
Status: NEW → ASSIGNED
Whiteboard: [no l10n impact][patch in progress] → [no l10n impact][needs review bienvenu or asuth]
Attachment #404674 - Flags: review?(bugmail)
Attachment #404674 - Flags: review?(bienvenu)
Attachment #404674 - Flags: review+
Whiteboard: [no l10n impact][needs review bienvenu or asuth] → [no l10n impact]
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
OS: Windows XP → All
Hardware: x86 → All
Resolution: --- → FIXED
Depends on: 520770
Depends on: 521026
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: