Minor edits

- fix coding style
- fix type hints
- fix/add links
- rewording

Author: cdoublev

__tests__/stylesheet.js

@@ -2322,12 +2322,17 @@ describe('CSSViewTransitionRule', () => {
 })
 
 /**
- * The following does not conform to the specification, which expects the setter
- * to do nothing and requires implementing it in each CSSRule child classes.
- *
  * @see {@link https://github.com/w3c/csswg-drafts/issues/8778}
+ *
+ * The specification wants the setter of CSSRule.cssText to do nothing, which
+ * requires implementing it in every CSSRule child class, as one cannot set a
+ * property when its getter is defined on the child class and its setter is
+ * defined on the parent class.
+ *
+ * Instead, CSSRule.cssText is defined as read-only: in strict mode, it throws
+ * an error.
  */
-it('throws an error when setting CSSRule.cssText', () => {
+test('Setting CSSRule.cssText does nothing', () => {
     const { cssRules: [cssRule] } = createStyleSheet('style {}')
     cssRule.cssText = 'override {}'
     expect(cssRule.cssText).toBe('style {}')

lib/cssom/CSSImportRule-impl.js

@@ -48,7 +48,6 @@ class CSSImportRuleImpl extends CSSRuleImpl {
     /**
      * @returns {string}
      * @see {@link https://drafts.csswg.org/cssom-1/#dom-cssrule-csstext}
-     * @see {@link https://github.com/w3c/csswg-drafts/issues/4828#issuecomment-1059910651}
      */
     get cssText() {
         const { _conditions, href, layerName, media } = this

lib/cssom/CSSRule.webidl

@@ -4,6 +4,7 @@
 // https://drafts.csswg.org/css-conditional-3/#extensions-to-cssrule-interface
 // https://drafts.csswg.org/css-counter-styles-3/#extensions-to-cssrule-interface
 // https://drafts.csswg.org/css-fonts-4/#om-fontfeaturevalues
+// https://github.com/w3c/csswg-drafts/issues/8778
 [Exposed=Window]
 interface CSSRule {
     const unsigned short STYLE_RULE = 1;

lib/parse/configure.js

@@ -2,9 +2,12 @@
 /**
  * @param {object} node
  * @returns {object}
+ * @see {@link https://drafts.csswg.org/css-cascade-5/#typedef-layer-name}
  * @see {@link https://drafts.csswg.org/css-mixins-1/#typedef-syntax-component}
  * @see {@link https://drafts.csswg.org/css-page-3/#syntax-page-selector}
  * @see {@link https://drafts.csswg.org/css-syntax-3/#urange-syntax}
+ * @see {@link https://drafts.csswg.org/css-values-5/#typedef-attr-name}
+ * @see {@link https://drafts.csswg.org/css-view-transitions-2/#typedef-pt-name-and-class-selector}
  * @see {@link https://drafts.csswg.org/mediaqueries-5/#mq-syntax}
  * @see {@link https://drafts.csswg.org/selectors-4/#white-space}
  *

lib/parse/parser.js

@@ -627,7 +627,7 @@ function parseCSSValue(input, definition, context, strategy = 'backtrack') {
 /**
  * @param {object} declaration
  * @param {object} [context]
- * @returns {object}
+ * @returns {object|null}
  */
 function parseCSSDeclaration(declaration, context = createContext()) {
     if (declaration.important && !context.rule.definition.cascading) {