Closed Bug 1543172 Opened 6 years ago Closed 2 years ago

Update Snippets section in the about:preferences with a link

Categories

(Firefox :: Settings UI, enhancement, P5)

enhancement

Tracking

()

RESOLVED WONTFIX

People

(Reporter: jcollings, Unassigned, Mentored)

Details

Attachments

(2 files)

Please update the about:preferences under the Snippets section with a "How it works" link: https://support.mozilla.org/kb/snippets-firefox-faq

We would like to update it similar to how it currently is for the Recommended by Pocket section and the "How it works" link (see attached screenshot).

Type: defect → enhancement
Priority: -- → P3

Hey Jean, is this part of an ongoing project? Is there a deadline that you're looking to get this fixed? If so, we should probably find an assignee (and link it to the ongoing project through the blocks/depends on field, if applicable).

Flags: needinfo?(jcollings)

(In reply to (behind on needinfos) Jared Wein [:jaws] (please needinfo? me) from comment #1)

Hey Jean, is this part of an ongoing project? Is there a deadline that you're looking to get this fixed? If so, we should probably find an assignee (and link it to the ongoing project through the blocks/depends on field, if applicable)

Hi Jared, there is no deadline and it is not a part of any ongoing project. We just wanted to give our users more information about snippets if they ever wanted to know more.

Hope this helps, thank you!

Flags: needinfo?(jcollings)

Thanks Jean. Kate, would you like to mentor this bug? I found that the code for the Recommend by Pocket line is at one of these two places, https://searchfox.org/mozilla-central/source/browser/components/newtab/lib/AboutPreferences.jsm#228-231,297-300, but I don't know how you do the localization here so I probably shouldn't mentor this bug.

Flags: needinfo?(khudson)
Priority: P3 → P5

Yep, happy to mentor this one!

Flags: needinfo?(khudson)
Mentor: khudson

Hi, could you assign this bug to me please? Would like to work on it.

Hi did anyone see my above message?

Hi can I work on this issue?

Hey. Can I take this one up?

Severity: normal → S3
Flags: needinfo?(khudson)

(In reply to Anshul Sahai from comment #8)

Hey. Can I take this one up?

Sure. Do you still want it?

Flags: needinfo?(asahai100)

Hi, Is this still available?

(In reply to sukhmeet2002 from comment #10)

Hi, Is this still available?

What do you mean?

Flags: needinfo?(sukhmeet2002)

(In reply to hgreen.9 from comment #11)

(In reply to sukhmeet2002 from comment #10)

Hi, Is this still available?

What do you mean?

I wanted to ask if anyone else is working on this issue.

Flags: needinfo?(sukhmeet2002)

(In reply to sukhmeet2002 from comment #12)

(In reply to hgreen.9 from comment #11)

(In reply to sukhmeet2002 from comment #10)

Hi, Is this still available?

What do you mean?

I wanted to ask if anyone else is working on this issue.

ok. I think it is still free but I would just check with the mentor of the bug before you start working on it

Redirect a needinfo that is pending on an inactive user to the triage owner.
:jaws, since the bug has recent activity, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(asahai100) → needinfo?(jaws)
Flags: needinfo?(jaws)

Hi, I can work on this issue, and I have done some research to check whether I was right or not I want to confirm, I have to work on this file - mozilla-unified/browser/components/newtab/lib/AboutPreferences.jsm line - 58-70, Please guide me.

If I am on right track, then suggest me How I can make changes as it will be my first UI related task

Assignee: nobody → tiwari.abhishektiwari23
Status: NEW → ASSIGNED

I have made changes, but I want to confirm whether I am on right track or not, It might also require some more changes, please have a look at my initial work and guide me accordingly

The following patch is waiting for review from an inactive reviewer:

ID Title Author Reviewer Status
D172355 Bug 1543172 - Update Snippets section in the about:preferences with a link r=Jean Collings tiwari.abhishektiwari23 Jean: Inactive

:tiwari.abhishektiwari23, could you please find another reviewer?

For more information, please visit auto_nag documentation.

Flags: needinfo?(tiwari.abhishektiwari23)

Jared, can you get this unstuck? Do we still want this and if so, who could review the patch?

Flags: needinfo?(tiwari.abhishektiwari23)
Flags: needinfo?(khudson)
Flags: needinfo?(jhirsch)

Hmm. This is a confusing situation: on one hand, this bug asks to add a link to a SUMO page, but that page says we are no longer supporting Snippets in Firefox as of version 89. On the other hand, the Snippets about:preferences checkbox is still there, and I see a lot of other snippets references in the source code.

I'm going to ask around and follow up. I'll leave the needinfo for now as a reminder.

Sorry, not sure how the status flags got flipped. Let's put those back.

Attachment #9322554 - Attachment description: Bug 1543172 - Update Snippets section in the about:preferences with a link r=Jean Collings → Bug 1543172 - Update Snippets section in the about:preferences with a link r?settings-reviewers

I've asked around on Slack, and discovered that we might want to pick Snippets back up at a future time, so this patch is still worth landing.

The right next step would be to request review from the settings-reviewers review group, so I've updated the phabricator revision to request a review from that group.

Sorry this patch got dropped, Abhishek! Please ping me if you have any other questions.

Flags: needinfo?(jhirsch)

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: tiwari.abhishektiwari23 → nobody
Status: ASSIGNED → NEW

The Snippets capability, server endpoint, and Settings UI section, was removed in bug 1715158, so I think we can close this one as WONTFIX.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: