Closed Bug 1642811 Opened 6 months ago Closed 6 months ago

Rename UI references of "blackbox" to "ignore" in Debugger

Categories

(DevTools :: Debugger, task, P3)

task

Tracking

(firefox79 fixed)

RESOLVED FIXED
Firefox 79
Tracking Status
firefox79 --- fixed

People

(Reporter: Harald, Assigned: rich)

References

(Blocks 2 open bugs)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

Follow up from https://github.com/firefox-devtools/ux/issues/84, we want to rename "blacklist" references to "ignore" in the UI.

https://searchfox.org/mozilla-central/search?q=blackbox&path=devtools%2Fclient%2Flocales%2Fen-US%2Fdebugger.properties

Blackbox sourceIgnore source
Unblackbox sourceUnignore source

A follow-up bug should handle the source renaming, which needs to be balanced with protocol breaking changes.

Blocks: dbg-blackbox
Summary: Rename UI references of "backlist" to "ignore" in Debugger → Rename UI references of "blacklist" to "ignore" in Debugger

Removed UI references to blacklist, replaced with ignore. Also updated the shortcut key to 'I' instead of 'D' since it's no longer Blacklist. Lastly updated tests to fix failing snapshot tests and jest tests.

Assignee: nobody → hello
Status: NEW → ASSIGNED

Janet,

since the current blackboxing docs are not the most read articles anyways, it seems we could update the docs right now: https://www.google.com/search?client=firefox-b-1-d&q=%28blackbox+or+blackboxing%29+site%3Adeveloper.mozilla.org%2Fen-US%2Fdocs%2FTools%2F

What do you think?

Flags: needinfo?(jswisher)
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/70a4c6e9cf6e
Rename UI references of "blacklist" to "ignore" in Debugger r=jlast,flod

Canceling ni, cmills mentioned that language updates are already happening.

Flags: needinfo?(jswisher)

I'm replacing "black list" on MDN, but haven't done anything with "black box". To me, "black box" does not have the same negative connotation as "black list". I'm open to suggestions for rewording, though.

I'm open to suggestions for rewording, though.

The new verb that landed with this patch is "ignore".

(In reply to :Harald Kirschner :digitarald from comment #7)

The new verb that landed with this patch is "ignore".

That's fine for "black list". Your link in Comment 3 was for "black box". That's what I was asking for alternatives for.

Right, we did mix those up in the issue name – my bad. "Ignore source" is the new name for "Blackbox source".

Oh shoot – that was my mistake in the commit message. Sorry for the confusion!

Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79
Summary: Rename UI references of "blacklist" to "ignore" in Debugger → Rename UI references of "blackbox" to "ignore" in Debugger
Blocks: 1642034
Blocks: 1646983
No longer blocks: 1642034
QA Whiteboard: [qa-79b-p2]

Updated the articles listed in comment #11, except Settings, which I moved into the "Archive" area of MDN, because it's obsolete and not referenced from anywhere else (that I found).

You need to log in before you can comment on or make changes to this bug.