Skip to content

Commit 2b6f8d6

Browse files
committed
Change spinner for FeedbackMessgae
1 parent 2dac8b2 commit 2b6f8d6

5 files changed

Lines changed: 8 additions & 4 deletions

File tree

.changeset/tough-parts-repeat.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@openproject/primer-view-components': patch
3+
---
4+
5+
Change loading spinner for FeedbackMessage and FeedbackDialog

app/assets/images/loading_indicator.svg

Lines changed: 0 additions & 1 deletion
This file was deleted.

app/components/primer/open_project/feedback_message.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def initialize(icon_arguments: {}, loading: false, **system_arguments)
3939

4040
def before_render
4141
if @loading
42-
@blankslate.with_visual_image(src: asset_path("loading_indicator.svg"), alt: I18n.t(:label_loading))
42+
@blankslate.with_visual_spinner(size: :large)
4343
elsif @icon_arguments[:icon] != :none
4444
@blankslate.with_visual_icon(size: :medium, **@icon_arguments)
4545
end

test/components/primer/open_project/feedback_dialog_test.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ def test_renders_loading_spinner
8181

8282
assert_selector("dialog.FeedbackDialog") do
8383
assert_selector(".Overlay-body h2", text: "Ups, something went wrong")
84-
assert_selector("img.blankslate-image[src^='/assets/loading_indicator']")
84+
assert_selector("svg[data-test-selector='blankslate-spinner']")
8585
end
8686
end
8787
end

test/components/primer/open_project/feedback_message_test.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ def test_renders_loading_spinner
4141
end
4242

4343
assert_selector("h2", text: "Ups, something went wrong")
44-
assert_selector("img.blankslate-image[src^='/assets/loading_indicator']")
44+
assert_selector("svg[data-test-selector='blankslate-spinner']")
4545
end
4646

4747
def test_renders_secondary_action

0 commit comments

Comments
 (0)