Closed Bug 176542 Opened 22 years ago Closed 20 years ago

Fix plugin error dialog

Categories

(Core Graveyard :: Plug-ins, defect)

x86
Windows XP
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 188557

People

(Reporter: goodvin, Assigned: timeless)

Details

Attachments

(2 files, 1 obsolete file)

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.2b) Gecko/20021020 Phoenix/0.3 I got an Illeagal Operation in Plugin popup advising me to close and restart Navigator. Should it be Phoenix instead? See attached image.
Status: UNCONFIRMED → NEW
Ever confirmed: true
This will need to be fixed in Mozilla files so it's probably not worth the effort until the tree opens for 1.3a av, can we change this to "restart the browser" or use some brandShortName or something like that? timeless, didn't you fix this for us a long time ago? has it regressed?
Severity: minor → trivial
that was for the string 'netscape' which is different from 'navigator', sorry :), also the crash string is considerably newer than my fix.
Attached patch patch v.1 (obsolete) (deleted) — Splinter Review
Easy fix, using generic word 'browser'.
Maybe we should take this opportunity and move the dialog wording to some less obscure place?
sure
Assignee: blaker → timeless
Component: General → Plug-ins
Product: Phoenix → Browser
QA Contact: asa → shrir
Summary: Wrong application name in Illegal Operation in Plugin popup → Fix plugin error dialog
Version: unspecified → other
Attachment #104521 - Attachment is obsolete: true
Comment on attachment 104541 [details] [diff] [review] correct file, mostly improved text r=av. It would also be nice if we get rid of this 'Plugin Downloader Plugin'. I don't think we use this term anywhere. 'Default Plug-in' is what people normally use. Also, hyphenated spelling of the word 'plug-in' is used throughout the application.
Attachment #104541 - Flags: review+
Comment on attachment 104541 [details] [diff] [review] correct file, mostly improved text +noDefaultPluginCheckboxMessage=I hate plugins. Is this really necessary? In fact, most of the changes to these locale strings seem to make them somewhat unprofessional and less useful...
Was the patch checked in? Since the patch is present, I don’t want it to fall of the radar.
I'll dup it against bug 188557 which got fixed today. *** This bug has been marked as a duplicate of 188557 ***
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → DUPLICATE
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: