Merge pull request #4068 from EimantasBlazevicius/master

aria-selected true for range selected as well
Hacker0x01 · Jun 15, 2023 · 5b0000b · 5b0000b
2 parents 2800dbf + 32d5409
commit 5b0000b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/day.jsx b/src/day.jsx
@@ -357,7 +357,7 @@ export default class Day extends React.Component {
       role="option"
       aria-disabled={this.isDisabled()}
       aria-current={this.isCurrentDay() ? "date" : undefined}
-      aria-selected={this.isSelected()}
+      aria-selected={this.isSelected() || this.isInRange()}
     >
       {this.renderDayContents()}
     </div>

diff --git a/test/day_test.js b/test/day_test.js
@@ -74,6 +74,17 @@ describe("Day", () => {
           .getAttribute("aria-selected");
         expect(ariaSelected).to.equal("true");
       });
+
+      it('should set aria-selected attribute to "true" if previous and after days selected', () => {
+        const day = newDate();
+        const startDate = subDays(day, 1);
+        const endDate = addDays(day, 1);
+        const shallowDay = renderDay(day, { startDate, endDate });
+        const ariaSelected = mount(shallowDay.getElement())
+          .getDOMNode()
+          .getAttribute("aria-selected");
+        expect(ariaSelected).to.equal("true");
+      });
     });
 
     describe("if not selected", () => {