Closed Bug 1193849 Opened 4 years ago Closed 4 years ago

Bugs with "Learn More" link for Mixed Content Blocker in Control Center

Categories

(Firefox :: Security, defect, P1)

42 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 44
Iteration:
44.1 - Oct 5
Tracking Status
firefox41 --- unaffected
firefox42 --- verified
firefox43 --- verified
firefox44 --- verified

People

(Reporter: tanvi, Assigned: ttaubert)

References

Details

(Whiteboard: [fxprivacy])

Attachments

(1 file)

The enable protection page is missing it's Learn More link.

The disable protection page has the Learn More link in line with the other text, when I think it is supposed to be on a line of it's own.

(The page also needs updating, but I think that is in another bug? https://support.mozilla.org/en-US/kb/how-does-content-isnt-secure-affect-my-safety?as=u&utm_source=inproduct)

We should see if we can uplift the fixes to FF 42.
Depends on: 1175702
Whiteboard: [fxprivacy] [triage]
(In reply to Tanvi Vyas [:tanvi] from comment #0)
> The enable protection page is missing it's Learn More link.

Yeah, the mockups didn't have them. I thought it was explicitly left out because we expect people to read before disabling MCB :) Quite easy to add one though if we want that.

> The disable protection page has the Learn More link in line with the other
> text, when I think it is supposed to be on a line of it's own.

It's an inline element, if it fits on the line then it won't have one of its own. If it doesn't it will wrap around.

> (The page also needs updating, but I think that is in another bug?
> https://support.mozilla.org/en-US/kb/how-does-content-isnt-secure-affect-my-
> safety?as=u&utm_source=inproduct)

Yes.

> We should see if we can uplift the fixes to FF 42.

Shouldn't be a problem.
Priority: -- → P2
Whiteboard: [fxprivacy] [triage] → [fxprivacy]
Blocks: 1188565
Is this possible to do for the 42 timeline?  Should be pretty easy and would be nice if we could get it in before aurora/42 because beta/42 on September 22nd.  I'm not sure this counts as post v1, since the learn more link existed before tracking protection v1.
Priority: P2 → P3
Added the "learn more" link to the security subview when mixed content blocking was disabled by the user. The links remain inline elements, they will wrap around as needed but will otherwise be on the same line as the owning paragraph. (That was how it was designed and what the mockups showed.)
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Attachment #8664798 - Flags: review?(paolo.mozmail)
Attachment #8664798 - Flags: feedback?(tanvi)
Flags: qe-verify?
Comment on attachment 8664798 [details] [diff] [review]
0001-Bug-1193849-Show-Learn-more-link-in-security-subview.patch

Thanks Tim!
Attachment #8664798 - Flags: feedback?(tanvi) → feedback+
Attachment #8664798 - Flags: review?(paolo.mozmail) → review+
https://hg.mozilla.org/mozilla-central/rev/c096530a4b4b
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Iteration: --- → 44.1 - Oct 5
Priority: P3 → P1
Comment on attachment 8664798 [details] [diff] [review]
0001-Bug-1193849-Show-Learn-more-link-in-security-subview.patch

Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]: Missing "learn more" link when opening the control center after disabling mixed content blocking.
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: Very low risk
[String/UUID change made/needed]: None
Attachment #8664798 - Flags: approval-mozilla-beta?
Attachment #8664798 - Flags: approval-mozilla-aurora?
Flags: qe-verify? → qe-verify+
QA Contact: petruta.rasa
Comment on attachment 8664798 [details] [diff] [review]
0001-Bug-1193849-Show-Learn-more-link-in-security-subview.patch

Polish of a new feature, taking it. Should be in 42 beta 3.
Attachment #8664798 - Flags: approval-mozilla-beta?
Attachment #8664798 - Flags: approval-mozilla-beta+
Attachment #8664798 - Flags: approval-mozilla-aurora?
Attachment #8664798 - Flags: approval-mozilla-aurora+
Reproduced using Firefox 42 beta 2 with the following pages:
https://people.mozilla.org/~tvyas/mixedcontent.html
https://people.mozilla.org/~tvyas/mixedboth.html

Verified as fixed using Firefox 42 beta 5, Developer Edition 43.0a2 and Nightly 44.0a1 2015-10-09 under Win 7 64-bit and Mac OS X 10.9.
You need to log in before you can comment on or make changes to this bug.