fix: remove unused d2l-navigation-button-close (#153)

README.md

@@ -63,7 +63,6 @@ Please note that overridding the `left` slot will prevent the Back link from dis
  </div>
  <div slot="right">
  <d2l-navigation-button text="A button">One Button</d2l-navigation-button>
- <d2l-navigation-button-close></d2l-navigation-button-close>
  <d2l-navigation-button text="Another button">Two Button</d2l-navigation-button>
  </div>
 </d2l-navigation-immersive>

demo/button-link.html

@@ -6,7 +6,6 @@
  import '@brightspace-ui/core/components/demo/demo-page.js';
  import '@brightspace-ui/core/components/icons/icon.js';
  import '../d2l-navigation-button.js';
- import '../d2l-navigation-button-close.js';
  import '../d2l-navigation-button-icon.js';
  import '../d2l-navigation-button-notification-icon.js';
  import '../d2l-navigation-link.js';
@@ -55,13 +54,6 @@ <h2>d2l-navigation-button-icon / d2l-navigation-link-icon</h2>
  </div>
  </d2l-demo-snippet>
 
- <h2>d2l-navigation-button-close</h2>
- <d2l-demo-snippet>
- <div class="wrapper">
- <d2l-navigation-button-close></d2l-navigation-button-close>
- </div>
- </d2l-demo-snippet>
-
  <h2>d2l-navigation-button-notification-icon</h2>
  <d2l-demo-snippet>
  <div class="wrapper">
@@ -107,7 +99,6 @@ <h2>Combined</h2>
  Settings
  </d2l-navigation-button>
  <d2l-navigation-button-notification-icon notification icon="tier3:notification-bell" text="Notification on" id="notified" notification-text="You have new notifications"></d2l-navigation-button-notification-icon>
- <d2l-navigation-button-close></d2l-navigation-button-close>
  </div>
  </d2l-demo-snippet>
 

demo/immersive-fullscreen.html

@@ -6,7 +6,6 @@
  import '@brightspace-ui/core/components/demo/demo-page.js';
  import '../components/d2l-navigation-iterator/d2l-navigation-iterator.js';
  import '../d2l-navigation-button.js';
- import '../d2l-navigation-button-close.js';
  import '../d2l-navigation-immersive.js';
  </script>
  <title>d2l-navigation-immersive demo</title>

demo/immersive-normal.html

@@ -5,7 +5,6 @@
  <script type="module">
  import '@brightspace-ui/core/components/demo/demo-page.js';
  import '../d2l-navigation-button.js';
- import '../d2l-navigation-button-close.js';
  import '../d2l-navigation-immersive.js';
  </script>
  <title>d2l-navigation-immersive demo</title>
@@ -30,7 +29,6 @@
  </div>
  <div slot="right">
  <d2l-navigation-button text="A button">One Button</d2l-navigation-button>
- <d2l-navigation-button-close></d2l-navigation-button-close>
  <d2l-navigation-button text="Another button">Two Button</d2l-navigation-button>
  </div>
  </d2l-navigation-immersive>

package.json

@@ -17,7 +17,6 @@
  "/components",
  "/lang",
  "d2l-navigation-band.js",
- "d2l-navigation-button-close.js",
  "d2l-navigation-button-icon.js",
  "d2l-navigation-button-notification-icon.js",
  "d2l-navigation-button.js",

test/button.test.js

@@ -1,5 +1,4 @@
 import '../d2l-navigation-button.js';
-import '../d2l-navigation-button-close.js';
 import '../d2l-navigation-button-icon.js';
 import '../d2l-navigation-button-notification-icon.js';
 import { expect, fixture, html, oneEvent } from '@open-wc/testing';
@@ -39,23 +38,6 @@ describe('Buttons', () => {
 
  });
 
- describe('d2l-navigation-button-close', () => {
-
- describe('accessibility', () => {
- it('should pass all aXe tests', async() => {
- const el = await fixture(html`<d2l-navigation-button-close></d2l-navigation-button-close>`);
- await expect(el).to.be.accessible();
- });
- });
-
- describe('constructor', () => {
- it('should construct', () => {
- runConstructor('d2l-navigation-button-close');
- });
- });
-
- });
-
  describe('d2l-navigation-notification-icon', () => {
 
  describe('accessibility', () => {

test/button.visual-diff.html

@@ -9,7 +9,6 @@
  import '@brightspace-ui/core/components/icons/icon.js';
  import '@brightspace-ui/core/components/typography/typography.js';
  import '../../d2l-navigation-button.js';
- import '../../d2l-navigation-button-close.js';
  import '../../d2l-navigation-button-icon.js';
  import '../../d2l-navigation-button-notification-icon.js';
  import { forceFocusVisible } from '@brightspace-ui/core/helpers/focus.js';
@@ -23,9 +22,6 @@
  <div class="visual-diff">
  <d2l-navigation-button id="disabled" text="Classes" disabled><d2l-icon icon="tier3:classes"></d2l-icon></d2l-navigation-button>
  </div>
- <div class="visual-diff">
- <d2l-navigation-button-close id="close"></d2l-navigation-button-close>
- </div>
  <div class="visual-diff">
  <d2l-navigation-button-icon id="icon-text" icon="tier3:classes" text="Classes"></d2l-navigation-button-icon>
  </div>

test/button.visual-diff.js

@@ -27,7 +27,6 @@ describe('d2l-navigation-button', () => {
  [
  { category: 'default', tests: ['normal', 'hover', 'focus'] },
  { category: 'disabled', tests: ['normal', 'hover', 'focus'] },
- { category: 'close', tests: ['normal', 'hover', 'focus'] },
  { category: 'icon-text', tests: ['normal', 'hover', 'focus'] },
  { category: 'icon-text-disabled', tests: ['normal', 'hover', 'focus'] },
  { category: 'icon-text-hidden', rectSelector: 'icon-text-hidden-container', tests: ['normal', 'hover', 'focus'] },