Closed
Bug 154634
Opened 22 years ago
Closed 22 years ago
if you collapse the message pane, the security status can be misleading
Categories
(MailNews Core :: Security: S/MIME, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
Future
People
(Reporter: sspitzer, Assigned: KaiE)
Details
Attachments
(1 file)
3.52 KB,
patch
|
bugzilla
:
review+
sspitzer
:
superreview+
|
Details | Diff | Splinter Review |
if you collapse the message pane, the security status can be misleading steps to reproduce: open mail, with the message pane open. view a signed message. hide the message pane click on a message that isn't signed. the security status will still show that the message is signed, and if you click on the security status area, it will give you status about the signed message, not the message you have currently selected in the thread pane. I think the fix is to clear (or hide) the security status area when the message pane is hidden.
Comment 1•22 years ago
|
||
S/MIME
Assignee: ssu → ssaux
Component: Security: General → S/MIME
Product: MailNews → PSM
QA Contact: junruh → carosendahl
Version: other → 2.3
Comment 2•22 years ago
|
||
Confirmed. Seth's suggestion to clear/hide the status bar since it is no longer active when the message pane is not viewable is correct.
Comment 3•22 years ago
|
||
Kai. Nominate for buffy.
Updated•22 years ago
|
Target Milestone: --- → Future
Assignee | ||
Updated•22 years ago
|
Assignee | ||
Comment 4•22 years ago
|
||
This should fix it. Seth, can you please r/sr?
Reporter | ||
Comment 5•22 years ago
|
||
Comment on attachment 95517 [details] [diff] [review] Patch v1 sr=sspitzer
Attachment #95517 -
Flags: superreview+
Comment 6•22 years ago
|
||
Comment on attachment 95517 [details] [diff] [review] Patch v1 R=ducarroz
Attachment #95517 -
Flags: review+
Assignee | ||
Comment 7•22 years ago
|
||
Fixed on trunk. The status bar icons should automatically hide, regardless which method to hide the message pane is used. I tested: - use menu - drag the splitter When you unhide the pane, and there is still the same message selected in the message list, the security status of that message should automatically reappear.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 8•22 years ago
|
||
if (gDBView) { gDBView.suppressMsgDisplay = now_hidden; } NotifyChangedMessagePaneVisibility(now_hidden); is duplicated; in theory you could move the if(){} into the common function.
Comment 9•22 years ago
|
||
Verified - status bar now behaves correctly using either menu or gui based functions. 20020904 Trunk Build
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•