626142 - Fix plug-in dtd

Status: RESOLVED FIXED

(Camino Graveyard :: Plug-ins, defect)

Description

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

The plug-in blocklisting and disabling uses http://mxr.mozilla.org/mozilla1.9.2/source/toolkit/locales/en-US/chrome/mozapps/plugins/plugins.dtd which is full of incorrect and Firefox-specific strings.  (Even worse, they're not localizable :( ).

We'll have to fork the file and fix the strings.

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: non-localizable fix

This is at least *a* fix; it fixes the three strings we care about by forking both the DTD and the XBL, splitting the strings, and inserting the links.

It requires a new section on cbo's Setup page at http://caminobrowser.org/documentation/setup/#disable_plugin to explain disabled plug-ins.  (We may want to add a similar link to the blocked plug-ins message.)  We might also want to just change the link to "More info." or something, because space is often at a premium (see the first Java instance on http://browserspy.dk/java.php).

I'm still hopeful I might be able to figure out stringbundle-in-XBL to make this localizable, but this at least makes it useable.

Comment 2

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

(In reply to comment #1)
> I'm still hopeful I might be able to figure out stringbundle-in-XBL to make
> this localizable, but this at least makes it useable.

I'm significantly less hopeful now.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 504194 [details] [diff] [review]
non-localizable fix

We may still want to change the URL for disabled (to go to hidden prefs, since other than Java, disabling is all backed by hidden prefs), but for now this at least gets an improved version of this UI ready to land with bug 558432 and friends.

Comment 4

[Stuart Morgan]

Comment on attachment 504194 [details] [diff] [review]
non-localizable fix

sr=smorgan

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

http://hg.mozilla.org/camino/rev/b4d8934369ae with 'target="_blank"' added to the two links.

We'll track any other changes to this mess in new bugs.