Skip to content

Commit e9cfef2

Browse files
committed
Fix broken FeedbackDialog aria-describedby value
Was referencing a non-existent id
1 parent 439f274 commit e9cfef2

File tree

2 files changed

+14
-0
lines changed

2 files changed

+14
-0
lines changed

app/components/primer/open_project/feedback_dialog.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ class FeedbackDialog < Primer::Component
1616
# @param system_arguments [Hash] <%= link_to_system_arguments_docs %>
1717
renders_one :feedback_message, lambda { |icon_arguments: {}, **system_arguments|
1818
system_arguments[:border] = false
19+
system_arguments[:id] = "#{@system_arguments[:id]}-description"
20+
1921
Primer::OpenProject::FeedbackMessage.new(icon_arguments: icon_arguments, **system_arguments)
2022
}
2123

test/components/primer/open_project/feedback_dialog_test.rb

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,18 @@ def test_renders
1919
end
2020
end
2121

22+
def test_renders_aria_describedby
23+
render_inline(Primer::OpenProject::FeedbackDialog.new(title: "Success message")) do |dialog|
24+
dialog.with_feedback_message do |message|
25+
message.with_heading(tag: :h2) { "Success" }
26+
end
27+
end
28+
29+
dialog_labelledby_id = page.find_css("dialog").first.attributes["aria-describedby"].value
30+
feedback_message_id = page.find_css(".FeedbackMessage").first.attributes["id"].value
31+
assert_equal dialog_labelledby_id, feedback_message_id
32+
end
33+
2234
def test_renders_additional_details
2335
render_inline(Primer::OpenProject::FeedbackDialog.new(title: "Success message")) do |dialog|
2436
dialog.with_feedback_message do |message|

0 commit comments

Comments
 (0)