@ons/design-system 53.1.1 → 55.0.0

package/components/checkboxes/_checkbox.scss

@@ -9,12 +9,11 @@ $checkbox-padding: 11px;
 
   &__input {
     appearance: none;
-    background: url(#{$static}/img/icons--check.svg) no-repeat center center;
-    background-color: $color-white;
-    background-size: 0;
-    border: 2px solid $color-input;
+    background-color: $color-input-bg;
+    border: 2px solid $color-input-border;
     border-radius: 0.2rem;
     box-sizing: border-box;
+    cursor: pointer;
     height: $checkbox-input-width;
     left: $checkbox-padding;
     position: absolute;
@@ -22,13 +21,30 @@ $checkbox-padding: 11px;
     width: $checkbox-input-width;
     z-index: 1;
 
+    // Check icon
+    &::after {
+      border: solid $color-input-border;
+      border-radius: 1px;
+      border-top-color: $color-input-bg;
+      border-width: 0 0 3px 3px;
+      box-sizing: border-box;
+      content: '';
+      height: 7px;
+      left: 2px;
+      opacity: 0;
+      position: absolute;
+      top: 4px;
+      transform: rotate(-45deg);
+      width: 14px;
+    }
+
     &:focus,
     &:checked {
       outline: none;
     }
 
-    &:checked {
-      background-size: 14px;
+    &:checked::after {
+      opacity: 1;
     }
 
     &:disabled {
@@ -36,9 +52,24 @@ $checkbox-padding: 11px;
       cursor: not-allowed;
     }
 
-    &:disabled + label {
-      color: $color-grey-35;
+    &:disabled:checked::after {
+      border-color: $color-border-disabled;
+    }
+
+    &:disabled + .ons-checkbox__label,
+    &:disabled:checked + .ons-checkbox__label {
+      color: $color-border-disabled;
       cursor: not-allowed;
+
+      &::before {
+        border: 1px solid $color-border-disabled;
+      }
+    }
+
+    &:disabled:checked + .ons-checkbox__label {
+      &::before {
+        box-shadow: 0 0 0 1px $color-border-disabled;
+      }
     }
   }
 
@@ -47,9 +78,9 @@ $checkbox-padding: 11px;
       padding: 0 0 0 1.85rem;
 
       &::before {
-        background: none;
-        border: none;
-        box-shadow: none;
+        background: none !important;
+        border: none !important;
+        box-shadow: none !important;
       }
 
       & > .ons-checkbox__label--with-description {
@@ -69,11 +100,12 @@ $checkbox-padding: 11px;
           background: none;
           border: none;
           box-shadow: none;
+          outline: none;
         }
       }
 
       &:focus {
-        box-shadow: 0 0 0 3px $color-focus;
+        @extend %ons-input-focus;
       }
     }
 
@@ -101,7 +133,7 @@ $checkbox-padding: 11px;
       }
 
       &:focus {
-        box-shadow: 0 0 0 3px $color-focus;
+        @extend %ons-input-focus;
       }
 
       & + .ons-checkbox__label {
@@ -122,7 +154,7 @@ $checkbox-padding: 11px;
 
     &::before {
       background: $color-white;
-      border: 1px solid $color-input;
+      border: 1px solid $color-input-border;
       border-radius: 3px;
       bottom: 0;
       content: '';
@@ -152,7 +184,8 @@ $checkbox-padding: 11px;
 
   &__input:checked + &__label::before {
     background: $color-grey-5;
-    box-shadow: 0 0 0 1px $color-input;
+    box-shadow: 0 0 0 1px $color-input-border;
+    outline: 1px solid transparent; // Add transparent outline because Windows High Contrast Mode doesn't show box-shadows
   }
 
   .ons-panel--error .ons-radio__input:checked ~ &__other > .ons-input--text:required:not(:focus) {
@@ -176,10 +209,10 @@ $checkbox-padding: 11px;
   &--toggle & {
     &__input {
       left: 0;
-      top: 0.1rem;
+      top: 0.2rem;
 
       &:focus {
-        box-shadow: 0 0 0 3px $color-focus;
+        @extend %ons-input-focus;
       }
     }
 

package/components/collapsible/_collapsible.scss

@@ -1,106 +1,82 @@
 $collapsible-caret-width: 1.5rem;
 
 .ons-collapsible {
-  &--initialised & {
-    &__heading {
-      color: $color-text-link;
-      cursor: pointer;
-      display: inline-block;
-      outline: none;
-      padding: 0 0 0 $collapsible-caret-width;
-      pointer-events: initial;
-      position: relative;
-
-      &::marker {
-        display: none;
-      }
+  &__heading {
+    color: $color-text-link;
+    cursor: pointer;
+    display: inline-block;
+    outline: none;
+    padding: 0 0 0 $collapsible-caret-width;
+    pointer-events: initial;
+    position: relative;
+
+    &::marker,
+    &::-webkit-details-marker {
+      display: none;
+    }
 
-      &:focus {
-        .ons-collapsible__title {
-          @extend %a-focus;
-          // extend collapsible focus background behind caret
-          margin-left: -$collapsible-caret-width;
-          padding-left: $collapsible-caret-width;
-        }
-        .ons-collapsible__icon .ons-svg-icon {
-          fill: $color-text-link-focus;
-        }
+    &:focus {
+      .ons-collapsible__title {
+        @extend %a-focus;
+        // extend collapsible focus background behind caret
+        margin-left: -$collapsible-caret-width;
+        padding-left: $collapsible-caret-width;
       }
-
-      &:hover:not(:focus) {
-        color: $color-text-link-hover;
-        .ons-collapsible__icon {
-          fill: $color-text-link-hover;
-        }
-        .ons-collapsible__title {
-          text-decoration: underline solid $color-text-link-hover 2px;
-        }
+      .ons-collapsible__icon .ons-svg-icon {
+        fill: $color-text-link-focus;
       }
     }
 
-    &__icon {
-      display: inline-block;
-      fill: $color-text-link;
-      height: $collapsible-caret-width;
-      left: -0.15rem;
-      position: absolute;
-      top: -0.25rem;
-      width: $collapsible-caret-width;
+    &:hover:not(:focus) {
+      color: $color-text-link-hover;
+      .ons-collapsible__icon {
+        fill: $color-text-link-hover;
+      }
+      .ons-collapsible__title {
+        text-decoration: underline solid $color-text-link-hover 2px;
+      }
     }
+  }
 
-    &__title {
-      font-size: 1rem;
-      font-weight: $font-weight-bold;
-      margin-bottom: 0;
-      text-decoration: underline;
-      text-underline-position: under;
-      transform: translateY(-1px);
-    }
+  &__icon {
+    display: inline-block;
+    fill: $color-text-link;
+    height: $collapsible-caret-width;
+    left: -0.15rem;
+    position: absolute;
+    top: -0.2rem;
+    width: $collapsible-caret-width;
+  }
 
-    &__content {
-      display: none;
-    }
+  &__title {
+    display: inline-block;
+    font-size: 1rem;
+    font-weight: $font-weight-bold;
+    margin-bottom: 0;
+    text-underline-position: under;
+    transform: translateY(-1px);
   }
 
-  &__icon {
-    display: none;
+  &__content {
+    border-left: 4px solid $color-borders-indent;
+    display: block;
+    margin: 1rem 0 0;
+    padding: 0 0 0 1.3em;
   }
 
-  &--open & {
+  &[open] & {
     &__icon {
       left: -0.1rem;
-      top: 0.1rem;
+      top: 0.2rem;
       transform: rotate(90deg);
     }
-
-    &__content {
-      border-left: 4px solid $color-borders-indent;
-      display: block;
-      margin: 1rem 0 0;
-      padding: 0 0 0 1.3em;
-
-      .ons-btn {
-        margin-bottom: 0.5rem;
-      }
-    }
-  }
-
-  &--accordion {
-    margin: 0;
-    &.ons-collapsible--open {
-      .ons-svg-icon {
-        left: 0.4rem;
-        position: absolute;
-        top: 0.25rem;
-      }
-    }
   }
 
   &--accordion & {
     &__heading {
       border-top: 1px solid $color-borders;
       margin: 0;
-      padding-bottom: 1rem;
+      padding-bottom: 0.9rem;
       padding-top: 1rem;
       width: 100%;
 
@@ -115,10 +91,6 @@ $collapsible-caret-width: 1.5rem;
 
     &__icon {
       top: 0.8rem;
-
-      @include mq(s) {
-        top: 1.15rem;
-      }
     }
 
     &__controls {
@@ -138,10 +110,12 @@ $collapsible-caret-width: 1.5rem;
       padding: 0;
     }
   }
+}
 
-  &--simple & {
-    &__icon {
-      top: 0.8rem;
+.ons-collapsible--accordion {
+  &[open] {
+    .ons-collapsible__icon {
+      top: 1.2rem;
     }
   }
 }

package/components/collapsible/_macro.njk

@@ -1,21 +1,16 @@
 {% macro onsCollapsible(params) %}
-    {% from "components/button/_macro.njk" import onsButton %}
-
-    <div
+    <details
         id="{{ params.id }}"
-        class="ons-collapsible ons-js-collapsible{% if params.isAccordion is defined and params.isAccordion %} ons-collapsible--accordion{% endif %}{% if params.classes is defined and params.classes %} {{ params.classes }}{% endif %}{% if params.variants is defined and params.variants %}{% if params.variants is not string %}{% for variant in params.variants %} ons-collapsible--{{ variant }}{% endfor %}{% else %} ons-collapsible--{{ params.variants }}{% endif %}{% endif %}"
-        {% if params.button is defined and params.button and params.button.close is defined and params.button.close %} data-btn-close="{{ params.button.close }}"{% endif %}
+        class="ons-collapsible ons-js-collapsible{% if params.isAccordion is defined and params.isAccordion %} ons-collapsible--accordion{% endif %}{% if params.classes is defined and params.classes %} {{ params.classes }}{% endif %}"
         {% if params.group is defined and params.group %} data-group="{{ params.group }}"{% endif %}
         {% if params.attributes is defined and params.attributes %}{% for attribute, value in (params.attributes.items() if params.attributes is mapping and params.attributes.items else params.attributes) %}{{ attribute }}{% if value is defined and value %}="{{ value }}"{% endif %} {% endfor %}{% endif %}
         {% if params.saveState is defined and params.saveState %} data-save-state="true"{% endif %}
         {% if params.open is defined and params.open %} data-open="true"{% endif %}
     >
-        <div
+        <summary
             class="ons-collapsible__heading ons-js-collapsible-heading"
             {% if params.headingAttributes is defined and params.headingAttributes %}{% for attribute, value in (params.headingAttributes.items() if params.headingAttributes is mapping and params.headingAttributes.items else params.headingAttributes) %}{{ attribute }}{% if value is defined and value %}="{{ value }}"{% endif %} {% endfor %}{% endif %}
         >
-            {# Required to use display flex on Safari as it's not currently supported: https://bugs.webkit.org/show_bug.cgi?id=167111 #}
-            <div class="ons-collapsible__controls">
                 {% set titleTag = params.titleTag | default("h2") %}
                 <{{ titleTag }} class="ons-collapsible__title">{{ params.title }}</{{ titleTag }}>
                 <span class="ons-collapsible__icon">
@@ -25,39 +20,11 @@
                             "iconType": "chevron"
                         })
                     }}</span>
-                {% if params.button is defined and params.button and params.button.open is defined and params.button.open and params.isAccordion is defined and params.isAccordion %}
-                    {{
-                        onsButton({
-                            "type": "button",
-                            "text": params.button.open,
-                            "classes": "ons-collapsible__btn ons-js-collapsible-button ons-u-d-no ons-u-d-no@xxs@s",
-                            "innerClasses": "ons-js-collapsible-button-inner",
-                            "variants": ["secondary", "small"],
-                            "attributes": params.button.attributes
-                        })
-                    }}
-                {% endif %}
-            </div>
-        </div>
+        </summary>
         <div id="{{ params.id }}-content" class="ons-collapsible__content ons-js-collapsible-content"
             {% if params.contentAttributes is defined and params.contentAttributes %}{% for attribute, value in (params.contentAttributes.items() if params.contentAttributes is mapping and params.contentAttributes.items else params.contentAttributes) %}{{ attribute }}{% if value is defined and value %}="{{ value }}"{% endif %} {% endfor %}{% endif %}
         >
-            {% if params.isAccordion is defined and params.isAccordion %}
-                {{ params.content | safe }}{{ caller() if caller }}
-            {% else %}
-                {{ params.content | safe }}{{ caller() if caller }}
-                {{
-                    onsButton({
-                        "type": "button",
-                        "text": params.button.close | default ('Hide this'),
-                        "buttonContext": (params.button.context | default(params.title)) + " " + params.button.contextSuffix | default("content"),
-                        "classes": "ons-js-collapsible-button ons-u-d-no " + (params.button.classes | default("ons-btn--secondary")),
-                        "innerClasses": "ons-js-collapsible-button-inner",
-                        "variants": "small",
-                        "attributes": params.button.attributes
-                    })
-                }}
-            {% endif %}
+            {{ params.content | safe }}{{ caller() if caller }}
         </div>
-    </div>
+    </details>
 {% endmacro %}

package/components/collapsible/_macro.spec.js

@@ -37,18 +37,6 @@ describe('macro: collapsible', () => {
     expect($('.ons-collapsible').hasClass('another-extra-class')).toBe(true);
   });
 
-  it('has provided variant style classes', () => {
-    const $ = cheerio.load(
-      renderComponent('collapsible', {
-        ...EXAMPLE_COLLAPSIBLE_BASIC,
-        variants: ['variant-a', 'variant-b'],
-      }),
-    );
-
-    expect($('.ons-collapsible--variant-a').length).toBe(1);
-    expect($('.ons-collapsible--variant-b').length).toBe(1);
-  });
-
   it('has provided title text', () => {
     const $ = cheerio.load(renderComponent('collapsible', EXAMPLE_COLLAPSIBLE_BASIC));
 
@@ -152,47 +140,6 @@ describe('macro: collapsible', () => {
     expect(iconsSpy.occurrences[0].iconType).toBe('chevron');
   });
 
-  it('has the expected button output', () => {
-    const faker = templateFaker();
-    const buttonSpy = faker.spy('button');
-
-    faker.renderComponent('collapsible', {
-      ...EXAMPLE_COLLAPSIBLE_BASIC,
-      button: {
-        close: 'Close this',
-        attributes: {
-          a: 123,
-          b: 456,
-        },
-      },
-    });
-
-    expect(buttonSpy.occurrences[0]).toEqual({
-      text: 'Close this',
-      type: 'button',
-      buttonContext: 'Title for collapsible content',
-      classes: 'ons-js-collapsible-button ons-u-d-no ons-btn--secondary',
-      innerClasses: 'ons-js-collapsible-button-inner',
-      variants: 'small',
-      attributes: {
-        a: 123,
-        b: 456,
-      },
-    });
-  });
-
-  it('has the expected data attribute when `button` is provided', () => {
-    const $ = cheerio.load(
-      renderComponent('collapsible', {
-        ...EXAMPLE_COLLAPSIBLE_BASIC,
-        button: {
-          close: 'Close this',
-        },
-      }),
-    );
-    expect($('.ons-collapsible').attr('data-btn-close')).toBe('Close this');
-  });
-
   it('calls with content', () => {
     const $ = cheerio.load(renderComponent('collapsible', { EXAMPLE_COLLAPSIBLE_BASIC }, 'Example content...'));
 

package/components/collapsible/collapsible.dom.js

@@ -2,20 +2,11 @@ import domready from '../../js/domready';
 
 async function initialiseCollapsibles() {
   const collapsibleComponents = [...document.querySelectorAll('.ons-js-collapsible')];
+  const accordionComponents = [...document.querySelectorAll('.ons-js-accordion')];
 
-  if (collapsibleComponents.length) {
-    const toggleAllButtons = [...document.querySelectorAll('.ons-js-collapsible-all')];
-
+  if (collapsibleComponents.length && !accordionComponents.length) {
     const Collapsible = (await import('./collapsible')).default;
-    const collapsibles = collapsibleComponents.map(element => new Collapsible(element));
-
-    if (toggleAllButtons.length) {
-      const CollapsibleGroup = (await import('./collapsible.group')).default;
-
-      toggleAllButtons.forEach(button => {
-        new CollapsibleGroup(button, collapsibles);
-      });
-    }
+    collapsibleComponents.map(element => new Collapsible(element));
   }
 }
 

package/components/collapsible/collapsible.js

@@ -2,79 +2,38 @@ export default class Collapsible {
   constructor(collapsibleElement) {
     this.saveState = collapsibleElement.getAttribute('data-save-state') === 'true';
     this.open = collapsibleElement.getAttribute('data-open') === 'true';
-
-    // State
     this.group = collapsibleElement.getAttribute('data-group');
-    this.isAccordion = collapsibleElement.classList.contains('ons-collapsible--accordion');
-    this.isSimpleAccordion = collapsibleElement.classList.contains('ons-collapsible--simple');
 
     // Elements
     this.collapsible = collapsibleElement;
     this.collapsibleHeader = this.collapsible.querySelector('.ons-js-collapsible-heading');
-    this.content = this.collapsible.querySelector('.ons-js-collapsible-content');
-    this.button = this.collapsible.querySelector('.ons-js-collapsible-button');
-    this.buttonInner = this.collapsible.querySelector('.ons-js-collapsible-button-inner');
 
     // Initialise
     const collapsibleId = collapsibleElement.getAttribute('id');
 
-    if (this.button) {
-      this.button.addEventListener('click', this.toggle.bind(this));
-      this.button.setAttribute('aria-controls', collapsibleId);
-      this.button.classList.remove('ons-u-d-no');
-      this.buttonOpen = this.buttonInner.innerHTML.trim();
-      this.closeButton = this.collapsible.getAttribute('data-btn-close') || this.buttonOpen;
-    }
-
-    this.collapsible.setAttribute('role', 'group');
-    this.collapsibleHeader.setAttribute('role', 'link');
-    this.collapsibleHeader.setAttribute('aria-controls', collapsibleId);
-
-    if (!this.isAccordion || this.isSimpleAccordion) {
-      this.collapsibleHeader.setAttribute('tabindex', 0);
-    }
     if (localStorage.getItem(collapsibleId) || this.open) {
       this.setOpen(true);
-    } else {
-      this.setOpen(false);
+      this.collapsible['setAttribute']('open', '');
     }
 
     this.collapsibleHeader.addEventListener('click', this.toggle.bind(this));
     this.collapsibleHeader.addEventListener('keydown', this.keyboardInteraction.bind(this));
-
-    this.collapsible.classList.add('ons-collapsible--initialised');
   }
 
   toggle(event) {
     event.preventDefault();
-
-    // Storing previous timestamp and checking against the current event's timestamp prevents event propagation
-    // to collapsibleHeader element when the button is in the collapsibleHeader, but does not prevent event propagation to the body for analytics
-    if (this.previousTimestamp !== event.timeStamp) {
-      this.previousTimestamp = event.timeStamp;
-      this.setOpen(!this.isOpen);
-    }
+    this.setOpen(!this.isOpen);
   }
 
   setOpen(open) {
     if (open !== this.isOpen) {
-      const openAttribute = open ? 'set' : 'remove';
-      const cls = open ? 'add' : 'remove';
       const action = open ? 'Open' : 'Close';
+      const openAttribute = open ? 'set' : 'remove';
 
       this.isOpen = open;
       this.collapsible[`${openAttribute}Attribute`]('open', '');
-      this.collapsible.classList[cls]('ons-collapsible--open');
-      this.collapsibleHeader.setAttribute('aria-expanded', open);
-      this.content.setAttribute('aria-hidden', !open);
-
       this.collapsibleHeader.setAttribute('data-ga-action', `${action} panel`);
 
-      if (this.button) {
-        this.button.setAttribute('data-ga-action', `${action} panel`);
-        this.buttonInner.innerHTML = open ? this.closeButton : this.buttonOpen;
-      }
-
       if (this.onOpen && this.onClose) {
         if (open) {
           this.onOpen();
@@ -100,7 +59,6 @@ export default class Collapsible {
         event.preventDefault();
         event.stopPropagation();
 
-        // Show answer content
         this.toggle(event);
         break;
     }