Closed
Bug 346796
Opened 19 years ago
Closed 19 years ago
Change descriptions for DOM Inspector and Talkback to match new NSIS descriptions
Categories
(Core :: General, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.8.1beta2
People
(Reporter: reed, Assigned: reed)
Details
(Keywords: fixed1.8.1)
Attachments
(2 files, 1 obsolete file)
|
808 bytes,
patch
|
timeless
:
review+
beltzner
:
approval1.8.1+
|
Details | Diff | Splinter Review |
|
662 bytes,
patch
|
reed
:
review+
beltzner
:
approval1.8.1+
|
Details | Diff | Splinter Review |
The descriptions for DOM Inspector and Talkback have changed in the NSIS installer, so we want to use the same descriptions in the extensions themselves.
For DOM Inspector, "Inspects resources used by web pages and extensions."
For Talkback, "Sends information about crashes to Mozilla."
I'll attach a patch in a bit.
Updated•19 years ago
|
Component: General → Installer
Product: Core → Firefox
Target Milestone: mozilla1.8rc2 → ---
Version: 1.8 Branch → 2.0 Branch
Comment 1•19 years ago
|
||
This only affects Firefox since Thunderbird and Sunbird only ship QFA and this page is not shown during a custom install when WFA is the only optional component in the installer.
Comment 2•19 years ago
|
||
s/WFA/QFA/
| Assignee | ||
Comment 3•19 years ago
|
||
(strings pulled from bug 333160)
Umm, it seems Talkback's install.rdf is in the private repo. I'll need somebody with access to the repo to change Talkback's description.
:rs -- This bug is for the change in the extensions' install.rdf file, not the NSIS installer. Please stop moving it. :P
Assignee: reed → nobody
Component: Installer → General
Depends on: 333160
Product: Firefox → Core
Target Milestone: --- → mozilla1.8rc2
Version: 2.0 Branch → 1.8 Branch
| Assignee | ||
Comment 4•19 years ago
|
||
Swapped to bug 346798 for NSIS description changes.
New Talkback description is "Sends information about program crashes to Mozilla."
| Assignee | ||
Comment 5•19 years ago
|
||
Anybody with access to Talkback code, please see comment #3 but use the description in comment #4.
Flags: blocking1.8.1?
Comment 6•19 years ago
|
||
Since you came up with the text go ahead and change the installer strings for it as well. Also, there is no dependency between fixing accessibility of the installer's components page and changing the strings. :P
No longer depends on: 346798
| Assignee | ||
Comment 7•19 years ago
|
||
Change DOM Inspector's description to "Inspects resources used by web pages and extensions."
| Assignee | ||
Updated•19 years ago
|
Target Milestone: mozilla1.8rc2 → mozilla1.8.1beta2
for the record, this is entirely wrong.
DOMI inspects Documents, not Resources (it does not inspect CSS, JS, Properties, DTDs, or even Resource Description Format files [RDF]) nor Pages (as the current description incorrectly states. Who ever heard of a XML or XUL page? XML, HTML, XUL documents all exist and web pages, but not even HTML pages).
And this description happens to preclude the application chrome itself, which is generally what it inspects when it helps extension developers develop their extensions. it generally does not inspect individual extensions. certainly if you inspect noscript or the beeroclock extension you're not inspecting the extension, you're inspecting the application.
Since people are going to ignore me anyway, I'm including for posterity or whatever my draft which will be ignored.
DOMI enables developers of web pages and extensions to inspect properties of their rendered documents and test changes to them.
I'm sorry about the word "rendered", beltzner complained about the word "live".
Unfortunately for me, my safari (1.3.2 v312.6) has "Show DOM Tree" and "Show Render Tree". Actually, afaik DOMI is capable of expressing the equivalent of what Safari has in show render tree.
Goals for a description:
1. it should indicate that it will help developers of web pages and extensions developer whatever they're developing
2. to some extent it should indicate what it actually can look at (which is document object models, how this is described is sort of left to the author's discretion as long as it doesn't violate any other constraint)
Nongoals for a description:
a. it should not imply that you can't inspect the "browser" or whatever application is hosting domi.
b. it should not imply that you can use it to inspect the contents of other running applications (e.g. windows explorer, internet explorer, safari) - note that some applications *can* do this, I have one called Microsoft Script Debugger, it's an out of process application which hooks other processes [something we'll want domi to do, but which it can't do today]
c. it should not imply that you can inspect file formats that domi can't handle. this is basically all file formats: css, jpg, gif, png, js, properties, rdf, xbl, dtd, ....
d. it should not imply that it can inspect resources. as with file formats, it doesn't and can't.
Comment 9•19 years ago
|
||
mconnor and I just realized that since we're keeping DOMi as app-managed, it actually won't be upgraded to the larger set of web tools, but instead will be phased out when we get the better Web Tools extension (ah, to the glorious future!)
So. This means two things:
1. We can call it "DOM Inspector". (cue applause)
2. We can describe it as: "Inspects the structure and properties of a window and its contents."
I'm going with this naming because:
- short & sweet, like we did with QFA
- verb phrase that describes what it does, not who its for and why it's a pancea for all their woes
- removes implications that timeless felt were evil
Let the flaming over this non-issue begin!
| Assignee | ||
Comment 10•19 years ago
|
||
Use "Inspects the structure and properties of a window and its contents."
Attachment #231549 -
Attachment is obsolete: true
Attachment #231659 -
Flags: review?(timeless)
Attachment #231549 -
Flags: review?(timeless)
Attachment #231659 -
Flags: review?(timeless) → review+
| Assignee | ||
Updated•19 years ago
|
Whiteboard: [checkin needed]
| Assignee | ||
Comment 11•19 years ago
|
||
Comment on attachment 231659 [details] [diff] [review]
Take two - v2
Requesting a1.8.1. This just changes the description of the DOM Inspector extension.
Attachment #231659 -
Flags: approval1.8.1?
Comment 12•19 years ago
|
||
Comment on attachment 231659 [details] [diff] [review]
Take two - v2
a=drivers. Please land on the MOZILLA_1_8_BRANCH.
Attachment #231659 -
Flags: approval1.8.1? → approval1.8.1+
| Assignee | ||
Updated•19 years ago
|
Whiteboard: [checkin needed] → [checkin needed] [checkin needed (1.8 branch)]
Updated•19 years ago
|
Flags: blocking1.8.1?
Comment 13•19 years ago
|
||
Attachment #231673 -
Flags: review?(reed)
| Assignee | ||
Comment 14•19 years ago
|
||
Comment on attachment 231673 [details] [diff] [review]
install.rdf changes for talkback
r=reed, thanks!
Requesting a1.8.1 for this small em:description change for Talkback.
Attachment #231673 -
Flags: review?(reed)
Attachment #231673 -
Flags: review+
Attachment #231673 -
Flags: approval1.8.1?
Comment 15•19 years ago
|
||
Comment on attachment 231673 [details] [diff] [review]
install.rdf changes for talkback
a=drivers. Allons-y sur le MOZILLA_1_8_BRANCH.
Attachment #231673 -
Flags: approval1.8.1? → approval1.8.1+
Comment 16•19 years ago
|
||
Landed install.rdf patch on talkback trunk and branch.
Comment 17•19 years ago
|
||
DOMi patch checked in on MOZILLA_1_8_BRANCH and trunk.
-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed] [checkin needed (1.8 branch)]
| Assignee | ||
Updated•19 years ago
|
Keywords: fixed1.8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•