Closed Bug 1708879 Opened 4 years ago Closed 8 months ago

(Proton) (regression) the padlock icon position in the address bar is not stable for secure and mixed content

Categories

(Firefox :: Address Bar, defect, P3)

defect

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox-esr78 --- unaffected
firefox88 --- unaffected
firefox89 --- wontfix
firefox90 --- fix-optional

People

(Reporter: soeren.hentzschel, Unassigned)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [proton-icons][proton-address-bar] [priority:2b])

Attachments

(1 file)

Attached video screencast

If you switch between tabs with secure and mixed content the position of the padlock icon in the address bar changes. This is a very noticeable regression in the user experience. I attached a screencast that shows both Firefox Nightly and Firefox 88 in comparison.

Priority: -- → P2
Whiteboard: [proton-icons][proton-address-bar] → [proton-icons][proton-address-bar] [priority:2b]
QA Whiteboard: [qa-regression-triage]

Hello! Attaching regression range made on Windows 10x64 using security.mixed_content.upgrade_display_content:false
Last good revision: bfc4564739af31f55d7c7370af45551b35722e0c
First bad revision: ef291044883cd93a7810d9466c48993ff4da974a
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=bfc4564739af31f55d7c7370af45551b35722e0c&tochange=ef291044883cd93a7810d9466c48993ff4da974a

Has Regression Range: --- → yes
Has STR: --- → yes
Regressed by: 1702289

Yeah the new icon doesnt treat the warning state as a badge/overlay, it centers the lock+triangle in the 16x16 grid. I'll need to look to see if we use that anywhere else. In another context it might be important for the whole to remain centered. If not, this might be something we can fix.

Still happens on Nightly.

Severity: -- → S4

(In reply to Drew Willcoxon :adw from comment #3)

Still happens on Nightly.

Yeah this is by design currently. The question is if we want to change the design and what other ramifications that might have.

It appears that the "Not Secure" label is now shown for mixed content pages. However, with the "Note Secure" label, there is a bit of a shift in the padlock's position compared to secure content. Would this be something that should be aligned in both cases?

Flags: needinfo?(jberman)

Hey yall,

I'm currently looking into this. Looks like this issue has been sitting around for a while. I think ideally the lock icon would be aligned and not shift, but i also think we should be connecting with design systems on this one because it's really about how do we do overlapping icons.

Looping in Jane - if you have someone who could take a look at this and collaborate please let us know.

Flags: needinfo?(jberman) → needinfo?(jwolf)

hi! Going to tag in Crystal here, we're looking at this actually right now with how we do badging on icons in the lock and shield.

Flags: needinfo?(jwolf) → needinfo?(cwong)

Hi all, I think we can close this bug for now as there will be a future implementation that will solve this issue by combining the lock and shield icon.

Flags: needinfo?(cwong)

Ok, since there is a redesign pending, we are going to close the bug as wontfix, but it could be reopened (or a new one filed) if after the next redesign, the problem will remain.

Status: NEW → RESOLVED
Closed: 8 months ago
Priority: P2 → P3
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: