Closed
Bug 402861
Opened 17 years ago
Closed 17 years ago
Use plugin instead of plug-in in UI and docs (similar to what FF does)
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: prometeo.bugs, Assigned: prometeo.bugs)
Details
Attachments
(1 file)
|
42.75 KB,
patch
|
kairo
:
review+
jst
:
review+
beltzner
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.8) Gecko/20061201 Firefox/2.0.0.8 (Ubuntu-feisty)
Build Identifier:
FF devs decided to use plugin (and plugins) throughout the UI and the help docs for version 2.0. I think SM should do the same, since even the new Add-On Manager uses that.
Reproducible: Always
| Assignee | ||
Comment 1•17 years ago
|
||
The other change I made is the removal of the reference to Netscape Plugin Finder Service, now called only Plugin Finder Service...
Attachment #287687 -
Flags: review?(kairo)
| Assignee | ||
Updated•17 years ago
|
Summary: Use plugin instead of plugin in UI and docs (similar to what FF does) → Use plugin instead of plug-in in UI and docs (similar to what FF does)
Updated•17 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•17 years ago
|
Assignee: general → prometeo.bugs
Comment 2•17 years ago
|
||
Comment on attachment 287687 [details] [diff] [review]
Proposed patch, fixing all of them at once
r=me from visual inspection for the suite/ parts, but the dom/ part (first file) technically needs a DOM peer to review it. This only achieves consistency with wording we use throughout toolkit and Firefox, so I hope that review should be no problem.
Attachment #287687 -
Flags: review?(kairo)
Attachment #287687 -
Flags: review?(jst)
Attachment #287687 -
Flags: review+
Comment 3•17 years ago
|
||
Comment on attachment 287687 [details] [diff] [review]
Proposed patch, fixing all of them at once
Fine with me, but given that this shows up in Firefox UI too, I'd like to have Beltzner sign off on this too.
Attachment #287687 -
Flags: review?(jst)
Attachment #287687 -
Flags: review?(beltzner)
Attachment #287687 -
Flags: review+
Comment 4•17 years ago
|
||
Comment on attachment 287687 [details] [diff] [review]
Proposed patch, fixing all of them at once
ui-r=beltzner
I notice that in my applications prefpane Quicktime is listed for a lot of things as "Quicktime Plug-in"; are we doing that, or is Quicktime?
Attachment #287687 -
Flags: review?(beltzner) → review+
Comment 5•17 years ago
|
||
Comment on attachment 287687 [details] [diff] [review]
Proposed patch, fixing all of them at once
Asking a1.9? for the dom/ (about:plugins) part - this is a spelling fix for the word "plugin" only
Attachment #287687 -
Flags: approval1.9?
Comment 6•17 years ago
|
||
Comment on attachment 287687 [details] [diff] [review]
Proposed patch, fixing all of them at once
a=beltzner
Attachment #287687 -
Flags: approval1.9? → approval1.9+
Comment 7•17 years ago
|
||
Checking in dom/locales/en-US/chrome/plugins.properties;
/cvsroot/mozilla/dom/locales/en-US/chrome/plugins.properties,v <-- plugins.properties
new revision: 1.2; previous revision: 1.1
done
Checking in suite/locales/en-US/profile/bookmarks.html;
/cvsroot/mozilla/suite/locales/en-US/profile/bookmarks.html,v <-- bookmarks.html
new revision: 1.22; previous revision: 1.21
done
Checking in suite/locales/en-US/chrome/common/utilityOverlay.dtd;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/utilityOverlay.dtd,v <-- utilityOverlay.dtd
new revision: 1.35; previous revision: 1.34
done
Checking in suite/locales/en-US/chrome/common/help/cs_nav_prefs_advanced.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/cs_nav_prefs_advanced.xhtml,v <-- cs_nav_prefs_advanced.xhtml
new revision: 1.52; previous revision: 1.51
done
Checking in suite/locales/en-US/chrome/common/help/cs_nav_prefs_navigator.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/cs_nav_prefs_navigator.xhtml,v <-- cs_nav_prefs_navigator.xhtml
new revision: 1.42; previous revision: 1.41
done
Checking in suite/locales/en-US/chrome/common/help/glossary.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/glossary.xhtml,v <-- glossary.xhtml
new revision: 1.48; previous revision: 1.47
done
Checking in suite/locales/en-US/chrome/common/help/help-glossary.rdf;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/help-glossary.rdf,v <-- help-glossary.rdf
new revision: 1.25; previous revision: 1.24
done
Checking in suite/locales/en-US/chrome/common/help/help-index1.rdf;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/help-index1.rdf,v <-- help-index1.rdf
new revision: 1.59; previous revision: 1.58
done
Checking in suite/locales/en-US/chrome/common/help/mail_help.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/mail_help.xhtml,v <-- mail_help.xhtml
new revision: 1.85; previous revision: 1.84
done
Checking in suite/locales/en-US/chrome/common/help/nav_help.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/nav_help.xhtml,v <-- nav_help.xhtml
new revision: 1.71; previous revision: 1.70
done
Checking in suite/locales/en-US/chrome/common/help/privacy_help.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/privacy_help.xhtml,v <-- privacy_help.xhtml
new revision: 1.49; previous revision: 1.48
done
Checking in suite/locales/en-US/chrome/common/help/suite-toc.rdf;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/suite-toc.rdf,v <-- suite-toc.rdf
new revision: 1.93; previous revision: 1.92
done
Checking in suite/locales/en-US/chrome/common/help/welcome_help.xhtml;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/help/welcome_help.xhtml,v <-- welcome_help.xhtml
new revision: 1.25; previous revision: 1.24
done
Checking in suite/locales/en-US/chrome/common/pref/pref-applications.dtd;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/pref/pref-applications.dtd,v <-- pref-applications.dtd
new revision: 1.17; previous revision: 1.16
done
Checking in suite/locales/en-US/chrome/common/pref/pref-scripts.dtd;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/pref/pref-scripts.dtd,v <-- pref-scripts.dtd
new revision: 1.15; previous revision: 1.14
done
Checking in suite/locales/en-US/chrome/common/pref/preftree.dtd;
/cvsroot/mozilla/suite/locales/en-US/chrome/common/pref/preftree.dtd,v <-- preftree.dtd
new revision: 1.57; previous revision: 1.56
done
Checking in suite/locales/en-US/chrome/mailnews/pref/mailPrefsOverlay.dtd;
/cvsroot/mozilla/suite/locales/en-US/chrome/mailnews/pref/mailPrefsOverlay.dtd,v <-- mailPrefsOverlay.dtd
new revision: 1.26; previous revision: 1.25
done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•