Problem
PR #794 introduced RatingSummaryWithSeparator which duplicates the query and render logic from the existing RatingSummary component. Any future rating display changes must be updated in two places.
Also: the separator dot between rating and view count is missing aria-hidden="true".
Solution
- Refactor
RatingSummaryWithSeparator to compose the existing RatingSummary (e.g. add an optional separator or children prop) instead of duplicating it
- Add
aria-hidden="true" to the decorative separator dot span
Acceptance Criteria
Branch
task/<issue>-rating-summary-dedup
Problem
PR #794 introduced
RatingSummaryWithSeparatorwhich duplicates the query and render logic from the existingRatingSummarycomponent. Any future rating display changes must be updated in two places.Also: the separator dot between rating and view count is missing
aria-hidden="true".Solution
RatingSummaryWithSeparatorto compose the existingRatingSummary(e.g. add an optionalseparatororchildrenprop) instead of duplicating itaria-hidden="true"to the decorative separator dot spanAcceptance Criteria
aria-hidden="true"Branch
task/<issue>-rating-summary-dedup