Closed Bug 253288 Opened 17 years ago Closed 17 years ago

At secure sites, the Lock icon is missing from the status bar

Categories

(Firefox :: General, defect, P1)

1.0 Branch
x86
Windows XP
defect

Tracking

()

VERIFIED FIXED
Firefox1.0beta

People

(Reporter: tracy, Assigned: steffen.wilberg)

References

(Blocks 1 open bug, )

Details

(Keywords: fixed-aviary1.0, regression, Whiteboard: [have patch])

Attachments

(3 files, 1 obsolete file)

Seen on windows Firefox branch build 0.9+ 2004-07-27-08-0.9

-Go to https://sourceforge.net

tested results: The domain name of the site appears in the staus bar but the
Lock icon is missing.  Double clicking does open the Page Info with security tab
selected as expected.  We're just missing the lock icon.

expected results: The domain name and the Lock icon both appear in the status bar.  

note: This works as expected with the same time stamped build on Linux.
Caused by the checkin for bug 245406.
Summary: At secure sites the Lock icon missing from the staus bar. → At secure sites, the Lock icon missing from the status bar.
Additionally, the domain seems to overlap the status bar area dedicated to
security, and at times it can also be truncated. See attachment.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040727
Firefox/0.9.1+
This is the same patch I already attached to bug 245406. Moving it to this bug
for clarity's sake.
Assignee: firefox → steffen.wilberg
Status: NEW → ASSIGNED
Comment on attachment 154490 [details] [diff] [review]
fix the button width on Winstripe

The button should be a bit wider than 20px when displaying the host name.

Remaining glitch:
On sites where a broken lock is displayed, like https://www.dab-bank.com/, the
icon is moved to the right. If there's no width specified, the icon position is
correct, but the button is extended to the left, i.e. a blank space is
displayed left of the icon.

Comments by Gerv:
I also see the blank space to the left of the icon; this seems to be caused by
the left and right margins on the empty label. I'm not quite sure why they are
appearing when the icon appears but not when it's not...
We _could_ fix the label margins problem by styling it to have zero margins
when
it's empty, but I'm not sure that's the right fix; this may be symptomatic of
some deeper misunderstanding/mistake.
Attachment #154490 - Flags: review?(mconnor)
Attachment #154490 - Flags: approval-aviary?
Flags: blocking-aviary1.0PR?
Flags: blocking-aviary1.0?
Keywords: regression
Target Milestone: --- → Firefox1.0beta
Flags: blocking-aviary1.0PR?
Flags: blocking-aviary1.0PR+
Flags: blocking-aviary1.0?
Flags: blocking-aviary1.0+
Priority: -- → P1
Why not change the width attribute to min-width instead of adding a bunch of
other rules?  I've done this in the classic jar and it fixed it for me. 
page-report-button, page-theme-button, and livemark-button rules could use the
same setting.
*** Bug 253345 has been marked as a duplicate of this bug. ***
Comment on attachment 154490 [details] [diff] [review]
fix the button width on Winstripe

r/a=ben@mozilla.org
Attachment #154490 - Flags: review?(mconnor)
Attachment #154490 - Flags: review+
Attachment #154490 - Flags: approval-aviary?
Attachment #154490 - Flags: approval-aviary+
Steffen: I'm assuming you'll be checking this in...

The last thing to do then is to fix the problem with the box widening when
there's an icon but no text (in the broken security case, for example). This is
caused by the default margins on the label. 

We need a CSS rule which styles the label when it's _missing_ a "value"
attribute (either that, or we need to change the code so it always has a value
attribute, but it's blank when it's not being used). I'm not sure I know how to
write such a rule in the first case; I've tried using the :not pseudoclass but I
can't get it to work.

Gerv
This fixes the problem with the hidden label shifting the broken icon to the
right:
   +/* XXXsw a label without a value still has margins, so hide it */
   +#security-button > label:not([value]) {
   +  display: none;
   +}

