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

The current core php-css-parser prefixing library does not support sass syntax "@supports"

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.5
    • Fix Version/s: 3.5.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      Before testing

      1. Log in as admin
      2. Make sure you're using the Boost theme
      3. Purge all of the caches (Site administration > Development > purge caches) to ensure the CSS is rebuilt

      Test

      1. Login as teacher.
      2. Create or navigate to a course.
      3. Add a label.
      4. Open the testcarousel.html file in any code editor and copy the file content.
      5. In the label Atto editor click the HTML mode button </>.
      6. Paste the carousel content into Atto.
      7. Save changes and navigate back to the course page.
      8. Verify your new carousel is functioning and you can side slides transitioning (moving).
      Show
      Before testing Log in as admin Make sure you're using the Boost theme Purge all of the caches (Site administration > Development > purge caches) to ensure the CSS is rebuilt Test Login as teacher. Create or navigate to a course. Add a label. Open the testcarousel.html file in any code editor and copy the file content. In the label Atto editor click the HTML mode button </>. Paste the carousel content into Atto. Save changes and navigate back to the course page. Verify your new carousel is functioning and you can side slides transitioning (moving).
    • Affected Branches:
      MOODLE_35_STABLE
    • Fixed Branches:
      MOODLE_35_STABLE
    • Pull from Repository:
    • Pull 3.5 Branch:
    • Pull Master Branch:
      MDL-61515-master

      Description

      When updating bootst to use Bootstrap 4 stable the lib/php-scss-parser used for css prefixing breaks on lines using @supports

       

       

        Attachments

          Activity

            People

            Assignee:
            basbrands Bas Brands
            Reporter:
            basbrands Bas Brands
            Peer reviewer:
            Ryan Wyllie
            Integrator:
            Andrew Nicols
            Tester:
            Janelle Barcega
            Participants:
            Component watchers:
            Bas Brands
            Votes:
            1 Vote for this issue
            Watchers:
            9 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Sep/18

                Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 25 minutes
                25m