Issue metadata
Sign in to add a comment
|
Security indicators: iOS shows page icon on mixed content sites |
||||||||||||||||||||||||
Issue descriptioniOS shows the page icon on mixed content sites, instead it should not show any icon in this state.
,
Sep 14 2016
This is working as intended, since there is no other way to get security details for downgraded HTTPS otherwise. We could poke iOS folks to finally implement Issue 533581, though. Also note that the icon is no ⓘ on iOS (Canary).
,
Sep 14 2016
,
Sep 14 2016
For iPad, we should show the (i) for all HTTP and mixed content sites, there should be no problem. This bug should be fixed. For iPhone, yes, this is a problem. I will investigate.
,
Sep 14 2016
How should this relate to Issue 646545 (Show ⓘ for HTTP on iPhones behind a flag)? Should there be a single flag for showing the icon for neutral pages, or one for mixed content and one for HTTP?
,
Sep 14 2016
,
Sep 14 2016
,
Sep 14 2016
Mixed content is a separate issue that has not been approved by ui-review and is not explicitly in the HTTP-bad plan right now. I think there are a few approaches: (1) We should indeed think about whether mixed content should be a part of HTTP bad. This is not clear to me right now. We *could* do this but it's not Approved or explicit right now. (2) We should prioritize 533581. I will talk to the iOS PMs separately to see what we should do.
,
Sep 14 2016
iOS PM here :). Regarding issue 533581, is this something that the security team can tackle with code review support from Bling? We haven't addressed the issue in comment #5 also. A related question: have we agreed on the design for verbose security state for iOS? The mocks I've seen were for Android. iOS has very little horizontal space so if we put all that, the URL won't have any space (there are back/forward buttons on iOS)
,
Sep 14 2016
> Regarding issue 533581, is this something that the security team can tackle with code review support from Bling? We haven't addressed the issue in comment #5 also. I've been tackling more and more of these, but we have limited time on the Enamel side for anything that isn't a small (≈10 lines) fix.
,
Sep 16 2016
Hello! :) Re: issue 533581, I could help PM the necessary mocks, but I don't know if we have implementation time if it's not a small fix. Do you have an estimate for how large of a change this would be? Re: c#5 on that bug I agree we should try to preserve that behavior and that would depend on an updated design, I think. Re: verbose security states -- Clank shares the same concerns (there are some Clank models with another back button) and for these reasons the plan right now is to only implement on tablets (or for iPhone in landscape mode, if that is >=600dp.) Does that sound acceptable to you?
,
Oct 19 2016
,
Oct 19 2016
Sorry for the delay in replying, Emily. justincohen@ or lgarron@ would know better than me how large of a change this would be. Are there new mocks ? Re: verbose security states, that sounds acceptable. I don't see how we can squeeze in more verbosity in compact views. Thanks.
,
Nov 23 2016
,
Nov 10 2017
,
Dec 1 2017
,
Dec 1 2017
Looks like this had been fixed. iOS Stable shows the info icon for mixed content sites which is working as intended. Please feel free to close the issue.
,
Dec 1 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by maxwalker@chromium.org
, Sep 14 2016