Rename UI references of "blackbox" to "ignore" in Debugger
Categories
(DevTools :: Debugger, task, P3)
Tracking
(firefox79 fixed)
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.
Blackbox source
→ Ignore source
Unblackbox source
→ Unignore source
…
A follow-up bug should handle the source renaming, which needs to be balanced with protocol breaking changes.
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
When it's renamed, the documentation related to it needs to be adjusted accordingly.
The places where I saw it being mentioned are:
- https://developer.mozilla.org/en-US/docs/Tools/Debugger/UI_Tour
- https://developer.mozilla.org/en-US/docs/Tools/Debugger/How_to
- https://developer.mozilla.org/en-US/docs/Tools/Debugger/How_to/Black_box_a_source (needs a general update)
- https://developer.mozilla.org/en-US/docs/Tools/Debugger/Set_event_listener_breakpoints
- https://developer.mozilla.org/en-US/docs/Tools/Debugger/Settings (needs a general update)
Sebastian
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
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.
Updated•5 years ago
|
Reporter | ||
Comment 3•5 years ago
|
||
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?
Reporter | ||
Comment 5•5 years ago
|
||
Canceling ni, cmills mentioned that language updates are already happening.
Comment 6•5 years ago
|
||
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.
Reporter | ||
Comment 7•5 years ago
|
||
I'm open to suggestions for rewording, though.
The new verb that landed with this patch is "ignore".
Comment 8•5 years ago
|
||
(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.
Reporter | ||
Comment 9•5 years ago
|
||
Right, we did mix those up in the issue name – my bad. "Ignore source" is the new name for "Blackbox source".
Assignee | ||
Comment 10•5 years ago
|
||
Oh shoot – that was my mistake in the commit message. Sorry for the confusion!
Comment 11•5 years ago
|
||
bugherder |
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 12•5 years ago
|
||
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).
Description
•