From f0d476eb590f9a974a86772babcf75a14ed7a53c Mon Sep 17 00:00:00 2001
From: niyap <38278268+niyap@users.noreply.github.com>
Date: Wed, 12 May 2021 09:15:24 +0300
Subject: [PATCH] fix(ui5-rating-indicator): Aria-readonly attribute is added
 when RatingIndicator in not editable (#3219)

---
 packages/main/src/RatingIndicator.hbs            | 2 +-
 packages/main/src/RatingIndicator.js             | 4 ++++
 packages/main/test/pages/RatingIndicator.html    | 2 +-
 packages/main/test/specs/RatingIndicator.spec.js | 5 +++++
 4 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/packages/main/src/RatingIndicator.hbs b/packages/main/src/RatingIndicator.hbs
index 110186de4a4e..1eb32582fb1a 100644
--- a/packages/main/src/RatingIndicator.hbs
+++ b/packages/main/src/RatingIndicator.hbs
@@ -6,7 +6,7 @@
 	aria-valuemax="{{maxValue}}"
 	aria-orientation="horizontal"
 	aria-disabled="{{_ariaDisabled}}"
-	?aria-readonly="{{readOnly}}"
+	aria-readonly="{{ariaReadonly}}"
 	tabindex="{{tabIndex}}"
 	@focusin="{{_onfocusin}}"
 	@focusout="{{_onfocusout}}"
diff --git a/packages/main/src/RatingIndicator.js b/packages/main/src/RatingIndicator.js
index dafa10d665c3..1cd5098f6965 100644
--- a/packages/main/src/RatingIndicator.js
+++ b/packages/main/src/RatingIndicator.js
@@ -294,6 +294,10 @@ class RatingIndicator extends UI5Element {
 	get _ariaDisabled() {
 		return this.disabled || undefined;
 	}
+
+	get ariaReadonly() {
+		return this.readonly ? "true" : undefined;
+	}
 }
 
 RatingIndicator.define();
diff --git a/packages/main/test/pages/RatingIndicator.html b/packages/main/test/pages/RatingIndicator.html
index f75e25ffd352..74d8ed45e105 100644
--- a/packages/main/test/pages/RatingIndicator.html
+++ b/packages/main/test/pages/RatingIndicator.html
@@ -47,7 +47,7 @@ <h3>test change event</h3>
 	<br>
 
 	<h3>readonly</h3>
-	<ui5-rating-indicator value="1" max-value="3" readonly></ui5-rating-indicator>
+	<ui5-rating-indicator id="rating-indicator-readonly" value="1" max-value="3" readonly></ui5-rating-indicator>
 	<br>
 	<br>
 	<br>
diff --git a/packages/main/test/specs/RatingIndicator.spec.js b/packages/main/test/specs/RatingIndicator.spec.js
index 7b385bd65b00..611901dcde0d 100644
--- a/packages/main/test/specs/RatingIndicator.spec.js
+++ b/packages/main/test/specs/RatingIndicator.spec.js
@@ -68,6 +68,8 @@ describe("Rating Indicator general interaction", () => {
 
 	it("Tests ACC attrs", () => {
 		const ratingIndicator = browser.$("#rating-indicator1").shadow$(".ui5-rating-indicator-root");
+		const ratingIndicatorReadOnly = browser.$("#rating-indicator-readonly").shadow$(".ui5-rating-indicator-root");
+
 		const TOOLTIP = "Rating";
 		const ARIA_LABEL = "Hello World";
 
@@ -76,5 +78,8 @@ describe("Rating Indicator general interaction", () => {
 
 		assert.strictEqual(ratingIndicator.getAttribute("title"), TOOLTIP,
 			"The default tooltip is displayed");
+
+		assert.notOk(ratingIndicator.getAttribute("aria-readonly"), "The aria-readonly attribute is not presented");
+		assert.strictEqual(ratingIndicatorReadOnly.getAttribute("aria-readonly"), 'true', "The aria-readonly attribute is presented");
 	});
 });