Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-66111

Consider if these (some IE specific) selectors can be safely removed

    XMLWordPrintable

Details

    • Bug
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 3.5.7, 3.6.5, 3.7, 3.8
    • None
    • HTML and CSS, Themes
    • MOODLE_35_STABLE, MOODLE_36_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE

    Description

      While playing with MDL-66109, bumping stylelint (css linter) to newer versions, I'm getting these warnings:

      Running "stylelint:css" (stylelint) task
      >> admin/tool/lp/styles.css
      >>  201:5  ✖  Invalid value for `background-image`   csstree/validator
      >> 
      >> admin/tool/policy/styles.css
      >>  29:5  ✖  Can't parse value "alpha(opacity=20)"   csstree/validator
      >> 
      >> blocks/community/styles.css
      >>  303:5  ✖  Can't parse value "alpha(opacity=35)"   csstree/validator
      >> 
      >> question/type/ddmarker/styles.css
      >>  81:5  ✖  Can't parse value "alpha(opacity=60)"   csstree/validator
      Warning: Task "stylelint:css" failed. Used --force, continuing.
      

      Apart from the 1st case, that looks correct for me... and I'm not sure what's being reported there... the {{alpha(opacity=) seem to be something from IE8 times (from what I've found searching out there).

      So, for your consideration... it would be great to have stylelint (future versions) passing in advance. For the records... this is what I've used to get the warnings above:

      -    "stylelint": "8.0.0",
      -    "stylelint-checkstyle-formatter": "0.1.0",
      -    "stylelint-csstree-validator": "1.1.1",
      +    "stylelint": "10.1.0",
      +    "stylelint-checkstyle-formatter": "0.1.2",
      +    "stylelint-csstree-validator": "1.4.1",
      

      Ciao

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              stronk7 Eloy Lafuente (stronk7)
              Amy Groshek, David Scotson, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona), Huong Nguyen, Bas Brands, Mathew May
              Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: