307286 - Devmo JS and CSS errors to check and fix

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: General, defect)

Description

[Deb Richardson [:dria] (plz NEEDINFO)]

From an email...

"I happened to visit DevMo while doing some extension debugging, so I had the
JavaScript Console open with strict mode enabled (javascript.options.strict =
true). Dozens of warnings and a few errors show up during a page load. I don't
see any critical problems, but they should probably be fixed at some point.

A few examples:

* Error: Selector expected.  Ruleset ignored due to bad selector.
Source File: http://developer.mozilla.org/css/wiki.css
Line: 672

* Error: Unknown property '_width'.  Declaration dropped.
Source File: http://developer.mozilla.org/css/base.css
Line: 225

* Warning: function diffcheck does not always return a value
Source File: http://developer.mozilla.org/en/docs/skins/common/wikibits.js
Line: 107

* Warning: variable element hides argument
Source File: http://developer.mozilla.org/en/docs/skins/devmo/prototype.js
Line: 931, Column: 8

(Most of the warning look like the last two)"

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

(In reply to comment #0)
> * Error: Selector expected.  Ruleset ignored due to bad selector.
> Source File: http://developer.mozilla.org/css/wiki.css
> Line: 672

Extra "}" that should be removed.

> * Error: Unknown property '_width'.  Declaration dropped.
> Source File: http://developer.mozilla.org/css/base.css
> Line: 225

This is a common hack for IE, so it's probably intended.

Comment 2

[Eric Shepherd [:sheppy]]

invalid with deki switch