From 2769b7730858753acafa053feb40104bf30c35fe Mon Sep 17 00:00:00 2001
From: web-padawan <iamkulykov@gmail.com>
Date: Sun, 6 Feb 2022 15:00:39 +0200
Subject: [PATCH] refactor: cleanup no longer used button roles

---
 .../src/vaadin-date-picker-overlay-content.js |  1 -
 .../date-picker/src/vaadin-date-picker.js     |  2 +-
 packages/date-picker/test/wai-aria.test.js    | 30 +------------------
 3 files changed, 2 insertions(+), 31 deletions(-)

diff --git a/packages/date-picker/src/vaadin-date-picker-overlay-content.js b/packages/date-picker/src/vaadin-date-picker-overlay-content.js
index 1ba002ca46..2128666527 100644
--- a/packages/date-picker/src/vaadin-date-picker-overlay-content.js
+++ b/packages/date-picker/src/vaadin-date-picker-overlay-content.js
@@ -199,7 +199,6 @@ class DatePickerOverlayContent extends ThemableMixin(DirMixin(PolymerElement)) {
           <template>
             <div
               part="year-number"
-              role="button"
               current$="[[_isCurrentYear(index)]]"
               selected$="[[_isSelectedYear(index, selectedDate)]]"
             >
diff --git a/packages/date-picker/src/vaadin-date-picker.js b/packages/date-picker/src/vaadin-date-picker.js
index f965c18a49..f48861ea77 100644
--- a/packages/date-picker/src/vaadin-date-picker.js
+++ b/packages/date-picker/src/vaadin-date-picker.js
@@ -145,7 +145,7 @@ class DatePicker extends DatePickerMixin(InputControlMixin(ThemableMixin(Element
           <slot name="prefix" slot="prefix"></slot>
           <slot name="input"></slot>
           <div id="clearButton" part="clear-button" slot="suffix" aria-hidden="true"></div>
-          <div part="toggle-button" slot="suffix" role="button" aria-hidden="true"></div>
+          <div part="toggle-button" slot="suffix" aria-hidden="true"></div>
         </vaadin-input-container>
 
         <div part="helper-text">
diff --git a/packages/date-picker/test/wai-aria.test.js b/packages/date-picker/test/wai-aria.test.js
index 089b13e585..6daf941aeb 100644
--- a/packages/date-picker/test/wai-aria.test.js
+++ b/packages/date-picker/test/wai-aria.test.js
@@ -6,11 +6,10 @@ import { activateScroller, getDefaultI18n, open } from './common.js';
 
 describe('WAI-ARIA', () => {
   describe('date picker', () => {
-    let datepicker, toggleButton, input, label, helper, error;
+    let datepicker, input, label, helper, error;
 
     beforeEach(() => {
       datepicker = fixtureSync(`<vaadin-date-picker helper-text="Week day" label="Date"></vaadin-date-picker>`);
-      toggleButton = datepicker.shadowRoot.querySelector('[part="toggle-button"]');
       input = datepicker.inputElement;
       label = datepicker.querySelector('[slot=label]');
       error = datepicker.querySelector('[slot=error-message]');
@@ -43,23 +42,6 @@ describe('WAI-ARIA', () => {
       expect(aria).to.include(error.id);
     });
 
-    it('should have button roles on buttons', () => {
-      // Indicate icon buttons as clickable. Especially helpful on touch devices.
-      expect(toggleButton.getAttribute('role')).to.equal('button');
-    });
-
-    // TODO: clarify if this is still needed
-    it.skip('should have label properties on buttons', () => {
-      // Give spoken names for the icon buttons.
-      expect(toggleButton.getAttribute('aria-label')).to.equal('Calendar');
-    });
-
-    // TODO: clarify if this is still needed
-    it.skip('should have label properties on buttons in correct locale', () => {
-      datepicker.set('i18n.calendar', 'kalenteri');
-      expect(toggleButton.getAttribute('aria-label')).to.equal('kalenteri');
-    });
-
     it('should set aria-haspopup attribute on the input', () => {
       expect(input.getAttribute('aria-haspopup')).to.equal('dialog');
     });
@@ -110,16 +92,6 @@ describe('WAI-ARIA', () => {
         expect(scroller.getAttribute('aria-hidden')).to.equal('true');
       });
 
-      it('should contain button role for years', () => {
-        // Indicate years as clickable.
-        const years = Array.from(yearScrollerContents).filter((el) => /\d+/.test(el.textContent));
-
-        expect(years).to.not.be.empty;
-        years.forEach((year) => {
-          expect(year.getAttribute('role')).to.equal('button');
-        });
-      });
-
       it('should have hidden state for dots', () => {
         // Do not speak dots between years.
         const dots = Array.from(yearScrollerContents).map((el) => el.nextElementSibling);