Closed Bug 1814426 Opened 1 year ago Closed 1 year ago

Replace "cookieBannerHandlingLearnMore" xul label with moz-support-link

Categories

(Toolkit :: UI Widgets, task, P3)

task

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: tgiles, Assigned: ebilitenelson, Mentored)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug, Whiteboard: [recomp] [lang=html] [lang=js])

Attachments

(1 file, 1 obsolete file)

To help Mozilla out with this bug, here's the steps:

  1. Comment here on the bug that you want to volunteer to help.
    This will tell others that you're working on the next steps.
  2. Download and build the Firefox source code
    • If you have any problems, please ask on Element/Matrix in the #introduction channel. They're there to help you get started.
    • You can also read the Developer Guide, which has answers to most development questions.
  3. Start working on this bug.
    • You'll need to navigate to about:preferences#privacy and scroll/find the "Cookie Banner Reduction" section.
    • The "Learn more" link here needs to be replaced with a <html:a is="moz-support-link" ...> element.
    • Make sure the link still takes you to a support.mozilla.org page after swapping in the new element.
    • If you have any problems with this bug, please comment on this bug and set the needinfo flag for me. Also, you can find me and my teammates on the #reusable-components channel on Element/Matrix most hours of most days.
  4. Build your change with mach build and verify your changes by ensuring the link navigates you to https://support.mozilla.org/en-US/kb/cookie-banner-reduction (currently this page doesn't exist, don't worry about that, just make sure the URL contains the "cookie-banner-reduction" part). Also check your changes for adherence to our style guidelines by using mach lint.
  5. Submit the patch (including an automated test, if applicable) for review. Mark me as a reviewer so I'll get an email to come look at your code.
  6. After a series of reviews and changes to your patch, I'll push it to autoland.
Mentor: tgiles
Keywords: good-first-bug
Whiteboard: [fidefe-reusable-components] → [fidefe-reusable-components] [lang=html] [lang=js]

I am looking into this bug and working on the next steps.

Flags: needinfo?(tgiles)
Assignee: nobody → ajayiabimbola94
Status: NEW → ASSIGNED
Flags: needinfo?(tgiles)

Sir, Can I make a patch for this, I have read the description properly, and my setup is also ready and there is no activity on this bug from last one week, so please give me a chance to give a try to this bug.

Abhishek, you are already assigned to 16 different bugs (11 of which are still open). You should focus on landing work first before picking up more and more bugs. It is more important to deliver quality work as opposed to a large quantity of work.

Hi! I've noticed that there wasn't any activity on this issue for a couple of weeks, may I try to work on it?

Opening this back up to first-time contributors. Ganna and Abhishek, I may assign this to one of you two if there is not interest in solving this bug. For the moment, this will be reserved for someone who has not yet contributed to Firefox and wants to get started. For those interested, please comment on the bug that you wish to work on this and consult Comment #0 and Comment #1

Assignee: ajayiabimbola94 → nobody
Status: ASSIGNED → NEW

hello, Tim, can i take over this bug and turn it into a hug instead

hello Tim, I realized the link in step 4 is broken.

4.Build your change with mach build and verify your changes by ensuring the link navigates you to https://support.mozilla.org/en-US/kb/cookie-banner-reduction (currently this page doesn't exist, don't worry about that, just make sure the URL contains the "cookie-banner-reduction" part). Also check your changes for adherence to our style guidelines by using mach lint.

Is it fine to report it as a bug, ?

sorry about this

Hi Brianna, the link in step 4 is expected to be broken. You do not need to file a bug for it, as we have a bug on file for it. Note: you cannot work on the linked bug, that is something that has to be handled by the Support folks, so please don't comment on Bug 1800663 asking to work that one 🙂

briannapatricia: Are you still working on this bug or should I give it a try

Assignee: nobody → ebilitenelson
Status: NEW → ASSIGNED

Hello tgiles,

Please overlook my submitted patch, I have been pointed to know that brianna is currently working on the task.

Sorry for the inconveniences @briannapatricia7

Attachment #9324591 - Attachment is obsolete: true

thanks Ebilite for the work done so far. Please take it up. thanks ..........go on with the bug

Thanks alot Briannapatricia, Will take it up now

Attachment #9324591 - Attachment is obsolete: false
Attachment #9324591 - Attachment is patch: true
Attachment #9324591 - Attachment mime type: text/x-phabricator-request → text/plain
Attachment #9324591 - Attachment is obsolete: true
Attachment #9324591 - Attachment is patch: false
Pushed by tgiles@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a33a5b32b786
Replaced "cookieBannerHandlingLearnMore" xul label with moz-support-link. r=tgiles
Blocks: 1824389
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
Regressions: 1825129
See Also: → 1800663
Whiteboard: [fidefe-reusable-components] [lang=html] [lang=js] → [recomp] [lang=html] [lang=js]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: