1524685 - The onboarding UI refers to the disable protection button as "Turn off Blocking Temporarily"

Status: RESOLVED INVALID

(www.mozilla.org :: Bedrock, defect)

Description

[(no longer active)]

Attachment: Screenshot

Please see the screenshot. Test URL: https://www.mozilla.org/en-US/firefox/65.0/content-blocking/start/

Comment 1

[(no longer active)]

[Tracking Requested - why for this release]: Our UI says one thing, the prompt says another thing.

Comment 2

[Liz Henry (:lizzard) (privacy/fingerprinting team)]

[Tracking Requested - why for this release]:

Comment 3

[Ryan VanderMeulen [:RyanVM]]

I'd consider this for a dot release ride-along assuming we can fix it without making string changes.

Comment 4

[(no longer active)]

Looks like this string is defined in bedrock:

https://github.com/mozilla/bedrock/blob/b31f5deecd41f98ef018684f4b57a36496349bbb/bedrock/firefox/templates/firefox/content-blocking-tour/index.html#L39

This is good news, since it may mean that the bug could be fixed without a change to Firefox.

Moving the bug to the right component.

Comment 5

[Alex Gibson [:agibson]]

Attachment: private-browsing-window.png

The URL is in the description is meant to only be opened in context of a Private Browsing window.

Attached is a screenshot of the strings displayed in Private Browsing mode.

Comment 6

[Alex Gibson [:agibson]]

Attachment: normal-browsing-window.png

Screenshot of strings for a normal browsing window

URL: https://www.mozilla.org/en-US/firefox/65.0/content-blocking/start/?newtab=true

Comment 7

[Alex Gibson [:agibson]]

Based on the above, I'm going to close this bug as invalid, since the two scenarios are intended behavior. But please feel free to re-open or let me know if you haver any questions. Thanks!

Comment 8

[(no longer active)]

[Tracking Requested - why for this release]:

Yes, thanks Alex. I'm almost certain that I encountered the original link out of context (e.g. on IRC or Slack) and I didn't realize we have different links for each context. Apologies for the confusion.