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

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

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.5.7, 3.6.5, 3.7, 3.8
    • Fix Version/s: None
    • Component/s: HTML and CSS, Themes
    • Labels:
    • Affected Branches:
      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

              • Assignee:
                Unassigned
                Reporter:
                stronk7 Eloy Lafuente (stronk7)
                Participants:
                Component watchers:
                Amy Groshek, David Scotson, Amaia Anabitarte, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón, Bas Brands, Amaia Anabitarte, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: