bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Security panel needs some cleanup

VERIFIED FIXED

Status

MailNews Core
Security: S/MIME
VERIFIED FIXED
16 years ago
9 years ago

People

(Reporter: Stephen Walker, Assigned: Stephane Saux)

Tracking

1.0 Branch
x86
All

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

16 years ago
The Security panel in the Account Manager is in need of some cleanup. The patch 
removes the unneeded orient="vertical" on the groupboxes, since they are 
vertically oriented by default, centers the textboxes with the buttons, and 
removes some (imo) unnecessary spacing.
(Reporter)

Comment 1

16 years ago
Created attachment 85188 [details]
Before/After
(Reporter)

Comment 2

16 years ago
Created attachment 85190 [details] [diff] [review]
patch

Comment 3

16 years ago
Comment on attachment 85190 [details] [diff] [review]
patch

Look good to me.
r=kaie
Attachment #85190 - Flags: review+

Comment 4

16 years ago
Can you post a screen shot showing what the dialog looks like with your changes?
(Reporter)

Comment 5

16 years ago
mscott, the right image in http://bugzilla.mozilla.org/showattachment.cgi?
attach_id=85188 is what the dialog looks like with the changes.

Comment 6

16 years ago
*duh* don't know how I missed that attachment.

I believe the spec shows the space before the cert text after the checkbox (for
encryption) and after the radio buttons (for the signing text). In addition I
think the original picture (and as such the spec) looks better with that space.

Can you please put back the two thin separators before these lines of text. Then
I'll sr it. Thanks.
(Reporter)

Comment 7

16 years ago
Created attachment 86267 [details] [diff] [review]
patch v1.1
Attachment #85190 - Attachment is obsolete: true

Comment 8

16 years ago
Comment on attachment 86267 [details] [diff] [review]
patch v1.1

Diffing your patches shows, the only change is, you are no longer removing two
of the thin separators. Fine with me.

r=kaie
Attachment #86267 - Flags: review+

Comment 9

16 years ago
Comment on attachment 86267 [details] [diff] [review]
patch v1.1

sr=mscott
Attachment #86267 - Flags: superreview+
C:\moz_src\mozilla\mailnews\extensions\smime\resources\content>cvs commit -m
"Bug 147438.  Security panel needs some cleanup.  Patch by Stephen Walker
<walk84@yahoo.com> r=kaie, sr=mscott" am-smime.xul
Checking in am-smime.xul;
/cvsroot/mozilla/mailnews/extensions/smime/resources/content/am-smime.xul,v  <--
 am-smime.xul
new revision: 1.9; previous revision: 1.8
done
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 11

16 years ago
Is this in the trunk as well?  It is not in the Trunk build as of 20020614.

Usually I mark these verified after landing on trunk, and then add the
verified1.0.1 keyword after landing on branch.

Comment 12

16 years ago
this bug fix is ONLY in the trunk. It was never nominated nor approved for the
branch. Nor do we want to take it for the branch. Why do you think it's in the
branch? (I'm asking in case we need to raise a flag) According to CVS it is in
the trunk and not the branch *phew*:

http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/mailnews/extensions/smime/resources/content/am-smime.xul

Comment 13

16 years ago
I mention this because I do not see the fix in either the Trunk or thebranch
builds on Windows 2k.  I still see the old UI (In particular, I am looking at
the box corner that matches up with the accounts button and still see the old
positioning.)

Comment 14

16 years ago
Hmmm...I'm using 2002061208 and I think see the changes in my trunk build. I
think the only visible change you should be able to notice is the texboxes and
the buttons being centered together. Plus the removal of one small piece of
veritcal space between the sign box and the encryption box. Everything else
wouldn't show up in the UI, it was under the hood cleanup. 

Comment 15

16 years ago
Ok - I see it now.  The before after images threw me off, since I was looking at
the placement of the lines inaddition to the other changes.  Very subtle fix.

verified.
Status: RESOLVED → VERIFIED

Updated

14 years ago
Component: Security: S/MIME → Security: S/MIME
Product: PSM → Core

Updated

10 years ago
Version: psm2.3 → 1.0 Branch

Updated

9 years ago
Component: Security: S/MIME → Security: S/MIME
Product: Core → MailNews Core
QA Contact: carosendahl → s.mime
You need to log in before you can comment on or make changes to this bug.