(I also tried to set that as a general rule in xul.css, but it hides all sorts
of labels in the Options dialog (especially in Web Features) which have
"xbl:text=label" instead of "label".)

With that fixed, we can use min-width. I had to play a lot with paddings to
make that behave, i.e. still 20px wide when the broken lock is displayed, and
showing the icon at the same position, whether "broken" or not.

I didn't touch Pinstripe since that does |display:none| on the buttons when no
icon is displayed.
Comment on attachment 154553 [details] [diff] [review]
also fix the broken lock icon

better patch, please have a look.
Attachment #154553 - Flags: review?(bugs)
Attachment #154553 - Flags: approval-aviary?
Nice one :-) That's what I couldn't work out. Go to it. I'm away for the next
nine days; thank you so much for clearing up after me :-)

Gerv
Comment on attachment 154553 [details] [diff] [review]
also fix the broken lock icon

> #page-report-button[blocked] {
>   list-style-image: url("chrome://browser/skin/Info.png");
>-  padding: 0px 3px 0px 3px; 
> }

... etc ...

what does this have to do with fixing the lock icon? What's the visual effect?
changed the test URL to verisign.com.  Sourceforge has been serving up mixed
content. 
> > #page-report-button[blocked] {
> >   list-style-image: url("chrome://browser/skin/Info.png");
> >-  padding: 0px 3px 0px 3px; 
> > }
> 
> ... etc ...
> 
> what does this have to do with fixing the lock icon?
I'm cleaning up the padding. Right now, we have statusbarpanel { padding: 0 4px; }
http://lxr.mozilla.org/aviarybranch/source/toolkit/themes/winstripe/global/global.css#127

Without these 4px left and right, the buttons have a width of 18px. But they're
set to 20px in browser.css. So the 4px in global.css are too much, as well as
the 3px in in some places in browser.css. I set the padding to 1px left and
right in global.css, so that I can use min-width for the lock icon and still get
a width of 20px if the broken lock is displayed.

What's the visual effect?
None :) The 3px padding is not visible because the button width is set to 20px
anyway.
Summary: At secure sites, the Lock icon missing from the status bar. → At secure sites, the Lock icon is missing from the status bar
Comment on attachment 154553 [details] [diff] [review]
also fix the broken lock icon

Please don't request approval until you've received necessary reviews. Thanks.
Attachment #154553 - Flags: approval-aviary?
Whiteboard: [have patch]
I thought Ben said Theme fixes did not need approval-aviary for checkin anyway.
Attachment #154490 - Attachment is obsolete: true
What' up with this bug?  we have a patch.  It's a security issue.  Why isn't it
checked in yet?
Because, like a bunch of other stuff, it's awaiting review. It's marked as a
blocker for 1.0PR, so it should get looked at by then. Have some patience...
These are indeed theme fixes; contact the maintainer of the themes to get
review, then check in ASAP :-)

Gerv
Blocks: 244691
Comment on attachment 154553 [details] [diff] [review]
also fix the broken lock icon

Kevin, can you jump in for Ben and review this, please? We need to fix this
before 1.0PR.
Attachment #154553 - Flags: superreview?(webmail)
Comment on attachment 154553 [details] [diff] [review]
also fix the broken lock icon

steffen, just go ahead and land this now... r+a=ben@mozilla.org
Attachment #154553 - Flags: superreview?(webmail)
Attachment #154553 - Flags: review?(bugs)
Attachment #154553 - Flags: review+
Attachment #154553 - Flags: approval-aviary+
Checked into the branch 08/09/2004 14:07.
The trunk doesn't have the restyled notification area yet.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Keywords: fixed-aviary1.0
windows FF0.9+ build 2004-08-10-08-0.9 is fixed.

Status: RESOLVED → VERIFIED
Appologies... crossed wires.
Blocks: lockicon
You need to log in before you can comment on or make changes to this bug.