Add-on icon is not hidden when pageproxystate=false (editing the URL bar)

RESOLVED FIXED in Firefox 55

Status

()

Firefox
Site Identity and Permission Panels
P1
normal
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: johannh, Assigned: johannh)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

3 months ago
We hide all icons and fade the (i) when the user has changed the page url.
Comment hidden (mozreview-request)

Comment 2

3 months ago
mozreview-review
Comment on attachment 8869016 [details]
Bug 1365911 - Hide extension identity icon on pageproxystate=false.

https://reviewboard.mozilla.org/r/140680/#review144142

Thanks for adding this - looks good to me. Adding mixedpuppy for the stamp of approval.

::: browser/themes/shared/identity-block/identity-block.inc.css:154
(Diff revision 1)
>    opacity: 0;
>    /* Only animate the shield in, when it disappears hide it immediately. */
>    transition: none;
>  }
>  
> +#urlbar[pageproxystate="invalid"] > #identity-box > #extension-icon,

It's not necessary, but I think it would be nice if all of the pageproxystate rules were put in one place:

http://searchfox.org/mozilla-central/search?q=pageproxystate&case=false&regexp=false&path=.css
Attachment #8869016 - Flags: review?(mwein) → review+

Updated

3 months ago
Attachment #8869016 - Flags: review?(mixedpuppy)
(Assignee)

Comment 3

3 months ago
(In reply to Matthew Wein [:mattw] from comment #2)
> It's not necessary, but I think it would be nice if all of the
> pageproxystate rules were put in one place:
> 
> http://searchfox.org/mozilla-central/
> search?q=pageproxystate&case=false&regexp=false&path=.css

I had the same thought but I'd like to solve that in a different bug. :)

Comment 4

3 months ago
(In reply to Johann Hofmann [:johannh] from comment #3)
> (In reply to Matthew Wein [:mattw] from comment #2)
> > It's not necessary, but I think it would be nice if all of the
> > pageproxystate rules were put in one place:
> > 
> > http://searchfox.org/mozilla-central/
> > search?q=pageproxystate&case=false&regexp=false&path=.css
> 
> I had the same thought but I'd like to solve that in a different bug. :)

Okay, sounds good :)
Can someone post a followup bug?

Comment 6

3 months ago
mozreview-review
Comment on attachment 8869016 [details]
Bug 1365911 - Hide extension identity icon on pageproxystate=false.

https://reviewboard.mozilla.org/r/140680/#review144188
Attachment #8869016 - Flags: review?(mixedpuppy) → review+
(Assignee)

Updated

3 months ago
Blocks: 1366200

Comment 7

3 months ago
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d808a3a6463
Hide extension identity icon on pageproxystate=false. r=mattw,mixedpuppy

Comment 8

3 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6d808a3a6463
Status: ASSIGNED → RESOLVED
Last Resolved: 3 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.